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

Use --release-channel=stable by default on releases #28322

Closed
radare opened this issue Sep 9, 2015 · 8 comments
Closed

Use --release-channel=stable by default on releases #28322

radare opened this issue Sep 9, 2015 · 8 comments

Comments

@radare
Copy link

radare commented Sep 9, 2015

Release tarballs should be compilable with just basic ./configure ; make ; sudo make install without having to pass special flags to configure. This is the case of the --release-channel option, that must be changed in the releases.

@alexcrichton
Copy link
Member

cc @brson, @edunham, this seems like a good idea to me!

@edunham
Copy link
Member

edunham commented Sep 9, 2015

I have no objections to this idea.

Please cc me [if|when] a PR adds this feature, since the change may necessitate some minor changes to avoid changing the buildslaves' behavior.

@brson
Copy link
Contributor

brson commented Sep 9, 2015

I think I'm ok with this, although there's also something to be said for distributing the same source we are building from and this will require flipping a bit somewhere (ofc we're not building from the source tarball at all and basically nobody has any idea whether it even works :).

Off-hand I don't have a smart way to do this. I guess we would need to add some additional metadata to the source directory for the configure script to scrape, or we could frob the configure script with sed when packaging the source tarball.

@alexcrichton
Copy link
Member

We could perhaps special case this and say "if .git isn't present we're a source tarball" and just piggy back on that logic.

@brson brson added the A-build label Sep 9, 2015
@brson
Copy link
Contributor

brson commented Sep 9, 2015

@alexcrichton Yeah, we could. I'm wary though that those may be two distinct cases.

@steveklabnik
Copy link
Member

Yeah, I feel vaguely the same re: repeatable builds and such, but @alexcrichton 's suggestion seems reasonable, but possibly footgun-ny...

@radare
Copy link
Author

radare commented Sep 10, 2015

Maybe just check if .git exists?

On 10 Sep 2015, at 01:21, Brian Anderson [email protected] wrote:

I think I'm ok with this, although there's also something to be said for distributing the same source we are building from and this will require flipping a bit somewhere (ofc we're not building from the source tarball at all and basically nobody has any idea whether it even works :).

Off-hand I don't have a smart way to do this. I guess we would need to add some additional metadata to the source directory for the configure script to scrape, or we could frob the configure script with sed when packaging the source tarball.


Reply to this email directly or view it on GitHub.

@brson
Copy link
Contributor

brson commented Sep 11, 2015

OK, let's just piggy-back on the .git check.

bltavares added a commit to bltavares/rust that referenced this issue Jun 12, 2016
> Release tarballs should be compilable with just basic ./configure ;
> make ; sudo make install without having to pass special flags to
> configure. This is the case of the --release-channel option, that must
> be changed in the releases.

This commit detects the presence of .git, as it happens on other parts
of `configure` to assume it is a tarball. Then it changes the default
value stored, before parsing the arguments, while still allowing it to
be overriden before any action verifying the flag is done.

Closes rust-lang#28322
bors added a commit that referenced this issue Jul 12, 2016
…arball, r=brson

Use --release-channel=stable by default on releases

> Release tarballs should be compilable with just basic ./configure ;
> make ; sudo make install without having to pass special flags to
> configure. This is the case of the --release-channel option, that must
> be changed in the releases.

This commit detects the presence of .git, as it happens on other parts
of `configure` to assume it is a tarball. Then it changes the default
value stored, before parsing the arguments, while still allowing it to
be overriden before any action verifying the flag is done.

Closes #28322
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

No branches or pull requests

5 participants