Merge lp:~wgrant/launchpad/genericise-cron.publish-ppa into lp:launchpad

Proposed by William Grant
Status: Merged
Merged at revision: 18800
Proposed branch: lp:~wgrant/launchpad/genericise-cron.publish-ppa
Merge into: lp:launchpad
Diff against target: 21 lines (+5/-9)
1 file modified
cronscripts/publishing/cron.publish-ppa (+5/-9)
To merge this branch: bzr merge lp:~wgrant/launchpad/genericise-cron.publish-ppa
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+357123@code.launchpad.net

Commit message

Make cron.publish-ppa take distro selection criteria as arguments.

Description of the change

This is live on production already.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cronscripts/publishing/cron.publish-ppa'
2--- cronscripts/publishing/cron.publish-ppa 2014-08-05 10:37:07 +0000
3+++ cronscripts/publishing/cron.publish-ppa 2018-10-19 05:27:09 +0000
4@@ -10,12 +10,8 @@
5
6 LPCURRENT=`dirname $0`/../..
7
8-# Publish Ubuntu PPAs.
9-$LPCURRENT/scripts/process-accepted.py -v --ppa -d ubuntu
10-$LPCURRENT/scripts/publish-distro.py -v --ppa -d ubuntu
11-$LPCURRENT/scripts/publish-distro.py -v --private-ppa -d ubuntu
12-
13-# Publish PPAs for other distros.
14-$LPCURRENT/scripts/process-accepted.py -v --ppa --all-derived
15-$LPCURRENT/scripts/publish-distro.py -v --ppa --all-derived
16-$LPCURRENT/scripts/publish-distro.py -v --private-ppa --all-derived
17+for DISTRO_ARG in "$@"; do
18+ $LPCURRENT/scripts/process-accepted.py -v --ppa $DISTRO_ARG
19+ $LPCURRENT/scripts/publish-distro.py -v --ppa $DISTRO_ARG
20+ $LPCURRENT/scripts/publish-distro.py -v --private-ppa $DISTRO_ARG
21+done