Merge lp:~robru/phablet-tools/dont-delete-silos into lp:phablet-tools

Proposed by Robert Bruce Park
Status: Merged
Approved by: Michał Sawicz
Approved revision: 286
Merged at revision: 286
Proposed branch: lp:~robru/phablet-tools/dont-delete-silos
Merge into: lp:phablet-tools
Diff against target: 21 lines (+0/-4)
1 file modified
citrain (+0/-4)
To merge this branch: bzr merge lp:~robru/phablet-tools/dont-delete-silos
Reviewer Review Type Date Requested Status
Michał Sawicz Approve
Robert Bruce Park (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Sergio Schvezov Approve
Review via email: mp+224340@code.launchpad.net

Commit message

Don't delete PPAs because this breaks ppa-purge. (LP: #1333184)

To post a comment you must log in.
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

looks good to me; we can top approve once someone acks on testing

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Bruce Park (robru) wrote :

Haha, no Jenkins, you can't pep8 my shell script! Looks like trunk is a little sloppy right now ;-)

Revision history for this message
Robert Bruce Park (robru) wrote :

I tested this myself, does that count?

review: Approve
Revision history for this message
Michał Sawicz (saviq) wrote :

Yes please!

review: Approve
Revision history for this message
Michał Sawicz (saviq) wrote :

Tested, works fine.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'citrain'
2--- citrain 2014-05-28 12:56:04 +0000
3+++ citrain 2014-06-24 18:12:38 +0000
4@@ -64,8 +64,6 @@
5 wget -qO- $HTTP/$SILO/$SOURCES \
6 | perl -ne 's/,//g; s/^Binary: // && print' \
7 | xargs --verbose sudo apt-get install --yes
8- sudo rm -rf /etc/apt/sources.list.d/ci-train*
9- sudo apt-get update -qq
10 ;;
11
12 device-install)
13@@ -81,8 +79,6 @@
14 sudo add-apt-repository $PPA/$SILO
15 sudo apt-get update -qq
16 sudo apt-get dist-upgrade --yes
17- sudo rm -rf /etc/apt/sources.list.d/ci-train*
18- sudo apt-get update -qq
19 ;;
20
21 device-upgrade)

Subscribers

People subscribed via source and target branches