MantisBT - CMake |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0006980 | CMake | CPack | public | 2008-05-07 10:56 | 2008-08-21 15:44 |
|
| Reporter | Erik Sjölund | |
| Assigned To | Bill Hoffman | |
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | unable to reproduce | |
| Platform | | OS | | OS Version | |
| Product Version | CMake-2-6 | |
| Target Version | | Fixed in Version | | |
|
| Summary | 0006980: CPACK_RPM_PACKAGE_ARCHITECTURE does not set the ARCH label in the generated rpm |
| Description | Setting
??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 ).
|
| Steps To Reproduce | |
| Additional Information | The attached patch shows one way to fix the problem. But the patch was written without much knowhow of how cmake works internally. |
| Tags | No tags attached. |
| Relationships | |
| Attached Files |  buildarch.patch (1,138) 2008-05-07 10:56
https://public.kitware.com/Bug/file/1447/buildarch.patch |
|
| 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 |
|
Notes |
|
|
(0011732)
|
|
Eric NOULARD
|
|
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
|
|
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
|
|
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
|
|
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
|
|
2008-05-16 10:13
|
|
|
Is this fixed in CVS now? |
|