Skip to content

you can't build unless you have git, so configure should require it #1887

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

Closed
wants to merge 3 commits into from
Closed

Conversation

db48x
Copy link
Contributor

@db48x db48x commented Feb 22, 2012

No description provided.

@brson
Copy link
Contributor

brson commented Feb 22, 2012

Thanks!

There is one situation where we do build without git - distribution tarballs. If you look here the configure script is conditionalized to not use git if the source is not inside a git repo. Makefile.in does some similar checks.

So to report this error we probably need to put it after we decide whether to set CFG_DISABLE_MANAGE_SUBMODULES, and only error if both CFG_DISABLE_MANAGE_SUBMODULES and CFG_GIT are not defined (or empty - not sure exactly).

I merged the commit fixing the python quoting in the other pull request.

@brson
Copy link
Contributor

brson commented Feb 29, 2012

@db48x Would you like to revise this pull request?

@db48x
Copy link
Contributor Author

db48x commented Feb 29, 2012

Actually, I did make a change that I didn't actually commit anywhere...

@brson
Copy link
Contributor

brson commented Mar 8, 2012

I apologize for not merging this yet. I will do it tomorrow.

@brson
Copy link
Contributor

brson commented Mar 8, 2012

Merged. Thanks.

@brson brson closed this Mar 8, 2012
@brson
Copy link
Contributor

brson commented Mar 8, 2012

I did rebase this before committing it.

celinval added a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024

I created a new template using the GitHub interface to see if that makes the template show up. I used a different name since the title reflects on the name of the file.
Kobzol pushed a commit to Kobzol/rust that referenced this pull request Dec 30, 2024
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Jan 2, 2025
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