View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014973CMakeCMakepublic2014-06-12 14:572015-01-05 08:38
ReporterAdam Strzelecki 
Assigned ToBrad King 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformAppleOSOSXOS Version10.9.3
Product Version 
Target VersionCMake 3.1Fixed in VersionCMake 3.1 
Summary0014973: [RFE] Produce shorted output for cmake/ccmake when no arguments specified
DescriptionPlease find attached patch.

Currently running cmake/ccmake without arguments is equal running it with --help which is bad idea since it doesn't really tell what to do.
Instead of that attached patch makes them emit shorter brief with hint:

====
$ cmake/bin/cmake
Usage

  cmake [options] <path-to-source>
  cmake [options] <path-to-existing-build>

Hint

  cmake . for in-source build (not recommended)
  cmake .. for out-of-source build inside subfolder
  cmake --help for more information

Note: Building in separate folder leaves source files intact.
===
TagsNo tags attached.
Attached Filespatch file icon 0001-Produce-shorter-output-on-no-arguments.patch [^] (5,752 bytes) 2014-06-12 14:57 [Show Content]
patch file icon 0001-Produce-shorter-output-on-no-arguments-2.patch [^] (6,561 bytes) 2014-06-12 16:25 [Show Content]

 Relationships

  Notes
(0036175)
Brad King (manager)
2014-06-12 15:31

For reference the mailing list thread leading to this patch is:

 Why simple `cmake` does not launch default generator on current directory?
 http://thread.gmane.org/gmane.comp.programming.tools.cmake.user/49711 [^]
(0036177)
Brad King (manager)
2014-06-12 15:41

I like the general idea of keeping the output to one terminal screen size.

I'd prefer not to give hints like "cmake ." because we do not know anything about the current working directory when the user reads this message. How about simply:

Usage

  cmake [options] <path-to-source>
  cmake [options] <path-to-existing-build>

Specify a source directory to (re-)generate a build system for
it in the current working directory.  Specify an existing build
directory to re-generate its build system.

Run 'cmake --help' for more information.


?
(0036179)
Adam Strzelecki (reporter)
2014-06-12 16:26

0001-Produce-shorter-output-on-no-arguments-2.patch now provides suggested above implementation.

NOTE: This is Git generated patch, so `git am` is preferred to keep commit information.
(0036184)
Adam Strzelecki (reporter)
2014-06-13 08:34

Please find pull request for your convenience here: https://github.com/Kitware/CMake/pull/110 [^]
(0036185)
Brad King (manager)
2014-06-13 08:41

I've applied the patch here:

 cmake,ccmake: Produce shorter output on no arguments
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=de7c2882 [^]

with a minor tweak to make the advice paragraph show up in the --help output too.
(0036186)
Brad King (manager)
2014-06-13 08:42

If you'd like to contribute patches regularly please read CONTRIBUTING.rst.
(0037565)
Robert Maynard (manager)
2015-01-05 08:38

Closing resolved issues that have not been updated in more than 4 months.

 Issue History
Date Modified Username Field Change
2014-06-12 14:57 Adam Strzelecki New Issue
2014-06-12 14:57 Adam Strzelecki File Added: 0001-Produce-shorter-output-on-no-arguments.patch
2014-06-12 15:31 Brad King Note Added: 0036175
2014-06-12 15:41 Brad King Note Added: 0036177
2014-06-12 16:25 Adam Strzelecki File Added: 0001-Produce-shorter-output-on-no-arguments-2.patch
2014-06-12 16:26 Adam Strzelecki Note Added: 0036179
2014-06-13 08:34 Adam Strzelecki Note Added: 0036184
2014-06-13 08:41 Brad King Note Added: 0036185
2014-06-13 08:42 Brad King Note Added: 0036186
2014-06-13 08:43 Brad King Assigned To => Brad King
2014-06-13 08:43 Brad King Status new => resolved
2014-06-13 08:43 Brad King Resolution open => fixed
2014-06-13 08:43 Brad King Fixed in Version => CMake 3.1
2014-06-13 08:43 Brad King Target Version => CMake 3.1
2015-01-05 08:38 Robert Maynard Note Added: 0037565
2015-01-05 08:38 Robert Maynard Status resolved => closed


Copyright © 2000 - 2018 MantisBT Team