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

Fix the returned location of packages. #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stevendgonzales
Copy link

A request to http://in.archive.ubuntu.com/ubuntu/dists/trusty/main/binary-amd64/ for
package account-plugin-aim_3.8.6-0ubuntu9_amd64.deb should return a file location of
http://in.archive.ubuntu.com/ubuntu/pool/main/e/empathy/account-plugin-aim_3.8.6-0ubuntu9_amd64.deb

The existing file location returned was incorrectly returning:
http://in.archive.ubuntu.com/ubuntu/dists/trusty/main/binary-amd64/pool/main/e/empathy/account-plugin-aim_3.8.6-0ubuntu9_amd64.deb

Unit test has been included as well as updating test that pointed at saucy repo (no longer there) to point at trusty.

closes #6

A request to http://in.archive.ubuntu.com/ubuntu/dists/trusty/main/binary-amd64/ for
package account-plugin-aim_3.8.6-0ubuntu9_amd64.deb shoudl return a file location of
http://in.archive.ubuntu.com/ubuntu/pool/main/e/empathy/account-plugin-aim_3.8.6-0ubuntu9_amd64.deb

The existing file location returned was incorrectly returning:
http://in.archive.ubuntu.com/ubuntu/dists/trusty/main/binary-amd64/pool/main/e/empathy/account-plugin-aim_3.8.6-0ubuntu9_amd64.deb

Unit test has been included as well as updating test that pointed at saucy repo (no longer there) to point at trusty.

closes gocd-contrib#6
@scottmuc
Copy link

I made a private build of this and deb-repo-query and set it up on a testing go server instance. With a deb package material only, I was able to use these changes with our internal apt-repo instance.

Would love to have a release of this so we don't use my private build on our go infrastructure :)

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.

PPA support
2 participants