Merge lp:~jbrowne/ubuntu/precise/elbcli/precise into lp:~awstools-dev/ubuntu/precise/elbcli/precise

Proposed by Jim Browne
Status: Merged
Merged at revision: 21
Proposed branch: lp:~jbrowne/ubuntu/precise/elbcli/precise
Merge into: lp:~awstools-dev/ubuntu/precise/elbcli/precise
Diff against target: 122 lines (+37/-31)
6 files modified
.bzrignore (+1/-0)
credential-file-path.template (+4/-1)
debian/README.source (+5/-0)
debian/control (+1/-1)
debian/copyright (+1/-1)
debian/patches/replace-commands.patch (+25/-28)
To merge this branch: bzr merge lp:~jbrowne/ubuntu/precise/elbcli/precise
Reviewer Review Type Date Requested Status
Scott Moser Pending
Review via email: mp+106748@code.launchpad.net

Description of the change

Changes to bring this package in sync with the state of the similar RDS CLI package.

Remaining differences:

- This package has no debian/watch file (reason noted in debian/README.source)
- The added Makefile builds the .fixperms target (removed in RDS package)

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added file '.bzrignore'
--- .bzrignore 1970-01-01 00:00:00 +0000
+++ .bzrignore 2012-05-22 06:55:19 +0000
@@ -0,0 +1,1 @@
1.pc
02
=== removed file 'README.txt'
=== modified file 'credential-file-path.template'
--- credential-file-path.template 2011-12-09 15:50:39 +0000
+++ credential-file-path.template 2012-05-22 06:55:19 +0000
@@ -1,2 +1,5 @@
1# Enter the AWS Keys without the < or >
2# These can be found at http://aws.amazon.com under Account->Security Credentials
1AWSAccessKeyId=<Write your AWS access ID>3AWSAccessKeyId=<Write your AWS access ID>
2AWSSecretKey=<Write your AWS secret key>
3\ No newline at end of file4\ No newline at end of file
5AWSSecretKey=<Write your AWS secret key>
6
47
=== modified file 'debian/README.source'
--- debian/README.source 2011-12-13 14:30:58 +0000
+++ debian/README.source 2012-05-22 06:55:19 +0000
@@ -77,3 +77,8 @@
7777
78Currently that list needs to be manually synced to the dependencies in78Currently that list needs to be manually synced to the dependencies in
79debian/control.79debian/control.
80
81== Testing ==
82debian/README.example contains an example of how to test some function of the
83package.
84
8085
=== modified file 'debian/control'
--- debian/control 2011-12-13 16:24:19 +0000
+++ debian/control 2012-05-22 06:55:19 +0000
@@ -5,7 +5,7 @@
5Build-Depends: debhelper (>= 7),5Build-Depends: debhelper (>= 7),
6 java6-runtime-headless | default-jre-headless,6 java6-runtime-headless | default-jre-headless,
7 quilt7 quilt
8Standards-Version: 3.9.18Standards-Version: 3.9.2
9Homepage: http://aws.amazon.com/developertools/25369Homepage: http://aws.amazon.com/developertools/2536
1010
11Package: elbcli11Package: elbcli
1212
=== modified file 'debian/copyright'
--- debian/copyright 2011-12-13 15:51:33 +0000
+++ debian/copyright 2012-05-22 06:55:19 +0000
@@ -151,7 +151,7 @@
151 SOFTWARE.151 SOFTWARE.
152152
153Copyright: Copyright (c) 2000 The Apache Software Foundation.153Copyright: Copyright (c) 2000 The Apache Software Foundation.
154Files: jdom-1.0.jar154Files: lib/jdom-1.0.jar
155License: Apache-1.1155License: Apache-1.1
156 The Apache Software License, Version 1.1156 The Apache Software License, Version 1.1
157 .157 .
158158
=== modified file 'debian/patches/replace-commands.patch'
--- debian/patches/replace-commands.patch 2011-12-13 14:52:14 +0000
+++ debian/patches/replace-commands.patch 2012-05-22 06:55:19 +0000
@@ -1,35 +1,32 @@
1Subject: re-write the wrapper commands from original source1Description: re-write the wrapper commands from original source
2This patch just rewrites the wrapper commands that are provided2 This patch just rewrites the wrapper commands that are provided
3in the zip file with one more tailored for installation.3 in the zip file with one more tailored for installation.
44
5To regenerate this patch:5To regenerate this patch:
6 files=$(for f in bin/*; do 6 files=$(for f in bin/*; do
7 case "${f##*/}" in (*.cmd|service|elb-cmd) continue;; esac7 case "${f##*/}" in (*.cmd|service|elb-cmd) continue;; esac
8 echo $f; done)8 echo $f; done)
9 # the only program that has a different argument to 'elb-cmd'9 # the only program that has a different argument to 'elb-cmd'
10 # than its filename is 'elb-version'. That has to be shortened to10 # than its filename is 'elb-version'. That has to be shortened to
11 # 'version'11 # 'version'
12 quilt delete replace-commands.patch12 patch=debian/patches/replace-commands.patch
13 rm debian/patches/replace-commands.patch13 quilt header ${patch##*/} > header.tmp
14 quilt new replace-commands.patch14 quilt delete ${patch##*/}
15 quilt add ${files}15 rm $patch
16 for f in ${files}; do16 quilt new ${patch##*/}
17 [ "${f}" = "bin/elb-version" ] && cmd="version" || cmd="${f#*/}"17 quilt add ${files}
18 cat debian/wrapper-header > "${f}" && \18 for f in ${files}; do
19 printf 'exec "$service" %s "$@"\n' "$cmd" >> "${f}"19 [ "${f}" = "bin/elb-version" ] && cmd="version" || cmd="${f#*/}"
20 done20 cat debian/wrapper-header > "${f}" && \
21 quilt refresh21 printf 'exec "$service" %s "$@"\n' "$cmd" >> "${f}"
2222 done
23To verify that each command called 'elb-cmd' with its command name, without23 quilt refresh
24this patch applied, you can do something like:24 # replace header
25 for f in $files; do 25 quilt header ${patch##*/} -r < header.tmp
26 cmd=$(awk '$1 ~ /elb-cmd$/ { print $2 }' $f); 26 rm -f header.tmp
27 [ "${f#*/}" != "$cmd" ] && echo -n "DIFFERENT: "
28 echo $f: $cmd
29 done
30
31Origin: Scott Moser <smoser@ubuntu.com>27Origin: Scott Moser <smoser@ubuntu.com>
32Last-Update: 2011-12-1328Last-Update: 2011-12-13
29
33--- a/bin/elb-apply-security-groups-to-lb30--- a/bin/elb-apply-security-groups-to-lb
34+++ b/bin/elb-apply-security-groups-to-lb31+++ b/bin/elb-apply-security-groups-to-lb
35@@ -1,7 +1,5 @@32@@ -1,7 +1,5 @@

Subscribers

People subscribed via source and target branches