(0014723)
Brad King (manager)
2009-01-29 14:35

This has been on my todo list for a long time. See issue 0006348, my comment at time 2008-02-14 17:14.

(0023373)
Hugo Heden (reporter)
2010-11-18 06:48
edited on: 2010-11-18 06:49

Another use-case is this: Somehow invoke a sub-process, "ant clean", whenever the "clean"-target is executed

http://www.mail-archive.com/cmake@cmake.org/msg11588.html [^]

(0023374)
Hugo Heden (reporter)
2010-11-18 06:51

Brad: I'm not completely sure that issue 0006348 is the same as this one, but I guess I'll take your word for it.

(0024642)
Eric NOULARD (developer)
2011-01-13 07:30

Just to note that the complete need would be
to be able to

1) add dependency TO builtin target like:

   add_dependencies(package MyDoxygenCustomTarget)

   which will enforce the "MyDoxygenCustomTarget" to be called
   whenever the builtin target "package" is built.

2) add dependency FROM builtin target like:

   add_dependencies(TestAllPackagesWhenInstalled install)

   which will enforce the builtin target "install" to
   be built whenever "TestAllPackagesWhenInstalled" is.

(0024643)
Hugo Heden (reporter)
2011-01-13 07:56

Thanks for noting that Eric, I don't think I had thought about that the _two_ directions of dependencies. You're right.

(0026520)
Brad King (manager)
2011-05-17 15:53

Moving to backlog. I think the approach mentioned in 0006348:0010505 will work but I have no time to work on this.

(0026589)
David Cole (manager)
2011-05-25 16:54

Not for 2.8.5 - postponing until a future release

(0027192)
Tomasz Majchrowski (reporter)
2011-08-12 07:51

Is there any chance to fix it for coming 2.8.6 ?

(0027193)
David Cole (manager)
2011-08-12 08:59
edited on: 2011-08-12 09:00

Not really a good chance of fixing this for 2.8.6 unless somebody proposes a reasonable patch that works, has tests associated with it, and does not seem too risky or de-stabilizing.

Brad's note from May 17 still stands: "Moving to backlog. I think the approach mentioned in 0006348:0010505 will work but I have no time to work on this."

It's in the "backlog" and it is assigned to nobody. That typically means it's not being actively pursued by anybody.

(0027638)
Eric NOULARD (developer)
2011-10-24 12:48

Targeting 2.8.7 because it has been raised on the ML request right after 2.8.6
and because I vote for this too :-]

(0027717)
David Genest (reporter)
2011-11-03 12:03

I would like to add a vote to this feature also. We are using cmake in a very Visual Studio Savy workplace. People here do not want to manually add files to the CMakeLists.

For this reason, we have cmake harvest the source tree at generation, and a script to compare the actual source tree files to know if a regeneration is needed. I know it is not the CMake way, but it has worked very well for us.

If we could add a dependency from ZERO_CHECK to our source tree verification command, the integration would be complete. (When the script detects a change, it touches the top-most CMakeLists.txt). Right now, we depend on our top level dependency, and sometimes the re-generation gets caught too late.

(0027852)
Chris Scharver (reporter)
2011-11-23 10:34

I would like this too, as it would allow us to easily build installers through a custom target depending on INSTALL.

(0027967)
David Cole (manager)
2011-12-13 19:21

Unset target version field; too late for a fix to go into 2.8.7; deferred until a future version.

(0028212)
David Cole (manager)
2012-01-07 10:03

Repeating/updating my own note: Not really a good chance of fixing this for 2.8.8 unless somebody proposes a reasonable patch that works, has tests associated with it, and does not seem too risky or de-stabilizing.

For more comments, see:

0008438:0027193

and

0006348:0010505

(0029232)
David Cole (manager)
2012-04-19 08:40

Unsetting target version, removing from roadmap, nobody is working on this.

When somebody begins active development work on this issue, I'll put it back on the roadmap. Until then, it's in the backlog...

(0032225)
franck.bonin (reporter)
2013-02-08 10:19

I would like this.
For me it would enabling signing CPack generated installer with a custom post package target. Let me try this :

IF(WIN32 AND NOT UNIX)
   add_custom_target(signpackage package COMMAND signtool.exe ...)
ELSEIF(${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
   add_custom_target(signpackage package COMMAND productsign ...)
ELSE()
   add_custom_target(signpackage package COMMAND echo no signing)
ENDIF()

add_dependencies(signpackage package)

(0033282)
ycollet (reporter)
2013-06-14 05:40

I just vote for this feature.

(0034392)
Bjoern Thiel (reporter)
2013-11-05 09:48

Me too.

(0034448)
Tomasz Majchrowski (reporter)
2013-11-13 14:48

vote++

(0036736)
Jean-Bernard Jansen (reporter)
2014-09-06 15:32

I would definitely like that feature too.

(0037050)
Pascal Bach (reporter)
2014-10-20 05:21

I'm trying to integrate an Ant build with CMake. And it would be nice if I could just add a custom_command that calls ant install and make it a dependency of the built-in install target. So this would be a great help.

(0037055)
Brad King (manager)
2014-10-20 09:35

Re 0008438:0037050: That is the opposite direction of this issue, which is about making a custom target depend on a builtin target. Do run extra steps during installation, use the install(SCRIPT) or install(CODE) command.

(0037368)
Josh Klontz (reporter)
2014-12-03 09:52

+1

(0038467)
Tom Hughes (reporter)
2015-04-07 12:57

+1

(0038820)
gonzalobg88@gmail.com (reporter)
2015-05-29 11:22

At the very minimum it should be stated in the documentation that this is not possible.

That is: in every single place in the docs were it says "You can add a target here" it should also say "(but remember there are no target names for all, install, test, ...)".

I just wasted an hour of my life trying to find out how to do this and scratching my head about why ALL or CMAKE_PROJECT_NAME do not work.

(0039488)
Benoît Bleuzé (reporter)
2015-09-28 11:12

As everyone else I would like to show my interest for this topic, as I would like to depend on the "package" target.

The ramifications of this particular issue seem to be to extensive for me to work on a first patch for cmake, but is there anything else I can do to push this issue closer to resolution?

(0039691)
Davy Durham (reporter)
2015-10-28 01:30

+1 for me too.

In the mean time, my issue is that I add_subdirectory(.. EXCLUDE_FROM_ALL) on a CMakeLists.txt which defines many targets, but I only want a couple of them built by default.

So my CMakeLists.txt looks like so...
   ...
   add_subdirectory(many_targets EXCLUDE_FROM_ALL)

   add_custom_target(build_anyway ALL)
   add_dependencies(build_anyway
      wanted_target_12
      wanted_target_32
   )

IMO, it would make perfect since to be able to say something like:
   ...
   add_subdirectory(many_targets EXCLUDE_FROM_ALL)

   add_dependencies(ALL
      wanted_target_12
      wanted_target_32
   )

It's just a slightly smaller bit of code, but also doesn't have the problem of having to make up a unique name for a faux target and is conceptually more straight-forward.

(0039907)
Christopher Sean Morrison (reporter)
2015-11-24 10:31

Ditto, need this. :)

(0039940)
arufanov (reporter)
2015-12-03 02:14

Also need this for sign dll & msi package based on package built-in target.

(0041496)
Kitware Robot (administrator)
2016-06-10 14:27

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
(0014723) Brad King (manager) 2009-01-29 14:35	This has been on my todo list for a long time. See issue 0006348, my comment at time 2008-02-14 17:14.

(0023373) Hugo Heden (reporter) 2010-11-18 06:48 edited on: 2010-11-18 06:49	Another use-case is this: Somehow invoke a sub-process, "ant clean", whenever the "clean"-target is executed http://www.mail-archive.com/cmake@cmake.org/msg11588.html [^]

(0023374) Hugo Heden (reporter) 2010-11-18 06:51	Brad: I'm not completely sure that issue 0006348 is the same as this one, but I guess I'll take your word for it.

(0024642) Eric NOULARD (developer) 2011-01-13 07:30	Just to note that the complete need would be to be able to 1) add dependency TO builtin target like: add_dependencies(package MyDoxygenCustomTarget) which will enforce the "MyDoxygenCustomTarget" to be called whenever the builtin target "package" is built. 2) add dependency FROM builtin target like: add_dependencies(TestAllPackagesWhenInstalled install) which will enforce the builtin target "install" to be built whenever "TestAllPackagesWhenInstalled" is.

(0024643) Hugo Heden (reporter) 2011-01-13 07:56	Thanks for noting that Eric, I don't think I had thought about that the _two_ directions of dependencies. You're right.

(0026520) Brad King (manager) 2011-05-17 15:53	Moving to backlog. I think the approach mentioned in 0006348:0010505 will work but I have no time to work on this.

(0026589) David Cole (manager) 2011-05-25 16:54	Not for 2.8.5 - postponing until a future release

(0027192) Tomasz Majchrowski (reporter) 2011-08-12 07:51	Is there any chance to fix it for coming 2.8.6 ?

(0027193) David Cole (manager) 2011-08-12 08:59 edited on: 2011-08-12 09:00	Not really a good chance of fixing this for 2.8.6 unless somebody proposes a reasonable patch that works, has tests associated with it, and does not seem too risky or de-stabilizing. Brad's note from May 17 still stands: "Moving to backlog. I think the approach mentioned in 0006348:0010505 will work but I have no time to work on this." It's in the "backlog" and it is assigned to nobody. That typically means it's not being actively pursued by anybody.

(0027638) Eric NOULARD (developer) 2011-10-24 12:48	Targeting 2.8.7 because it has been raised on the ML request right after 2.8.6 and because I vote for this too :-]

(0027717) David Genest (reporter) 2011-11-03 12:03	I would like to add a vote to this feature also. We are using cmake in a very Visual Studio Savy workplace. People here do not want to manually add files to the CMakeLists. For this reason, we have cmake harvest the source tree at generation, and a script to compare the actual source tree files to know if a regeneration is needed. I know it is not the CMake way, but it has worked very well for us. If we could add a dependency from ZERO_CHECK to our source tree verification command, the integration would be complete. (When the script detects a change, it touches the top-most CMakeLists.txt). Right now, we depend on our top level dependency, and sometimes the re-generation gets caught too late.

(0027852) Chris Scharver (reporter) 2011-11-23 10:34	I would like this too, as it would allow us to easily build installers through a custom target depending on INSTALL.

(0027967) David Cole (manager) 2011-12-13 19:21	Unset target version field; too late for a fix to go into 2.8.7; deferred until a future version.

(0028212) David Cole (manager) 2012-01-07 10:03	Repeating/updating my own note: Not really a good chance of fixing this for 2.8.8 unless somebody proposes a reasonable patch that works, has tests associated with it, and does not seem too risky or de-stabilizing. For more comments, see: 0008438:0027193 and 0006348:0010505

(0029232) David Cole (manager) 2012-04-19 08:40	Unsetting target version, removing from roadmap, nobody is working on this. When somebody begins active development work on this issue, I'll put it back on the roadmap. Until then, it's in the backlog...

(0032225) franck.bonin (reporter) 2013-02-08 10:19	I would like this. For me it would enabling signing CPack generated installer with a custom post package target. Let me try this : IF(WIN32 AND NOT UNIX) add_custom_target(signpackage package COMMAND signtool.exe ...) ELSEIF(${CMAKE_SYSTEM_NAME} MATCHES "Darwin") add_custom_target(signpackage package COMMAND productsign ...) ELSE() add_custom_target(signpackage package COMMAND echo no signing) ENDIF() add_dependencies(signpackage package)

(0033282) ycollet (reporter) 2013-06-14 05:40	I just vote for this feature.

(0034392) Bjoern Thiel (reporter) 2013-11-05 09:48	Me too.

(0034448) Tomasz Majchrowski (reporter) 2013-11-13 14:48	vote++

(0036736) Jean-Bernard Jansen (reporter) 2014-09-06 15:32	I would definitely like that feature too.

(0037050) Pascal Bach (reporter) 2014-10-20 05:21	I'm trying to integrate an Ant build with CMake. And it would be nice if I could just add a custom_command that calls ant install and make it a dependency of the built-in install target. So this would be a great help.

(0037055) Brad King (manager) 2014-10-20 09:35	Re 0008438:0037050: That is the opposite direction of this issue, which is about making a custom target depend on a builtin target. Do run extra steps during installation, use the install(SCRIPT) or install(CODE) command.

(0037368) Josh Klontz (reporter) 2014-12-03 09:52	+1

(0038467) Tom Hughes (reporter) 2015-04-07 12:57	+1

(0038820) gonzalobg88@gmail.com (reporter) 2015-05-29 11:22	At the very minimum it should be stated in the documentation that this is not possible. That is: in every single place in the docs were it says "You can add a target here" it should also say "(but remember there are no target names for all, install, test, ...)". I just wasted an hour of my life trying to find out how to do this and scratching my head about why ALL or CMAKE_PROJECT_NAME do not work.

(0039488) Benoît Bleuzé (reporter) 2015-09-28 11:12	As everyone else I would like to show my interest for this topic, as I would like to depend on the "package" target. The ramifications of this particular issue seem to be to extensive for me to work on a first patch for cmake, but is there anything else I can do to push this issue closer to resolution?

(0039691) Davy Durham (reporter) 2015-10-28 01:30	+1 for me too. In the mean time, my issue is that I add_subdirectory(.. EXCLUDE_FROM_ALL) on a CMakeLists.txt which defines many targets, but I only want a couple of them built by default. So my CMakeLists.txt looks like so... ... add_subdirectory(many_targets EXCLUDE_FROM_ALL) add_custom_target(build_anyway ALL) add_dependencies(build_anyway wanted_target_12 wanted_target_32 ) IMO, it would make perfect since to be able to say something like: ... add_subdirectory(many_targets EXCLUDE_FROM_ALL) add_dependencies(ALL wanted_target_12 wanted_target_32 ) It's just a slightly smaller bit of code, but also doesn't have the problem of having to make up a unique name for a faux target and is conceptually more straight-forward.

(0039907) Christopher Sean Morrison (reporter) 2015-11-24 10:31	Ditto, need this. :)

(0039940) arufanov (reporter) 2015-12-03 02:14	Also need this for sign dll & msi package based on package built-in target.

(0041496) Kitware Robot (administrator) 2016-06-10 14:27	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
2009-01-29 10:40	Hugo Heden	New Issue
2009-01-29 14:27	Bill Hoffman	Status	new => assigned
2009-01-29 14:27	Bill Hoffman	Assigned To	=> Brad King
2009-01-29 14:35	Brad King	Note Added: 0014723
2009-01-29 14:35	Brad King	Relationship added	related to 0006348
2010-11-18 06:48	Hugo Heden	Note Added: 0023373
2010-11-18 06:49	Hugo Heden	Note Edited: 0023373
2010-11-18 06:51	Hugo Heden	Note Added: 0023374
2011-01-13 07:30	Eric NOULARD	Note Added: 0024642
2011-01-13 07:56	Hugo Heden	Note Added: 0024643
2011-04-14 14:27	David Cole	Target Version	=> CMake 2.8.5
2011-05-17 15:53	Brad King	Note Added: 0026520
2011-05-17 15:53	Brad King	Assigned To	Brad King =>
2011-05-17 15:53	Brad King	Status	assigned => backlog
2011-05-25 16:54	David Cole	Note Added: 0026589
2011-05-25 16:54	David Cole	Target Version	CMake 2.8.5 =>
2011-08-12 07:51	Tomasz Majchrowski	Note Added: 0027192
2011-08-12 08:59	David Cole	Note Added: 0027193
2011-08-12 09:00	David Cole	Note Edited: 0027193
2011-10-24 12:48	Eric NOULARD	Note Added: 0027638
2011-10-24 12:48	Eric NOULARD	Target Version	=> CMake 2.8.7
2011-11-03 12:03	David Genest	Note Added: 0027717
2011-11-23 10:34	Chris Scharver	Note Added: 0027852
2011-12-13 19:21	David Cole	Note Added: 0027967
2011-12-13 19:21	David Cole	Target Version	CMake 2.8.7 =>
2012-01-03 16:21	David Cole	Target Version	=> CMake 2.8.8
2012-01-07 10:03	David Cole	Note Added: 0028212
2012-02-13 09:48	Brad King	Relationship added	related to 0012960
2012-02-14 10:40	Brad King	Relationship added	related to 0012961
2012-04-19 08:40	David Cole	Note Added: 0029232
2012-04-19 08:40	David Cole	Target Version	CMake 2.8.8 =>
2012-04-26 08:15	Brad King	Relationship added	related to 0013170
2013-02-08 10:19	franck.bonin	Note Added: 0032225
2013-06-14 05:40	ycollet	Note Added: 0033282
2013-11-05 09:48	Bjoern Thiel	Note Added: 0034392
2013-11-13 14:48	Tomasz Majchrowski	Note Added: 0034448
2014-09-06 15:32	Jean-Bernard Jansen	Note Added: 0036736
2014-10-20 05:21	Pascal Bach	Note Added: 0037050
2014-10-20 09:35	Brad King	Note Added: 0037055
2014-12-03 09:52	Josh Klontz	Note Added: 0037368
2015-04-07 12:57	Tom Hughes	Note Added: 0038467
2015-05-29 11:22	gonzalobg88@gmail.com	Note Added: 0038820
2015-09-28 11:12	Benoît Bleuzé	Note Added: 0039488
2015-10-28 01:30	Davy Durham	Note Added: 0039691
2015-11-24 10:31	Christopher Sean Morrison	Note Added: 0039907
2015-12-03 02:14	arufanov	Note Added: 0039940
2016-06-10 14:27	Kitware Robot	Note Added: 0041496
2016-06-10 14:27	Kitware Robot	Status	backlog => resolved
2016-06-10 14:27	Kitware Robot	Resolution	open => moved
2016-06-10 14:27	Kitware Robot	Assigned To	=> Kitware Robot
2016-06-10 14:30	Kitware Robot	Status	resolved => closed