Bug 2555 - scourge needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel)
Summary: scourge needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel)
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords: Junior_job
Depends on:
Blocks:
 
Reported: 2011-08-28 14:08 CEST by Philippe Didier
Modified: 2011-09-22 22:16 CEST (History)
2 users (show)

See Also:
Source RPM: scourge
CVE:
Status comment:


Attachments

Description Philippe Didier 2011-08-28 14:08:42 CEST
After some discussion on dev mailing list, in order to switch from GLUT
(mesaglut) to freeglut in cauldron : this package needs to have its spec file
modified and need to be rebuilt with a modified BuilRequire .

see:
 https://bugs.mageia.org/show_bug.cgi?id=2412


This package needs mesa-common-devel as BuildRequire.... 
we can't know easily if it needs each of the components of this meta package
and so if it needs or not mesaglut-devel! and if it may be built with
freeglut-devel instead of mesaglut-devel...


Maybe in a first time is it necessary to write in extenso the BuildRequires in
its spec file
BR mesaegl-devel
BR mesagl-devel
BR mesaglu-devel
BR mesaglut-devel
BR mesaglw-devel
BR mesaglesv1-devel
BR mesaglesv2-devel

instead of BR mesa-common-devel

and then try to build without
mesaglut-devel (sometimes no more needed!)
...
and if necessary try to build 
with freeglut-devel
Philippe Didier 2011-08-28 14:09:58 CEST

CC: (none) => lists.jjorge, philippedidier

Manuel Hiebel 2011-08-28 15:16:45 CEST

Keywords: (none) => Junior_job

Jani Välimaa 2011-08-29 19:48:05 CEST

Summary: this package needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel) => scourge needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel)

Comment 1 José Jorge 2011-08-31 18:36:01 CEST
Change done : it has built with freeglut-devel. Will be pushed as soon as my mentor comes back.

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 2 Philippe Didier 2011-09-03 04:02:01 CEST
Not shure it may be built with your modification on the Build System :
you substituted freeglut-devel to mesa-common-devel ...

in fact you ought to remove mesa-common-devel (which is a meta package)
and add 6 of its components
BR mesaegl-devel
BR mesagl-devel
BR mesaglu-devel
BR mesaglw-devel
BR mesaglesv1-devel
BR mesaglesv2-devel
and 
BR freeglut-devel (which replaces the 7th component : mesaglut-devel !)


If you don't have a real Build System you can't really verify whether all the BuildRequires are in the spec :
You can build a rpm on your computer if the needed foo-devel-rpm was already installed : the %configure process will find the headers... and %make %install will work... on your computer.
Even if the spec file doesn't have the line "BuildRequires foo-devel" you can build the package on your own computer !

But that won't work on the Mageia BS ! The build system is "empty", it loads the required *-devel.rpms listed in the spec file as BR  and then %config %make and %install
if a BR is missing in the spec file, the %config process aborts on the BS... (failure)

If you push it as it is, you will certainly get a failure...

Nevertheless : you can verify this on your computer :
1) uninstall mesa-common-devel 
2) Then try to build scourge with your spec file (which has not this BR)
3) No missing BuildRequire !
4) but the %configure process will certainly complain for missing gl and glu library
5) Modify the spec file the way I suggest 
6) try to build
7) Missing BuildRequires !
7) add the missing devel-rpms
8) Build => OK


In fact, after  a quick glance at the configure.in file of the source, it seems that only mesagl-devel and mesaglu-devel are needed (seems not to need mesaegl, mesaglw, mesaglesv1, mesaglesv2, and not freeglut no more...)

You may discuss it with your mentor

Status: RESOLVED => REOPENED
Resolution: FIXED => (none)

Comment 3 José Jorge 2011-09-03 16:51:28 CEST
Yes but no : sdl-devel depends on mesaglu1-devel.
Then freeglut-devel depends on mesagl1-devel. So it ends up with

lib64mesaglu1-devel-7.11-2.mga2.tainted
lib64mesagl1-devel-7.11-2.mga2.tainted

And configure is happy ;-) Still, you are right for freeglut-devel : I tried to remove it and it compiles, so it only BR mesagl-devel at the end.

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED

Comment 4 Philippe Didier 2011-09-22 20:40:25 CEST
Hi José !

I reopen this bug report because  it seems that scourge has not been pushed in the BS...
:(

Status: RESOLVED => REOPENED
Resolution: FIXED => (none)

Comment 5 José Jorge 2011-09-22 22:16:19 CEST
Sorry : my mentor was bored with all that, and bumped me to packager ;-)
Then I forgot to submit this one. Just done.

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.