View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0014769 | CMake | CPack | public | 2014-02-24 09:04 | 2015-04-06 09:07 | ||||
Reporter | Javier | ||||||||
Assigned To | |||||||||
Priority | normal | Severity | feature | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | Linux | OS Version | ||||||
Product Version | CMake 2.8.12.2 | ||||||||
Target Version | CMake 3.2 | Fixed in Version | CMake 3.2 | ||||||
Summary | 0014769: CPackRPM: need CPACK_RPM_<Component>_PACKAGE_<PRE/POST>_REQUIRES | ||||||||
Description | We need to support the addition of PRE and POST Requires per component. For instance: CPACK_RPM_COMPONENTA_PACKAGE_PRE_REQUIRES="postgres >= 1.0" Should generate this header: Requires(pre): postgres >= 1.0 | ||||||||
Additional Information | Here is the RPM documentation link: http://www.rpm.org/wiki/PackagerDocs/Dependencies#ScriptletDependencies [^] | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | pre_post_rpm_install.patch [^] (2,638 bytes) 2014-10-07 13:37 [Show Content] | ||||||||
Relationships | |
Relationships |
Notes | |
(0037001) Evgeny Kalishenko (reporter) 2014-10-07 13:40 |
Hi, I've created the patch that seems to solve the problem, it's attached. Should I do the whole stages from http://www.cmake.org/Wiki/CMake/Git/Develop [^] or it's so small that can be reviewed and applied by CPack developer? |
(0037006) Brad King (manager) 2014-10-08 11:20 |
Re 0014769:0037001: Thanks for working on a patch. The best contribution path is documented in CONTRIBUTING.rst at the top of the CMake source tree in Git. Please read that and send a "git format-patch"-formatted patch to the mailing list: http://www.cmake.org/mailman/listinfo/cmake-developers [^] That will get a broader audience for review than the issue tracker. |
(0037253) Evgeny Kalishenko (reporter) 2014-11-21 16:22 |
I think the issue can be resolved after approving the patch http://www.cmake.org/pipermail/cmake-commits/2014-October/020907.html [^] |
(0037273) Brad King (manager) 2014-11-25 09:11 |
Re 0014769:0037253: Yes, thanks. Those were committed here: CPackRPM: Support pre(post) install script requirements http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=add4e50d [^] CPackRPM: Support PREUN and POSTUN requirements http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9ed546ff [^] |
(0038446) Robert Maynard (manager) 2015-04-06 09:07 |
Closing resolved issues that have not been updated in more than 4 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2014-02-24 09:04 | Javier | New Issue | |
2014-10-07 13:37 | Evgeny Kalishenko | File Added: pre_post_rpm_install.patch | |
2014-10-07 13:40 | Evgeny Kalishenko | Note Added: 0037001 | |
2014-10-08 11:20 | Brad King | Note Added: 0037006 | |
2014-11-21 16:22 | Evgeny Kalishenko | Note Added: 0037253 | |
2014-11-25 09:11 | Brad King | Note Added: 0037273 | |
2014-11-25 09:11 | Brad King | Status | new => resolved |
2014-11-25 09:11 | Brad King | Resolution | open => fixed |
2014-11-25 09:11 | Brad King | Fixed in Version | => CMake 3.2 |
2014-11-25 09:11 | Brad King | Target Version | => CMake 3.2 |
2015-04-06 09:07 | Robert Maynard | Note Added: 0038446 | |
2015-04-06 09:07 | Robert Maynard | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |