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

Allow editing of mirror archive url #677

Merged
merged 2 commits into from
Nov 29, 2017

Conversation

sliverc
Copy link
Contributor

@sliverc sliverc commented Nov 21, 2017

Description of the Change

Allow editing of mirror archive url. This is handy in case a mirror is down or slow and user wants to move to another identical mirror without loosing snapshots.

Checklist

  • unit-test added (if change is algorithm)
  • functional test added/updated (if change is functional)
  • man page updated (if applicable)
  • bash completion updated (if applicable)
  • documentation updated
  • author name in AUTHORS

This is needed in case a mirror has moved or is down and need to move
to new mirror.
@sliverc
Copy link
Contributor Author

sliverc commented Nov 21, 2017

Added documentation in PR aptly-dev/www.aptly.info#57

@@ -0,0 +1,2 @@
Downloading ftp://ftp.ru.debian.org/debian/dists/wheezy/Release...
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be downloading here from ftp.ch.debian.org here? (New URL?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed that's not correct. Preparation of archiveRoot was missing. Should be fixed now.

Copy link
Contributor

@smira smira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thank you!

@smira smira merged commit a037615 into aptly-dev:master Nov 29, 2017
@sliverc sliverc deleted the edit_mirror_archive_url branch November 30, 2017 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants