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

Not Found if repository url doesn't end with ".git" #111

Closed
spebern opened this issue Jul 12, 2017 · 6 comments
Closed

Not Found if repository url doesn't end with ".git" #111

spebern opened this issue Jul 12, 2017 · 6 comments
Labels
bug describes a defect in supported/advertised functionality

Comments

@spebern
Copy link

spebern commented Jul 12, 2017

If I execute magit-status in certain repos I get the following error:
Not Found: "GET", "/repos/////issues", nil, nil, ((message . "Not Found") (documentation_url "https://developer.github.com/v3"))

The repository url returned by (magit-get "remote" (magithub-source--remote) "url") is [email protected]:spebern/Bailador

As soon as a repositry url does not end with ".git" it won't match the regex parsing that url, because
".git" is not optional.

@zakame
Copy link

zakame commented Jul 19, 2017

Just ran into this my now, mainly on Go repos fetched via go get.

@vermiculus
Copy link
Owner

@zakame, do you have steps to reproduce? :-)

@zakame
Copy link

zakame commented Jul 19, 2017

@vermiculus sure, for me it was getting docker-machine-kvm via go get:

[zakame:~] 2 % GOPATH=~/altgo go get -u -v -x github.com/dhiltgen/docker-machine-kvm 
github.com/dhiltgen/docker-machine-kvm (download)
created GOPATH=/home/zakame/altgo; see 'go help gopath'
cd .
git clone https://github.com/dhiltgen/docker-machine-kvm /home/zakame/altgo/src/github.com/dhiltgen/docker-machine-kvm
cd /home/zakame/altgo/src/github.com/dhiltgen/docker-machine-kvm
git submodule update --init --recursive
cd /home/zakame/altgo/src/github.com/dhiltgen/docker-machine-kvm
git show-ref
cd /home/zakame/altgo/src/github.com/dhiltgen/docker-machine-kvm
git submodule update --init --recursive
github.com/docker/machine (download)
cd .
git clone https://github.com/docker/machine /home/zakame/altgo/src/github.com/docker/machine
cd /home/zakame/altgo/src/github.com/docker/machine
git submodule update --init --recursive
cd /home/zakame/altgo/src/github.com/docker/machine
git show-ref
cd /home/zakame/altgo/src/github.com/docker/machine
git submodule update --init --recursive
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
cd /home/zakame/altgo/src/github.com/docker/machine
git config remote.origin.url
github.com/libvirt/libvirt-go (download)
cd .
git clone https://github.com/libvirt/libvirt-go /home/zakame/altgo/src/github.com/libvirt/libvirt-go
cd /home/zakame/altgo/src/github.com/libvirt/libvirt-go
git submodule update --init --recursive
cd /home/zakame/altgo/src/github.com/libvirt/libvirt-go
git show-ref
cd /home/zakame/altgo/src/github.com/libvirt/libvirt-go
git submodule update --init --recursive
WORK=/tmp/go-build974707010
...
cp $WORK/github.com/dhiltgen/docker-machine-kvm.a /home/zakame/altgo/pkg/linux_amd64/github.com/dhiltgen/docker-machine-kvm.a
θ71° [zakame:~] 10m12s % 

Note that the git clones above does use the GitHub URL of the repo directly, not the proper .git URL, so I suppose this might be a Go issue as well. Visiting any repos ~/altgo/src/github.com/ with magithub results in

Not Found: "GET", "/repos/////issues", nil, nil, ((message . "Not Found") (documentation_url . "https://developer.github.com/v3"))

which I think might be caused by magithub--url->repo failing to match what remote it can find in .git/config.

@vermiculus vermiculus added the bug describes a defect in supported/advertised functionality label Jul 30, 2017
@vermiculus
Copy link
Owner

Related to #105

@vermiculus
Copy link
Owner

vermiculus commented Jul 30, 2017

Closing as a duplicate of #105. Thanks for the information though – it was invaluable!

@zakame
Copy link

zakame commented Jul 30, 2017

Thanks @vermiculus @spebern !

@vermiculus vermiculus marked this as a duplicate of #105 Jul 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug describes a defect in supported/advertised functionality
Projects
None yet
Development

No branches or pull requests

3 participants