Bug 2555

Summary: scourge needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel)
Product: Mageia Reporter: Philippe Didier <philippedidier>
Component: RPM PackagesAssignee: Mageia Bug Squad <bugsquad>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: lists.jjorge, philippedidier
Version: CauldronKeywords: Junior_job
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: scourge CVE:
Status comment:

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