-
Notifications
You must be signed in to change notification settings - Fork 14
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
Convert option.snapshots to Hash before passing to publish_update #147
Convert option.snapshots to Hash before passing to publish_update #147
Conversation
@sepulworld Any idea on when you can get around to this? |
I can look at this soon sorry I missed the notification that there was a pull request |
That would be great, thanks. |
Let me do some testing on it this week. At the latest Ill have a response and/or merge of this patch this weekend. |
Thank you for catching this bug. I was able to replicate it on the test/Dockerfile environment. Before I merge can you add c.example 'Update snapshot',
'aptly-cli publish_update --snapshots testrepo_single_snap_to_pub,main --prefix . --distribution=precise --gpg_skip' right after line 322 in bin/aptly-cli |
@@ -322,7 +322,7 @@ command :publish_update do |c| | |||
c.example 'description', 'aptly-cli publish_update --forceoverwrite --distribution precise' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add example of a 'snapshot update' here perhaps.
c.example 'update snapshot',
'aptly-cli publish_update --snapshots testrepo_single_snap_to_pub,main --prefix . --distribution=precise --gpg_skip'
Hi, I can't seem to get
publish_update
to work. With this patch applied I can run it like this:aptly-cli publish_update --snapshots jessie-snapshot,main --prefix . --distribution=jessie
. I'm pretty sure I'm just too stupid to guess the correct syntax to make it work without the patch, but here is a pull request just in case. If I'm just not using it right, please tell me.