Merge ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:fix-build-sources-list-v2 into ubuntu-security-tools:master

Proposed by Spyros Seimenis
Status: Merged
Merge reported by: Spyros Seimenis
Merged at revision: 6f2e94ec7b5035e1b0ed9f67faeba28568ee08d2
Proposed branch: ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:fix-build-sources-list-v2
Merge into: ubuntu-security-tools:master
Diff against target: 47 lines (+16/-18)
1 file modified
build-tools/build-sources-list (+16/-18)
Reviewer Review Type Date Requested Status
Rodrigo Figueiredo Zaiden Approve
Review via email: mp+422380@code.launchpad.net
To post a comment you must log in.
e46e999... by Spyros Seimenis

Rework build-sources-list to handle trusty apps

17d5895... by Spyros Seimenis

build-sources-list: Make the code cleaner

6f2e94e... by Spyros Seimenis

build-sources-list: Fix trusty infra case

Revision history for this message
Rodrigo Figueiredo Zaiden (rodrigo-zaiden) wrote :

LGTM! Thanks :)

review: Approve
Revision history for this message
Spyros Seimenis (sespiros) wrote :

Merged (by squash and rebase). Thanks for reviewing!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/build-tools/build-sources-list b/build-tools/build-sources-list
2index e33e781..29fba67 100755
3--- a/build-tools/build-sources-list
4+++ b/build-tools/build-sources-list
5@@ -115,26 +115,24 @@ EOM
6 emit_ppa_overlay "snappy-dev" "$ppa_overlay" "${ppa_overlay_release}"
7
8 # ESM overlay ppas
9- elif echo "$r" | grep -q "esm" ; then
10- if echo "$r" | grep -E "trusty|precise"; then
11- ppa_overlay_release=$(echo "$r" | cut -f 1 -d '/')
12- else
13- ppa_overlay_release=$(echo "$r" | cut -f 2 -d '/')
14- fi
15- if [ "$ppa_overlay_release" = "precise" ] ; then
16- ppa_overlay="esm"
17- emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
18- echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
19- else
20- # newer ESM releases are in the esm-infra-security and
21- # esm-apps-security ppas
22+ elif echo "$r" | grep -q "/esm" ; then
23+ ppa_overlay_release=$(echo "$r" | cut -f 1 -d '/')
24+ ppa_overlay="esm"
25+ if [ $ppa_overlay_release = "trusty" ]; then
26 ppa_overlay="esm-infra-security"
27- emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
28- echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
29- ppa_overlay="esm-apps-security"
30- emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
31- echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
32 fi
33+
34+ emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
35+ echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
36+ # newer ESM releases are in the esm-infra-security and
37+ # esm-apps-security ppas
38+ elif echo "$r" | grep -q "esm-" ; then
39+ ppa_overlay_release=$(echo "$r" | cut -f 2 -d '/')
40+ product=$(echo "$r" | cut -f 1 -d '/')
41+ ppa_overlay="$product-security"
42+
43+ emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
44+ echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
45 # normal ubuntu releases.
46 else
47 if [ "$r" = "precise" ] ; then

Subscribers

People subscribed via source and target branches