From Fedora Project Wiki
(Initial Change proposal)
 
 
(14 intermediate revisions by 3 users not shown)
Line 1: Line 1:
{{admon/important | Comments and Explanations | The page source contains comments providing guidance to fill out each section. They are invisible when viewing this page. To read it, choose the "view source" link.<br/> '''Copy the source to a ''new page'' before making changes!  DO NOT EDIT THIS TEMPLATE FOR YOUR CHANGE PROPOSAL.'''}}
 
 
{{admon/tip | Guidance | For details on how to fill out this form, see the [https://docs.fedoraproject.org/en-US/program_management/changes_guide/ documentation].}}
 
 
 
<!-- The actual name of your proposed change page should look something like: Changes/Your_Change_Proposal_Name.  This keeps all change proposals in the same namespace -->
 
<!-- The actual name of your proposed change page should look something like: Changes/Your_Change_Proposal_Name.  This keeps all change proposals in the same namespace -->
  
Line 25: Line 21:
  
 
== Current status ==
 
== Current status ==
[[Category:ChangePageIncomplete]]
+
[[Category:ChangeAcceptedF35]]
 
<!-- When your change proposal page is completed and ready for review and announcement -->
 
<!-- When your change proposal page is completed and ready for review and announcement -->
 
<!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler -->
 
<!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler -->
Line 43: Line 39:
 
ON_QA -> change is fully code complete
 
ON_QA -> change is fully code complete
 
-->
 
-->
* FESCo issue: <will be assigned by the Wrangler>
+
* FESCo issue: [https://pagure.io/fesco/issue/2614 2614]
* Tracker bug: <will be assigned by the Wrangler>
+
* Tracker bug: [https://bugzilla.redhat.com/show_bug.cgi?id=1967205 #1967205]
* Release notes tracker: <will be assigned by the Wrangler>
+
* Release notes tracker: [https://pagure.io/fedora-docs/release-notes/issue/701 #701]
  
 
== Detailed Description ==
 
== Detailed Description ==
Line 52: Line 48:
 
== Feedback ==
 
== Feedback ==
 
<!-- Summarize the feedback from the community and address why you chose not to accept proposed alternatives. This section is optional for all change proposals but is strongly suggested. Incorporating feedback here as it is raised gives FESCo a clearer view of your proposal and leaves a good record for the future. If you get no feedback, that is useful to note in this section as well. For innovative or possibly controversial ideas, consider collecting feedback before you file the change proposal. -->
 
<!-- Summarize the feedback from the community and address why you chose not to accept proposed alternatives. This section is optional for all change proposals but is strongly suggested. Incorporating feedback here as it is raised gives FESCo a clearer view of your proposal and leaves a good record for the future. If you get no feedback, that is useful to note in this section as well. For innovative or possibly controversial ideas, consider collecting feedback before you file the change proposal. -->
 +
* [https://bugzilla.redhat.com/show_bug.cgi?id=1968508#c4 sdl-config was broken] until sdl12-compat-0.0.1~git.20210609.efe9791-1.fc35
 +
* [https://bugzilla.redhat.com/show_bug.cgi?id=1967544 perl-SDL would fail 10 of 58 tests] if it checked for SDL initialization properly
  
 
== Benefit to Fedora ==
 
== Benefit to Fedora ==
Line 66: Line 64:
 
<!-- What work do the feature owners have to accomplish to complete the feature in time for release?  Is it a large change affecting many parts of the distribution or is it a very isolated change? What are those changes?-->
 
<!-- What work do the feature owners have to accomplish to complete the feature in time for release?  Is it a large change affecting many parts of the distribution or is it a very isolated change? What are those changes?-->
 
** Package [https://github.com/libsdl-org/sdl12-compat libsdl12-compat] ([https://bugzilla.redhat.com/show_bug.cgi?id=1960960 RH#1960960])
 
** Package [https://github.com/libsdl-org/sdl12-compat libsdl12-compat] ([https://bugzilla.redhat.com/show_bug.cgi?id=1960960 RH#1960960])
** Adjust {{package|SDL}} to not ship the main library package and use the one from <code>libsdl12-compat</code>
+
** Retire {{package|SDL}} completely, as [https://github.com/libsdl-org/sdl12-compat/issues/34 replacement development headers are available]
  
 
* Other developers: N/A <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
* Other developers: N/A <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
Line 105: Line 103:
 
-->
 
-->
  
1. Swap SDL for sdl12-compat: <code>dnf swap SDL sdl12-compat</code>
+
1. Swap <code>SDL</code> for <code>sdl12-compat</code>: <code>dnf swap SDL sdl12-compat</code>
  
2. Run something that uses SDL 1.2 like {{package|quake3}} and see that it works.
+
2. Run something that uses SDL 1.2 like {{package|icebreaker}} and see that it works.
  
 
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
Line 124: Line 122:
  
 
<!-- If you cannot complete your feature by the final development freeze, what is the backup plan?  This might be as simple as "Revert the shipped configuration".  Or it might not (e.g. rebuilding a number of dependent packages).  If you feature is not completed in time we want to assure others that other parts of Fedora will not be in jeopardy.  -->
 
<!-- If you cannot complete your feature by the final development freeze, what is the backup plan?  This might be as simple as "Revert the shipped configuration".  Or it might not (e.g. rebuilding a number of dependent packages).  If you feature is not completed in time we want to assure others that other parts of Fedora will not be in jeopardy.  -->
* Contingency mechanism: Restore the <code>SDL</code> package in {{package|SDL}}  <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
+
* Contingency mechanism: Unretire {{package|SDL}}. <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
<!-- When is the last time the contingency mechanism can be put in place?  This will typically be the beta freeze. -->
 
<!-- When is the last time the contingency mechanism can be put in place?  This will typically be the beta freeze. -->
 
* Contingency deadline: Final Freeze  <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
* Contingency deadline: Final Freeze  <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
<!-- Does finishing this feature block the release, or can we ship with the feature in incomplete state? -->
 
<!-- Does finishing this feature block the release, or can we ship with the feature in incomplete state? -->
 
* Blocks release? N/A (not a System Wide Change), Yes/No <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
* Blocks release? N/A (not a System Wide Change), Yes/No <!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
 
  
 
== Documentation ==
 
== Documentation ==
<!-- Is there upstream documentation on this change, or notes you have written yourself?  Link to that material here so other interested developers can get involved. -->
+
* [https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/VO33YWAYCQXAX5TZSWFE5Y5PNN7F7XRA/ A discussion on devel mailing list]
 
+
* [https://docs.google.com/spreadsheets/d/1u8Rq3LVQYYgu28sBuxrZ371QolbiZu5z_LjENc4ddZs/ Upstream's compatibility list]
<!-- REQUIRED FOR SYSTEM WIDE CHANGES -->
+
* [https://bugzilla.redhat.com/show_bug.cgi?id=1960960 sdl12-compat package review]
N/A (not a System Wide Change)
 
  
 
== Release Notes ==
 
== Release Notes ==

Latest revision as of 14:04, 10 June 2021


Replace SDL 1.2 with sdl12-compat using SDL 2.0

Summary

This Change proposes to replace SDL 1.2 with sdl12-compat, which uses SDL 2.0.

Owner


Current status

Detailed Description

SDL 1.2 development ended long ago, with SDL 2.0 replacing it. However, many older games still use SDL 1.2 and cannot change to SDL 2.0. In order to help move SDL 1.2 games into the modern world, let's replace SDL 1.2 with sdl12-compat, which uses SDL 2.0.

Feedback

Benefit to Fedora

Switching SDL 1.2 powered games to use sdl12-compat offers significant advantages:

  • Automatic support for Wayland with SDL 2.0.16+
  • Native support for PipeWire for audio
  • Massively improved support for inputs (including gamepads)

Ultimately, SDL 2.0 is actively maintained and developed. We want applications that use SDL to use an actively maintained codebase.

Scope

  • Other developers: N/A
  • Policies and guidelines: N/A (not needed for this Change)
  • Trademark approval: N/A (not needed for this Change)
  • Alignment with Objectives: N/A

Upgrade/compatibility impact

The SDL package would be transparently upgraded to libsdl12-compat package and games using it should just transparently start using SDL 2.0.


How To Test

1. Swap SDL for sdl12-compat: dnf swap SDL sdl12-compat

2. Run something that uses SDL 1.2 like Package-x-generic-16.pngicebreaker and see that it works.


User Experience

There shouldn't be a noticeable user impact, other than possibly a smoother experience because applications are using SDL 2.0.

Dependencies

Contingency Plan

  • Contingency mechanism: Unretire Package-x-generic-16.pngSDL.
  • Contingency deadline: Final Freeze
  • Blocks release? N/A (not a System Wide Change), Yes/No

Documentation

Release Notes

Games that use SDL 1.2 will now transparently use SDL 2.0 through the sdl12-compat package. This makes it so applications that historically used SDL 1.2 now use SDL 2.0.