/pcp

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[bintray] add Debian repo HTTP headers #49

Merged
merged 1 commit into from Nov 9, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@superjamie @natoscott
@superjamie
Contributor

superjamie commented Nov 7, 2015

Bintray provide a "Set Me Up" button which provides copy-paste commands to install a Debian repo into apt.

However, the current instructions don't work, they currently suggest to paste "deb https://dl.bintray.com/pcp/trusty {distribution} {components}" but apt requires the last to fields to be something like "trusty main" or "jessie main".

This patch sets the "distribution" and "components" Debian repo properties using the Bintray HTTP headers as documented at https://bintray.com/docs/api/#_debian_upload and so should correct the "Set Me Up" commands so users can copy-paste the provided instructions.
@superjamie
[bintray] add Debian repo HTTP headers 
Bintray provide a "Set Me Up" button which provides copy-paste commands to install a Debian repo into apt.

However, the current instructions don't work, they currently suggest to paste "deb https://dl.bintray.com/pcp/trusty {distribution} {components}" but apt requires the last to fields to be something like "trusty main" or "jessie main".

This patch sets the "distribution" and "components" Debian repo properties using the Bintray HTTP headers as documented at https://bintray.com/docs/api/#_debian_upload and so should correct the "Set Me Up" commands so users can copy-paste the provided instructions.
ee3f79c

@natoscott natoscott merged commit ee3f79c into performancecopilot:master Nov 9, 2015

@superjamie superjamie deleted the superjamie:bintray-deb-headers branch Nov 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment