Opened 3 years ago
Closed 3 years ago
#351 closed defect (invalid)
There are two SDL 1.x packages (SDL and sdl)
Reported by: | martini | Owned by: | |
---|---|---|---|
Priority: | Feedback Pending | Milestone: | |
Component: | spec files | Version: | |
Severity: | low | Keywords: | sdl |
Cc: |
Description
Hi
There are two SDL 1.x packages on the RPM. One is SDL and other sdl. It seems that the upper case is the updated one.
I guess that the lower case one should be removed to avoid confusion with the users.
Regards
Attachments (1)
Change History (5)
by , 3 years ago
Attachment: | SDL_RPM.png added |
---|
comment:1 by , 3 years ago
Component: | rpm → spec files |
---|---|
Priority: | trivial → Feedback Pending |
comment:2 by , 3 years ago
Hi
I checked the specs on the rpm SVN with grep and found:
[C:\home\downloads\svn\rpm\spec]grep -r "Requires: sdl" |more trunk/SPECS/sdl_gfx.spec:BuildRequires: sdl-devel trunk/SPECS/sdl_gfx.spec:Requires: sdl-devel trunk/SPECS/sdl_image.spec:BuildRequires: sdl-devel trunk/SPECS/sdl_image.spec:Requires: sdl-devel trunk/SPECS/sdl_mixer.spec:Requires: sdl-devel trunk/SPECS/sdl_net.spec:Requires: sdl trunk/SPECS/sdl_sound.spec:BuildRequires: sdl-devel libmikmod-devel libogg-devel libsmpeg-devel libvorbis-devel libflac-devel
Also
[C:\home\downloads\svn\rpm\spec]grep -r "sdl-dev" |more trunk/SPECS/sdl_gfx.spec:BuildRequires: sdl-devel trunk/SPECS/sdl_gfx.spec:Requires: sdl-devel trunk/SPECS/sdl_image.spec:BuildRequires: sdl-devel trunk/SPECS/sdl_image.spec:Requires: sdl-devel trunk/SPECS/sdl_mixer.spec:Requires: sdl-devel trunk/SPECS/sdl_sound.spec:BuildRequires: sdl-devel libmikmod-devel libogg-devel libsmpeg-devel libvorbis-devel libflac-devel
It seems that the only ones requiring "sdl" (lowercase) are the rest of sdl components that are already replaced with the SDL in uppercase.
I also did the "grep -r "sdl"" search and even that the result was longer, only the sdl packages did a direct reference to "sdl", the rest was other *sdl* text combined things not related.
For the moment I think it will safe to remove the "sdl" and "sdl-*" lowercase packages.
Regards
comment:3 by , 3 years ago
I dont know what you tried but your assumption is wrong.
sdl_gfx needs sdl-devel. sdl_net needs sdl. So both will not work when the sdl are removed. As I said all would need a change to work proper. If you insist in removing then you need one who changes the wrong ones. We at bww will not do it, as sdl 1.x is outdated. Sure we will look at it if a price tag is attached.
comment:4 by , 3 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
as no one stepped further to have some payment to overcome this all I resolve it as invalid.
this is a non issue, when the packer uses the right SDL requires. Those libs are not meant to install by hand. And unfortunately there are still wrong spec files around, which requires sdl. and it seems no one cares about them. for sure we at bww don't care about hose wrong aones, as they are not from us. feel free to check the wrong specs and ask the author to fix it. when this all is done we might remove the lower case ones.