Skip to content

Use absolute path for GIT_REPO_URL file:// URL #2051

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

Merged
merged 1 commit into from
Dec 23, 2019

Conversation

bruceadams
Copy link
Contributor

file:// URLs do not support relative paths. This change uses $PWD to make an absolute path based on where the user is running Crates on their local machine.

This fixes issue #2030

file:// URLs do not support relative paths.
This change uses $PWD to make an absolute path based on where the
user is running Crates on their local machine.

This fixes issue rust-lang#2030
@rust-highfive
Copy link

r? @carols10cents

(rust_highfive has picked a reviewer for you, use r? to override)

@smarnach
Copy link
Contributor

Thanks for the fix! I wasn't sure whether environment variables are expanded during processing of .env (it's not executed by the shell after all), but apparently this works fine.

A file: URL including the scheme does not support relative paths, but the git2 crate could reasonably accept a relative path without the scheme (like "tmp/index-bare") and interpret it as a relative file path (like the git command-line client does), but it doesn't.

@bors r+

@bors
Copy link
Contributor

bors commented Dec 23, 2019

📌 Commit 6105bea has been approved by smarnach

bors added a commit that referenced this pull request Dec 23, 2019
Use absolute path for GIT_REPO_URL file:// URL

file:// URLs do not support relative paths. This change uses $PWD to make an absolute path based on where the user is running Crates on their local machine.

This fixes issue #2030
@bors
Copy link
Contributor

bors commented Dec 23, 2019

⌛ Testing commit 6105bea with merge 46972f2...

@bors
Copy link
Contributor

bors commented Dec 23, 2019

☀️ Test successful - checks-travis
Approved by: smarnach
Pushing 46972f2 to master...

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.

5 participants