View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006980CMakeCPackpublic2008-05-07 10:562008-08-21 15:44
ReporterErik Sjölund 
Assigned ToBill Hoffman 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionunable to reproduce 
PlatformOSOS Version
Product VersionCMake-2-6 
Target VersionFixed in Version 
Summary0006980: CPACK_RPM_PACKAGE_ARCHITECTURE does not set the ARCH label in the generated rpm
DescriptionSetting

??SET(CPACK_RPM_PACKAGE_ARCHITECTURE "noarch")

doesn't set the architecture label of the generated rpm package to "noarch". In other words the output of
rpm -qp --queryformat '%{ARCH}\n' generated-rpm-file

should be "noarch" but is "i386" on my computer ( Ubuntu 8.04, i386-system ).


Additional InformationThe attached patch shows one way to fix the problem. But the patch was written without much knowhow of how cmake works internally.
TagsNo tags attached.
Attached Filespatch file icon buildarch.patch [^] (1,138 bytes) 2008-05-07 10:56 [Show Content]

 Relationships

  Notes
(0011732)
Eric NOULARD (developer)
2008-05-07 11:50

The patch is fine, but I may propose on which may be more
consistent with CPackRPM.cmake as a whole.

Sidenote to Bill or any CMake developer:
If (aka when) you have CPackRPM related bug
If you find it useful you may systematically
add me to the monitor list of the bug.

I would be pleased to help If I can.
(0011787)
Eric NOULARD (developer)
2008-05-10 07:06

I was wrong the patch from Erik is good
I wouldn't have done it in another.

We don't want to have
"Buildarch:" item if not enforced by the user,
rpmbuild would guess the appropriate host arch.
(0011810)
Erik Sjölund (reporter)
2008-05-12 04:44

A sidenote:

CPACK_RPM_PACKAGE_BUILDARCH

in the buildarch.patch is just a temporary variable. Maybe it should be called TMP_BUILDARCH or something like that ...
(0011828)
Eric NOULARD (developer)
2008-05-12 16:07

Yes that(s right we should reserve CPACK_RPM prefix to variable
which may be set by the user.
(0011969)
Bill Hoffman (manager)
2008-05-16 10:13

Is this fixed in CVS now?

 Issue History
Date Modified Username Field Change
2008-05-07 10:56 Erik Sjölund New Issue
2008-05-07 10:56 Erik Sjölund File Added: buildarch.patch
2008-05-07 11:50 Eric NOULARD Note Added: 0011732
2008-05-10 07:06 Eric NOULARD Note Added: 0011787
2008-05-12 04:44 Erik Sjölund Note Added: 0011810
2008-05-12 16:07 Eric NOULARD Note Added: 0011828
2008-05-16 10:13 Bill Hoffman Note Added: 0011969
2008-05-16 10:13 Bill Hoffman Status new => assigned
2008-05-16 10:13 Bill Hoffman Assigned To => Bill Hoffman
2008-08-21 15:44 Bill Hoffman Status assigned => closed
2008-08-21 15:44 Bill Hoffman Resolution open => unable to reproduce


Copyright © 2000 - 2018 MantisBT Team