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.

Sign up for GitHub

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

Jump to bottom

make: build quiet as default #5827

Merged
merged 1 commit into from
Sep 12, 2016
Merged

make: build quiet as default #5827

merged 1 commit into from
Sep 12, 2016

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Sep 6, 2016

Currently every application needs to set this variable to not be flooded with gcc build lines. Is there a reason why we don't set it to 1 as default?

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: build system Area: Build system Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR labels Sep 6, 2016
@kaspar030
Copy link
Contributor

+1

@thomaseichinger
Copy link
Member

+1
(But you have a typo in the PR title)

@miri64 miri64 changed the title make: build quite as default make: build quiet as default Sep 6, 2016
@kaspar030
Copy link
Contributor

typo in the PR title

and commit title. :)

@miri64
Copy link
Member Author

miri64 commented Sep 6, 2016

I know. But I didn't want to "squash" before an ACK ;-P

@miri64 miri64 force-pushed the make/enh/quiet-per-default branch from b0e3d34 to 043eb67 Compare September 6, 2016 16:20
@OlegHahm OlegHahm added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 11, 2016
@OlegHahm
Copy link
Member

ACK

@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Sep 12, 2016
@miri64 miri64 merged commit 55a08ae into RIOT-OS:master Sep 12, 2016
@miri64 miri64 deleted the make/enh/quiet-per-default branch September 12, 2016 15:00
@miri64 miri64 added this to the Release 2016.10 milestone Oct 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@OlegHahm OlegHahm

Labels
Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2016.10
Development

Successfully merging this pull request may close these issues.
4 participants
@miri64 @kaspar030 @thomaseichinger @OlegHahm