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
1=== added file '.bzrignore'
2--- .bzrignore 1970-01-01 00:00:00 +0000
3+++ .bzrignore 2012-05-22 06:55:19 +0000
4@@ -0,0 +1,1 @@
5+.pc
6
7=== removed file 'README.txt'
8=== modified file 'credential-file-path.template'
9--- credential-file-path.template 2011-12-09 15:50:39 +0000
10+++ credential-file-path.template 2012-05-22 06:55:19 +0000
11@@ -1,2 +1,5 @@
12+# Enter the AWS Keys without the < or >
13+# These can be found at http://aws.amazon.com under Account->Security Credentials
14 AWSAccessKeyId=<Write your AWS access ID>
15-AWSSecretKey=<Write your AWS secret key>
16\ No newline at end of file
17+AWSSecretKey=<Write your AWS secret key>
18+
19
20=== modified file 'debian/README.source'
21--- debian/README.source 2011-12-13 14:30:58 +0000
22+++ debian/README.source 2012-05-22 06:55:19 +0000
23@@ -77,3 +77,8 @@
24
25 Currently that list needs to be manually synced to the dependencies in
26 debian/control.
27+
28+== Testing ==
29+debian/README.example contains an example of how to test some function of the
30+package.
31+
32
33=== modified file 'debian/control'
34--- debian/control 2011-12-13 16:24:19 +0000
35+++ debian/control 2012-05-22 06:55:19 +0000
36@@ -5,7 +5,7 @@
37 Build-Depends: debhelper (>= 7),
38 java6-runtime-headless | default-jre-headless,
39 quilt
40-Standards-Version: 3.9.1
41+Standards-Version: 3.9.2
42 Homepage: http://aws.amazon.com/developertools/2536
43
44 Package: elbcli
45
46=== modified file 'debian/copyright'
47--- debian/copyright 2011-12-13 15:51:33 +0000
48+++ debian/copyright 2012-05-22 06:55:19 +0000
49@@ -151,7 +151,7 @@
50 SOFTWARE.
51
52 Copyright: Copyright (c) 2000 The Apache Software Foundation.
53-Files: jdom-1.0.jar
54+Files: lib/jdom-1.0.jar
55 License: Apache-1.1
56 The Apache Software License, Version 1.1
57 .
58
59=== modified file 'debian/patches/replace-commands.patch'
60--- debian/patches/replace-commands.patch 2011-12-13 14:52:14 +0000
61+++ debian/patches/replace-commands.patch 2012-05-22 06:55:19 +0000
62@@ -1,35 +1,32 @@
63-Subject: re-write the wrapper commands from original source
64-This patch just rewrites the wrapper commands that are provided
65-in the zip file with one more tailored for installation.
66+Description: re-write the wrapper commands from original source
67+ This patch just rewrites the wrapper commands that are provided
68+ in the zip file with one more tailored for installation.
69
70 To regenerate this patch:
71- files=$(for f in bin/*; do
72- case "${f##*/}" in (*.cmd|service|elb-cmd) continue;; esac
73- echo $f; done)
74- # the only program that has a different argument to 'elb-cmd'
75- # than its filename is 'elb-version'. That has to be shortened to
76- # 'version'
77- quilt delete replace-commands.patch
78- rm debian/patches/replace-commands.patch
79- quilt new replace-commands.patch
80- quilt add ${files}
81- for f in ${files}; do
82- [ "${f}" = "bin/elb-version" ] && cmd="version" || cmd="${f#*/}"
83- cat debian/wrapper-header > "${f}" && \
84- printf 'exec "$service" %s "$@"\n' "$cmd" >> "${f}"
85- done
86- quilt refresh
87-
88-To verify that each command called 'elb-cmd' with its command name, without
89-this patch applied, you can do something like:
90- for f in $files; do
91- cmd=$(awk '$1 ~ /elb-cmd$/ { print $2 }' $f);
92- [ "${f#*/}" != "$cmd" ] && echo -n "DIFFERENT: "
93- echo $f: $cmd
94- done
95-
96+ files=$(for f in bin/*; do
97+ case "${f##*/}" in (*.cmd|service|elb-cmd) continue;; esac
98+ echo $f; done)
99+ # the only program that has a different argument to 'elb-cmd'
100+ # than its filename is 'elb-version'. That has to be shortened to
101+ # 'version'
102+ patch=debian/patches/replace-commands.patch
103+ quilt header ${patch##*/} > header.tmp
104+ quilt delete ${patch##*/}
105+ rm $patch
106+ quilt new ${patch##*/}
107+ quilt add ${files}
108+ for f in ${files}; do
109+ [ "${f}" = "bin/elb-version" ] && cmd="version" || cmd="${f#*/}"
110+ cat debian/wrapper-header > "${f}" && \
111+ printf 'exec "$service" %s "$@"\n' "$cmd" >> "${f}"
112+ done
113+ quilt refresh
114+ # replace header
115+ quilt header ${patch##*/} -r < header.tmp
116+ rm -f header.tmp
117 Origin: Scott Moser <smoser@ubuntu.com>
118 Last-Update: 2011-12-13
119+
120 --- a/bin/elb-apply-security-groups-to-lb
121 +++ b/bin/elb-apply-security-groups-to-lb
122 @@ -1,7 +1,5 @@

Subscribers

People subscribed via source and target branches