View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0013809 | CMake | CMake | public | 2012-12-21 09:04 | 2016-06-10 14:31 | ||||
Reporter | Vassil Vassilev | ||||||||
Assigned To | Kitware Robot | ||||||||
Priority | normal | Severity | feature | Reproducibility | always | ||||
Status | closed | Resolution | moved | ||||||
Platform | trunk | OS | OS X | OS Version | 10.4.10 | ||||
Product Version | |||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0013809: add_dependencies and --graphviz | ||||||||
Description | Hi, I am using cmake in the context of C# and I don't use add_library or add_executable but I have my own functions that do that trick. I resolve the dependencies between the libraries (since have my home-grown csharp_add_library) using add_dependencies. Using that however doesn't allow me to print the dependencies between the targets, which is something IMO natural to expect. Am I doing something wrong in general or somebody just has to teach cmGraphVizWriter to react on cmTarget::UTILITY? If that is the case I can propose the attached patch. Cheers, Vassil PS1: Note that I am newbie (I've started using cmake 3 days ago). PS2: I had hard time building cmake sources in debug mode. Why there is no ./bootstrap --enable-debug switch? | ||||||||
Steps To Reproduce | 1. add_custom_target(... libA ...) 2. add_custom_target(... libB ...) 3. add_dependencies(libA libB) 4. cmake --graphviz=file.dot path_to_some_source | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | cmGraphVizWriterPlusUtility.diff [^] (494 bytes) 2012-12-21 09:04 [Show Content]
0001-Update-a-wrong-comment.-It-was-maybe-correct-a-long-.patch [^] (1,207 bytes) 2015-06-23 19:03 [Show Content] 0001-Update-documentation-of-cmTarget-AddUtility.patch [^] (1,152 bytes) 2015-06-23 19:04 [Show Content] 0002-graphviz-Enable-support-for-utility-dependencies-and.patch [^] (7,626 bytes) 2015-06-23 19:05 [Show Content] | ||||||||
Relationships | ||||||
|
Relationships |
Notes | |
(0038963) Andreas Hartmetz (reporter) 2015-06-23 19:07 |
I also have/had a need to see targets added as dependencies with add_dependency(), in a different context. The attached patch starting with 0002 does it. The first (please disregard the first 0001* upload, it's an older version) updates the AFAICT now very wrong documentation of cmTarget::AddUtility(). |
(0038983) awinterz (reporter) 2015-06-25 10:49 |
I'm also interested in the 0002 patch. any news on this one? |
(0039002) Brad King (manager) 2015-06-30 09:55 |
Re 0013809:0038963: Thanks for working on a patch. Please read CONTRIBUTING.rst for the preferred contribution path. Here are some comments on "0002-graphviz-Enable-support-for-utility-dependencies-and.patch". The name "utility target" is mostly used internally. Public-facing documentation, commands, and options generally refers to a "custom target". Please rename GRAPHVIZ_UTILITY_TARGETS to GRAPHVIZ_CUSTOM_TARGETS. Rather than documenting the possible inclusion of "internal cmake targets" we should teach the implementation to exclude them automatically. The dependency graph should not depend on which CMake generator is used, but rather only on the project-defined targets. |
(0039003) Andreas Hartmetz (reporter) 2015-06-30 18:39 |
There seems to be no particular property in internal targets except the name (there are two special names). Do you prefer a filter by name or a new property? My own opinion ist that if you're debugging dependencies, you might want to see all of them. But I don't care that much. About the contribution guidelines, what I'm doing wrong is not posting patches to the mailing list, right? I already have enough mailing list subscriptions and I'm trying to avoid any new ones if at all possible... and in most projects posts from non-subscribers will rot in moderation forever. |
(0039012) Brad King (manager) 2015-07-02 08:41 |
Re 0013809:0039003: Just to make sure we're talking about the same special targets, what are the "two special names" to which you refer? FYI, on mailman mailing lists you can set the "nomail" option after subscribing to be allowed to post but never receive mail from the list when not explicitly Cc-ed. |
(0042179) Kitware Robot (administrator) 2016-06-10 14:28 |
Resolving issue as `moved`. This issue tracker is no longer used. Further discussion of this issue may take place in the current CMake Issues page linked in the banner at the top of this page. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2012-12-21 09:04 | Vassil Vassilev | New Issue | |
2012-12-21 09:04 | Vassil Vassilev | File Added: cmGraphVizWriterPlusUtility.diff | |
2015-06-23 19:03 | Andreas Hartmetz | File Added: 0001-Update-a-wrong-comment.-It-was-maybe-correct-a-long-.patch | |
2015-06-23 19:04 | Andreas Hartmetz | File Added: 0001-Update-documentation-of-cmTarget-AddUtility.patch | |
2015-06-23 19:05 | Andreas Hartmetz | File Added: 0002-graphviz-Enable-support-for-utility-dependencies-and.patch | |
2015-06-23 19:07 | Andreas Hartmetz | Note Added: 0038963 | |
2015-06-25 10:49 | awinterz | Note Added: 0038983 | |
2015-06-30 08:56 | Brad King | Relationship added | related to 0015636 |
2015-06-30 09:55 | Brad King | Note Added: 0039002 | |
2015-06-30 18:39 | Andreas Hartmetz | Note Added: 0039003 | |
2015-07-02 08:41 | Brad King | Note Added: 0039012 | |
2016-06-10 14:28 | Kitware Robot | Note Added: 0042179 | |
2016-06-10 14:28 | Kitware Robot | Status | new => resolved |
2016-06-10 14:28 | Kitware Robot | Resolution | open => moved |
2016-06-10 14:28 | Kitware Robot | Assigned To | => Kitware Robot |
2016-06-10 14:31 | Kitware Robot | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |