Bug 2556

Summary:	torcs needs a spec cleaning (switch BR from mesaglut-devel to freeglut-devel)
Product:	Mageia	Reporter:	Philippe Didier <philippedidier>
Component:	RPM Packages	Assignee:	Mageia Bug Squad <bugsquad>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	cazzaniga.sandro, philippedidier, stormi-mageia
Version:	Cauldron	Keywords:	Junior_job
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	torcs	CVE:
Status comment:

Description Philippe Didier 2011-08-28 14:10:49 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:11:21 CEST

CC: (none) => philippedidier, stormi

Comment 1 Samuel Verschelde 2011-08-28 14:40:53 CEST

My commit to this package only was a description fix, the packager who imported it would be a better match I think (it was Kharec).

Assignee: bugsquad => cazzaniga.sandro

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

Keywords: (none) => Junior_job

Jani Välimaa 2011-08-29 19:47:30 CEST

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

Comment 2 Sandro CAZZANIGA 2011-08-30 11:09:29 CEST

Yep, I'm doing it today.

Comment 3 Philippe Didier 2011-09-03 02:43:29 CEST

Sorry, but the modification of the spec file seems not to bring what is asked :

Line 18 you substitute freeglut-devel to mesaglu-devel (it's not mesaglut-devel...)

Indeed the mandriva spec file was not very clean :
in line 18 BR mesaglu-devel  was a  BuildRequire :  but in fact it's useless :
Line 19 BuildRequires asked mesa-common-devel !
that means, in extenso 7 different BR:
BR mesaegl-devel
BR mesagl-devel
BR mesaglu-devel !!!!
BR mesaglut-devel
BR mesaglw-devel
BR mesaglesv1-devel
BR mesaglesv2-devel

As you can see mesaglu-devel was twice a BR !

and mesaglut-devel is always required after your modification !


As you already modified line 18 (BR freeglut-devel which is a substitute to mesaglut-devel)

you may suppress in line 19 mesa-common-devel
and add these 6 BR 
BR mesaegl-devel
BR mesagl-devel
BR mesaglu-devel 
BR mesaglw-devel
BR mesaglesv1-devel
BR mesaglesv2-devel

be careful to write mesaglu and not mesaglut !

Thanks

If it's not clear enough don't hesitate to write a new comment !

Sandro CAZZANIGA 2011-09-03 13:43:53 CEST

Assignee: cazzaniga.sandro => bugsquad

Comment 4 Sandro CAZZANIGA 2011-09-04 11:22:46 CEST

I've commited my changes, wait for an experimented packager for check it.

CC: (none) => cazzaniga.sandro

Comment 5 Philippe Didier 2011-09-15 19:18:21 CEST

Funda did some checks and several BR modifications... before being able to build with new png-devel

But it is resolved and fixed

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