Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pretty up the build #1561

Merged
merged 1 commit into from
Dec 29, 2014
Merged

Pretty up the build #1561

merged 1 commit into from
Dec 29, 2014

Conversation

longjon
Copy link
Contributor

@longjon longjon commented Dec 11, 2014

After #1472.

The Makefile build currently produces a hard-to-follow terminal-scrolling mess of commands, parameters, shell hacks, etc. This PR makes some simple changes for a prettier build (in the style of, e.g., the linux kernel, last I checked).

Build commands are displayed as "CMD input_file". The full commands can be printed using Q= make, or by changing Makefile.config (currently, pretty is on by default with an up-to-date Makefile.config).

The method is described at http://stackoverflow.com/questions/2969912/pretty-print-makefiles.

@jeffdonahue
Copy link
Contributor

cool, LGTM -- feel free to merge

(see the compacted build outputs at https://travis-ci.org/BVLC/caffe/jobs/44188830#L2491)

Unset Q (in the environment) to see full commands, or edit in
Makefile.config.
longjon added a commit to longjon/caffe that referenced this pull request Dec 29, 2014
longjon added a commit to longjon/caffe that referenced this pull request Dec 29, 2014
@longjon
Copy link
Contributor Author

longjon commented Dec 29, 2014

Now merging. Keep in mind that you won't get the pretty build with an existing Makefile.config unless you add the new Q ?= @ line.

longjon added a commit that referenced this pull request Dec 29, 2014
@longjon longjon merged commit dcd7f7c into BVLC:dev Dec 29, 2014
@longjon longjon deleted the pretty-build branch December 30, 2014 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants