| View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
| 0014649 | CMake | CMake | public | 2013-12-16 02:35 | 2014-06-02 08:37 | ||||
| Reporter | Arunmozhi | ||||||||
| Assigned To | Stephen Kelly | ||||||||
| Priority | normal | Severity | minor | Reproducibility | always | ||||
| Status | closed | Resolution | no change required | ||||||
| Platform | OS | OS Version | |||||||
| Product Version | |||||||||
| Target Version | Fixed in Version | ||||||||
| Summary | 0014649: INTERFACE_LIBRARY does not allow defined properties | ||||||||
| Description | An INTERFACE library allows only whitelisted properties and not user-defined properties. Only user can decide if the defined property can be applicable for an INTERFACE library or not. | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files |  CMakeLists.txt [^] (298 bytes) 2013-12-16 02:35 [Show Content]
| ||||||||
| Relationships | |
| Relationships |
| Notes | |
|
(0034781) Stephen Kelly (developer) 2013-12-16 06:35 edited on: 2013-12-16 06:36 |
You need to be more specific. Post some code which shows why the property on the INTERFACE library needs to be set, and no alternative is sensible. http://sscce.org/ [^] |
|
(0034782) Arunmozhi (reporter) 2013-12-16 08:00 |
User defined properties can be used for any reason to store some target specific information globally. I simply do not see a reason to block it for a specific target type. If you really looking for a usage requirement, I store the type of some custom targets in a specific property. I assumed that when I query this type property from any target (other than those custom targets), it shall return empty string. But I saw error when I queried the property from a interface_library target and hence raised this bug. There will definitely be alternative way of doing things, but I see this issue more from the theoretical and compatibility point of view rather than practical reasons. The module which I wrote for previous versions of cmake now gives error due to this. This module generates Android.mk for our project, so definitely not for a simple usage of cmake. |
|
(0034859) Stephen Kelly (developer) 2013-12-23 11:17 |
> The module which I wrote for previous versions of cmake now gives error due to this. This is acceptable. The module pre-dates the INTERFACE_LIBRARY type. Another module might attempt to read the LOCATION of an INTERFACE_LIBRARY target for example, which makes no sense at all. It just so happens that your module doesn't otherwise trip up on INTERFACE_LIBRARY targets, but it is a new type of object with new semantics and it is reasonable to require new handling for it. |
|
(0036065) Robert Maynard (manager) 2014-06-02 08:37 |
Closing resolved issues that have not been updated in more than 4 months. |
| Notes |
| Issue History | |||
| Date Modified | Username | Field | Change |
| 2013-12-16 02:35 | Arunmozhi | New Issue | |
| 2013-12-16 02:35 | Arunmozhi | File Added: CMakeLists.txt | |
| 2013-12-16 06:35 | Stephen Kelly | Note Added: 0034781 | |
| 2013-12-16 06:36 | Stephen Kelly | Note Edited: 0034781 | |
| 2013-12-16 08:00 | Arunmozhi | Note Added: 0034782 | |
| 2013-12-23 11:17 | Stephen Kelly | Note Added: 0034859 | |
| 2013-12-23 11:17 | Stephen Kelly | Status | new => resolved |
| 2013-12-23 11:17 | Stephen Kelly | Resolution | open => no change required |
| 2013-12-23 11:17 | Stephen Kelly | Assigned To | => Stephen Kelly |
| 2014-06-02 08:37 | Robert Maynard | Note Added: 0036065 | |
| 2014-06-02 08:37 | Robert Maynard | Status | resolved => closed |
| Issue History |
| Copyright © 2000 - 2018 MantisBT Team |