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

Only stamp on optimized/release builds #849

Closed
AustinSchuh opened this issue Feb 5, 2016 · 5 comments
Closed

Only stamp on optimized/release builds #849

AustinSchuh opened this issue Feb 5, 2016 · 5 comments
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Configurability platforms, toolchains, cquery, select(), config transitions type: feature request

Comments

@AustinSchuh
Copy link
Contributor

I'd like to set our repo up so by default, -c dbg or -c fastbuild run without stamping, but -c opt runs with stamping enabled.

bazel.rc:
build --stamp=yes

will do it for everything. Is there a way to only enable stamping for -c opt by default?

@ulfjack
Copy link
Contributor

ulfjack commented Feb 5, 2016

There's no way to do that right now, but I think it's a very valuable
feature request. We might want that internally as well.

On Fri, Feb 5, 2016 at 1:06 AM, Austin Schuh notifications@github.com
wrote:

I'd like to set our repo up so by default, -c dbg or -c fastbuild run
without stamping, but -c opt runs with stamping enabled.

bazel.rc:
build --stamp=yes

will do it for everything. Is there a way to only enable stamping for -c
opt by default?


Reply to this email directly or view it on GitHub
#849.

@dslomov
Copy link
Contributor

dslomov commented Feb 8, 2016

Assigning to Damien for further triage.

@damienmg damienmg added the P2 We'll consider working on this in future. (Assignee optional) label Feb 8, 2016
@damienmg
Copy link
Contributor

damienmg commented Feb 8, 2016

Ok that's a valuable FR but we haven't the bandwidth to work on it right now.

As a workaround you can put that in your bazelrc:

build:opt --stamp
build:opt -c opt

build:dbg --nostamp
build:dbg -c dbg

build:fastbuild --nostamp
build:fastbuild -c fastbuild

And then call bazel with --config dbg instead of -c dbg

@helenalt
Copy link
Contributor

@ulfjack given the workaround provided by damenmg above, is this still a feature worth considering or do we close this issue?

@ulfjack
Copy link
Contributor

ulfjack commented Feb 15, 2018

I think that's good enough.

@ulfjack ulfjack closed this as completed Feb 15, 2018
@aiuto aiuto added team-Configurability platforms, toolchains, cquery, select(), config transitions and removed z-category: extensibility > configurability labels Feb 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Configurability platforms, toolchains, cquery, select(), config transitions type: feature request
Projects
None yet
Development

No branches or pull requests

6 participants