View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013809CMakeCMakepublic2012-12-21 09:042016-06-10 14:31
ReporterVassil Vassilev 
Assigned ToKitware Robot 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionmoved 
PlatformtrunkOSOS XOS Version10.4.10
Product Version 
Target VersionFixed in Version 
Summary0013809: add_dependencies and --graphviz
DescriptionHi,
  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 Reproduce1. add_custom_target(... libA ...)
2. add_custom_target(... libB ...)
3. add_dependencies(libA libB)
4. cmake --graphviz=file.dot path_to_some_source
TagsNo tags attached.
Attached Filesdiff file icon cmGraphVizWriterPlusUtility.diff [^] (494 bytes) 2012-12-21 09:04 [Show Content]
patch file icon 0001-Update-a-wrong-comment.-It-was-maybe-correct-a-long-.patch [^] (1,207 bytes) 2015-06-23 19:03 [Show Content]
patch file icon 0001-Update-documentation-of-cmTarget-AddUtility.patch [^] (1,152 bytes) 2015-06-23 19:04 [Show Content]
patch file icon 0002-graphviz-Enable-support-for-utility-dependencies-and.patch [^] (7,626 bytes) 2015-06-23 19:05 [Show Content]

 Relationships
related to 0015636closedKitware Robot alias libraries and and --graphviz 

  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.

 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


Copyright © 2000 - 2018 MantisBT Team