Skip to content

Pub: dependency on https://host/dir/ installs into the wrong directory #46

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
DartBot opened this issue Jun 5, 2015 · 3 comments
Closed
Assignees
Labels
type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@DartBot
Copy link

DartBot commented Jun 5, 2015

Originally opened as dart-lang/sdk#3493

This issue was originally filed by [email protected]


Paths ending in slash don't install properly - the git repo is checked out into packages/ instead of packages/foo/.
In particular, code.google.com recommends repo URLs ending in /.

I think it's detecting the repo name as the empty string, and so running 'git clone $repo packages/$reponame' does the wrong thing.

(Also, shouldn't it be naming the checkout dir after the dependency name in my pubspec? Those are guaranteed to be unique...)

e.g.

$ cat pubspec.yaml
dependencies:
  googleapis:
    git: https://code.google.com/p/foo/

$ pub install
Cloning into '/usr/local/google/home/sammccall/gits/dartapiarysamples/packages'...
Dependencies installed!

$ ls -a packages
. .. .git

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/2108507?v=3" align="left" width="48" height="48"hspace="10"> Comment by dgrove


cc @munificent.
Set owner to @nex3.
Added Area-Pub, Triaged labels.

@DartBot DartBot added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) Fixed labels Jun 5, 2015
@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


Added this to the M1 milestone.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


Fixed by dart-lang/sdk@4b76d74.


Added Fixed label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

2 participants