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)

SDL_RPM.png (74.1 KB ) - added by martini 3 years ago.

Download all attachments as: .zip

Change History (5)

by martini, 3 years ago

Attachment: SDL_RPM.png added

comment:1 by Silvan Scherrer, 3 years ago

Component: rpmspec files
Priority: trivialFeedback Pending

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.

comment:2 by martini, 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 Silvan Scherrer, 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 Silvan Scherrer, 3 years ago

Resolution: invalid
Status: newclosed

as no one stepped further to have some payment to overcome this all I resolve it as invalid.

Note: See TracTickets for help on using tickets.