Merge lp:~laney/ubuntu-packaging-guide/new-software-no-sign into lp:ubuntu-packaging-guide

Proposed by Iain Lane
Status: Merged
Merged at revision: 158
Proposed branch: lp:~laney/ubuntu-packaging-guide/new-software-no-sign
Merge into: lp:ubuntu-packaging-guide
Diff against target: 79 lines (+17/-12)
2 files modified
ubuntu-packaging-guide/libraries.rst (+4/-4)
ubuntu-packaging-guide/packaging-new-software.rst (+13/-8)
To merge this branch: bzr merge lp:~laney/ubuntu-packaging-guide/new-software-no-sign
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Approve
Review via email: mp+113834@code.launchpad.net

Description of the change

We should be consistent about signing/not-signing builds at least within the same page.

To post a comment you must log in.
120. By Daniel Holbach

merged lp:~laney/ubuntu-packaging-guide/new-package-no-debuild

121. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

122. By Daniel Holbach

merged lp:~mitya57/ubuntu-packaging-guide/add-guide-css

123. By Daniel Holbach

merged lp:~jjesse/ubuntu-packaging-guide/fixes-bug950227

124. By Daniel Holbach

merged lp:~andrewsomething/ubuntu-packaging-guide/lp942754

Revision history for this message
Daniel Holbach (dholbach) wrote :

Hello Iain,

http://developer.ubuntu.com/packaging/html/getting-set-up.html explains how to set up a GPG key. Personally, I'd assume that people would have read it beforehand as it's one of the first articles in the list. Do you think we should link to it in the Packaging New Software article?

Alternatively I think we should at least explain the impact of "-us -uc", ie they won't be able to upload it to a PPA for example.

Let me know what you think.

125. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

126. By Daniel Holbach

merged lp:~andrewsomething/ubuntu-packaging-guide/fix-single-html

127. By Daniel Holbach

Upload to quantal. (LP: #1004017)

128. By Daniel Holbach

Added COPYING.GPL3 text, fixed a mistake in debian/copyright. Thanks Séb.

129. By Daniel Holbach

merged lp:~takluyver/ubuntu-packaging-guide/python-english

130. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

131. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

132. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

133. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

134. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

135. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

136. By Daniel Holbach

workaround for bug 1043232, disable LANGS, so the build for Japanese PDFs is not attempted

137. By Daniel Holbach

merged lp:~donkirkby/ubuntu-packaging-guide/typos

138. By Benjamin Drung

Remove trailing spaces.

139. By Benjamin Drung

Remove tabs and trailing spaces in debian/copyright.

140. By Benjamin Drung

Run wrap-and-sort.

141. By Benjamin Drung

Prepare the upload to Debian.

We use Ubuntu Developers <email address hidden> as
Maintainer. Ubuntu Developers, who also are DDs and want to do the
uploads to Debian, should add themselves to the Uploader list.

142. By Andrew Starr-Bochicchio

Merge lp:~dholbach/ubuntu-packaging-guide/fix-add-languages-script

143. By Benjamin Drung

Add a tutorial with a specific example of how to fix a bug. (LP: #1045396)

144. By Benjamin Drung

Fix typo from previous merge.

145. By Benjamin Drung

Compress source tarball with xz.

146. By Benjamin Drung

Release 0.2.3 to Debian unstable.

147. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

148. By Daniel Holbach

merged lp:~mitya57/ubuntu-packaging-guide/fixes

149. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

150. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

151. By Daniel Holbach

fix a typo and a syntax error

152. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

153. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

154. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

155. By Benjamin Drung

Fix typo (consider appeared twice).

156. By Launchpad Translations on behalf of ubuntu-packaging-guide-team

Launchpad automatic translations update.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Ping?

157. By Daniel Holbach

merged lp:~doctormo/ubuntu-packaging-guide/ubuntu-styled-images

158. By Iain Lane

Minor language corrections/improvements.

"the compile" sounds odd.

159. By Iain Lane

Don't sign packages until we need to (for upload). Explain a bit what signing is and point to getting started.

160. By Iain Lane

Merge my previous branch

Revision history for this message
Iain Lane (laney) wrote :

I added some more text, let me know what you think

Revision history for this message
Daniel Holbach (dholbach) wrote :

Looks great. Thanks a bunch!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu-packaging-guide/libraries.rst'
2--- ubuntu-packaging-guide/libraries.rst 2012-03-20 22:02:49 +0000
3+++ ubuntu-packaging-guide/libraries.rst 2012-09-17 08:28:18 +0000
4@@ -109,9 +109,9 @@
5
6 $ bzr builddeb -- -nc
7
8-The ``-nc`` will cause it to finish at the end of the compile without removing
9-the build. Change to the build and run ``dpkg-gensymbols`` for the library
10-package::
11+The ``-nc`` will cause it to finish at the end of compilation without removing
12+the built files. Change to the build and run ``dpkg-gensymbols`` for the
13+library package::
14
15 $ cd ../build-area/libnova-0.12.2/
16 $ dpkg-gensymbols -plibnova-0.12-2 > symbols.diff
17@@ -123,7 +123,7 @@
18 Which will create a file named similar to ``dpkg-gensymbolsnY_WWI`` that lists
19 all the symbols. It also lists the current package version. We can remove the
20 packaging version from that listed in the symbols file because new symbols are
21-not generally added by new packaging versions::
22+not generally added by new packaging versions, but by the upstream developers::
23
24 $ sed -i s,-0ubuntu2,, dpkg-gensymbolsnY_WWI
25
26
27=== modified file 'ubuntu-packaging-guide/packaging-new-software.rst'
28--- ubuntu-packaging-guide/packaging-new-software.rst 2012-08-28 23:27:17 +0000
29+++ ubuntu-packaging-guide/packaging-new-software.rst 2012-09-17 08:28:18 +0000
30@@ -51,7 +51,7 @@
31
32 $ make
33
34-If the compile completes successfully you can install and run the program::
35+If compilation completes successfully you can install and run the program::
36
37 $ sudo make install
38 $ hello
39@@ -135,7 +135,7 @@
40 $ cd ../../
41
42 ``bzr builddeb`` is a command to build the package in its current location.
43-The ``-us -uc`` tell it there is no need to GPG sign the compile. The result
44+The ``-us -uc`` tell it there is no need to GPG sign the package. The result
45 will be placed in ``..``.
46
47 You can view the contents of the package with::
48@@ -163,11 +163,16 @@
49 After making a fix to the packaging you can rebuild using ``-nc`` "no clean"
50 without having to build from scratch::
51
52- $ bzr builddeb -- -nc
53+ $ bzr builddeb -- -nc -us -uc
54
55 Having checked that the package builds locally you should ensure it builds on a
56-clean system using ``pbuilder``. If you haven't set up ``pbuilder-dist``
57-yet, :doc:`do so now<./getting-set-up>`::
58+clean system using ``pbuilder``. Since we are going to upload to a PPA
59+(Personal Package Archive) shortly, this upload will need to be *signed* to
60+allow Launchpad to verify that the upload comes from you (you can tell the
61+upload will be signed because the ``-us`` and ``-uc`` flags are not passed to
62+``bzr builddeb`` like they were before). For signing to work you need to have
63+set up GPG. If you haven't set up ``pbuilder-dist`` or GPG yet, :doc:`do so
64+now<./getting-set-up>`::
65
66 $ bzr builddeb -S
67 $ cd ../build-area
68@@ -178,9 +183,9 @@
69
70 $ bzr push lp:~<lp-username>/+junk/hello-package
71
72-Uploading it to a PPA (Personal Package Archive) will ensure it builds
73-and give an easy way for you and others to test the binary packages.
74-You will need to set up a PPA in Launchad then upload with ``dput``::
75+Uploading it to a PPA will ensure it builds and give an easy way for you and
76+others to test the binary packages. You will need to set up a PPA in Launchad
77+then upload with ``dput``::
78
79 $ dput ppa:<lp-username> hello_2.7-0ubuntu1.changes
80

Subscribers

People subscribed via source and target branches