| 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 | |||||||||
| 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 |