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
diff --git a/build-tools/build-sources-list b/build-tools/build-sources-list
index e33e781..29fba67 100755
--- a/build-tools/build-sources-list
+++ b/build-tools/build-sources-list
@@ -115,26 +115,24 @@ EOM
115 emit_ppa_overlay "snappy-dev" "$ppa_overlay" "${ppa_overlay_release}"115 emit_ppa_overlay "snappy-dev" "$ppa_overlay" "${ppa_overlay_release}"
116116
117 # ESM overlay ppas117 # ESM overlay ppas
118 elif echo "$r" | grep -q "esm" ; then118 elif echo "$r" | grep -q "/esm" ; then
119 if echo "$r" | grep -E "trusty|precise"; then119 ppa_overlay_release=$(echo "$r" | cut -f 1 -d '/')
120 ppa_overlay_release=$(echo "$r" | cut -f 1 -d '/')120 ppa_overlay="esm"
121 else121 if [ $ppa_overlay_release = "trusty" ]; then
122 ppa_overlay_release=$(echo "$r" | cut -f 2 -d '/')
123 fi
124 if [ "$ppa_overlay_release" = "precise" ] ; then
125 ppa_overlay="esm"
126 emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
127 echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
128 else
129 # newer ESM releases are in the esm-infra-security and
130 # esm-apps-security ppas
131 ppa_overlay="esm-infra-security"122 ppa_overlay="esm-infra-security"
132 emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
133 echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
134 ppa_overlay="esm-apps-security"
135 emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
136 echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
137 fi123 fi
124
125 emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
126 echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
127 # newer ESM releases are in the esm-infra-security and
128 # esm-apps-security ppas
129 elif echo "$r" | grep -q "esm-" ; then
130 ppa_overlay_release=$(echo "$r" | cut -f 2 -d '/')
131 product=$(echo "$r" | cut -f 1 -d '/')
132 ppa_overlay="$product-security"
133
134 emit_private_ppa_overlay "ubuntu-esm" "$ppa_overlay" "${ppa_overlay_release}"
135 echo "# Requires gpg keyid DBB1FC89762BF6B96707C4059BC0A1A1622CF918"
138 # normal ubuntu releases.136 # normal ubuntu releases.
139 else137 else
140 if [ "$r" = "precise" ] ; then138 if [ "$r" = "precise" ] ; then

Subscribers

People subscribed via source and target branches