View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
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 ). | ||||||||
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. | ||||||||
Attached Files | buildarch.patch [^] (1,138 bytes) 2008-05-07 10:56 [Show Content] | ||||||||
Relationships | |
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? |
Notes |
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 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |