View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0009654CMakeCPackpublic2009-10-03 15:422009-11-22 13:31
ReporterDavid Moreau 
Assigned ToEric NOULARD 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake-2-8 
Target VersionFixed in VersionCMake-2-8 
Summary0009654: [CPackRPM] %files section in spec file shall not contain all directories
DescriptionCPackRPM generates a spec file with a %files sections which contains all files of the projets but also all the directories. Directories that are not part of the project should not been include in this section (can conflict with other packages).

For example, if a project is storing a script in /etc/init.d/, the RPM should looks like :
rpm -qpl test.rpm
/etc/init.d/script
and NOT
/etc
/etc/init.d/
/etc/init.d/script

The patch included is based on CMake 2.6.4
TagsNo tags attached.
Attached Filespatch file icon CPackRPM.cmake.patch [^] (1,370 bytes) 2009-10-03 15:42 [Show Content]
patch file icon bug-9654-erk1.patch [^] (1,658 bytes) 2009-10-03 17:26 [Show Content]
patch file icon bug-9654-kami.patch [^] (1,694 bytes) 2009-10-07 04:33 [Show Content]

 Relationships
related to 0009927closedEric NOULARD CPack RPM generator does not seem to install symlinks 
related to 0012305closedEric NOULARD RPM should include directories 
related to 0012542closedEric NOULARD Uninstalling a cpack-generated rpm leaves empty directories behind 

  Notes
(0017935)
Eric NOULARD (developer)
2009-10-03 17:30

This bug is a sibling for CMake-2-6:
http://public.kitware.com/Bug/view.php?id=9653 [^]

The second patch has been made against the CMake-2-8 branch.
The patch suppressed the %install section added by the primary patch
because I think it is not necessary.
I even think we should not use the %install spec file section at all.
It should not be necessary because cpack did already do that for us,
moreover using it will make rpmbuild fail when trying to %install.
(0017981)
Alex Neundorf (developer)
2009-10-06 14:09

So this bug here is for CMake 2.8, and 0009653 is the same but for CMake 2.6 ?

+# WE MUST DO ANYTHING in those steps because they

Is this correct in the patch or should it be "WE MUST NOT DO ANYTHING" ?

Alex
(0017982)
Eric NOULARD (developer)
2009-10-06 14:15

Yes the bugs are siblings bugs for 2.6 and 2.8 I think kami
did file 2 bugs because mantis was requiring CMake version.

The right sentence is:
WE MUST NOT DO ANYTHING

you are right.

The meaning is we shouldn't do anything in the %install step of the spec
file because CPack should already have done his work for us.
(0017984)
David Moreau (reporter)
2009-10-06 14:19

>I think kami
>did file 2 bugs because mantis was requiring CMake version.
Right!
(0017986)
Alex Neundorf (developer)
2009-10-06 14:27

Eric, can you please attach a patch against current cvs HEAD which is ready for committing, including your comments about the %install section ?

Thanks
Alex
(0017987)
Alex Neundorf (developer)
2009-10-06 14:28

And can you also have a look at 0009584 ?

Thanks
Alex
(0018005)
David Moreau (reporter)
2009-10-07 04:36

I have just uploaded the corresponding patch (same as bug-9654-erk1.patch but with the correct comment and based on the CVS Head)
Hope it's useful

Kami
(0018006)
Eric NOULARD (developer)
2009-10-07 05:10

Thank you Kami that is useful.
I'll test kami's patch tonight, I'm not able to do it right now.

To Alex:
I did add monitoring on 0009584,
I will add simple comment right now
and will look deeper into it tonight too.
(0018013)
Eric NOULARD (developer)
2009-10-07 14:42

The latest patch from kami is OK for me too.
As far as I'm concerned it could be applied.
(0018369)
Eric NOULARD (developer)
2009-11-09 13:47

I'm preparing to check-in this patch to CVS HEAD.
Local tests currently running.
(0018370)
Eric NOULARD (developer)
2009-11-09 14:07

Local Tests OK.
Just committed to CVS HEAD.
(0018375)
Alex Neundorf (developer)
2009-11-09 15:11

I can't find you in the list of people I can assign this bug to, probably you don't have enough privileges in Mantis.
Can you ask Bill to elevate you to a level so I can assign this bug to you (so you can close it then) ?

Alex
(0018536)
Eric NOULARD (developer)
2009-11-22 13:31

may be include in 2.8.1 with the related bug fix

 Issue History
Date Modified Username Field Change
2009-10-03 15:42 David Moreau New Issue
2009-10-03 15:42 David Moreau File Added: CPackRPM.cmake.patch
2009-10-03 17:26 Eric NOULARD File Added: bug-9654-erk1.patch
2009-10-03 17:30 Eric NOULARD Note Added: 0017935
2009-10-06 14:09 Alex Neundorf Note Added: 0017981
2009-10-06 14:15 Eric NOULARD Note Added: 0017982
2009-10-06 14:19 David Moreau Note Added: 0017984
2009-10-06 14:27 Alex Neundorf Note Added: 0017986
2009-10-06 14:28 Alex Neundorf Note Added: 0017987
2009-10-07 04:33 David Moreau File Added: bug-9654-kami.patch
2009-10-07 04:36 David Moreau Note Added: 0018005
2009-10-07 05:10 Eric NOULARD Note Added: 0018006
2009-10-07 14:42 Eric NOULARD Note Added: 0018013
2009-10-07 15:56 Alex Neundorf Status new => assigned
2009-10-07 15:56 Alex Neundorf Assigned To => Alex Neundorf
2009-11-09 13:47 Eric NOULARD Note Added: 0018369
2009-11-09 14:07 Eric NOULARD Note Added: 0018370
2009-11-09 15:11 Alex Neundorf Note Added: 0018375
2009-11-22 08:20 Alex Neundorf Assigned To Alex Neundorf =>
2009-11-22 08:20 Alex Neundorf Assigned To => Alex Neundorf
2009-11-22 09:31 Alex Neundorf Assigned To Alex Neundorf => Eric NOULARD
2009-11-22 12:45 Eric NOULARD Relationship added related to 0009927
2009-11-22 13:31 Eric NOULARD Note Added: 0018536
2009-11-22 13:31 Eric NOULARD Status assigned => closed
2009-11-22 13:31 Eric NOULARD Resolution open => fixed
2009-11-22 13:31 Eric NOULARD Fixed in Version => CMake-2-8
2011-06-25 03:51 Eric NOULARD Relationship added related to 0012305
2011-10-27 09:33 Alex Neundorf Relationship added related to 0012542


Copyright © 2000 - 2018 MantisBT Team