Merge lp:~josephjamesmills/ubuntu-packaging-guide/fixed-bug975026 into lp:ubuntu-packaging-guide

Proposed by Joseph Mills
Status: Rejected
Rejected by: Simon Quigley
Proposed branch: lp:~josephjamesmills/ubuntu-packaging-guide/fixed-bug975026
Merge into: lp:ubuntu-packaging-guide
Diff against target: 45 lines (+18/-9)
2 files modified
debian/changelog (+6/-0)
ubuntu-packaging-guide/getting-set-up.rst (+12/-9)
To merge this branch: bzr merge lp:~josephjamesmills/ubuntu-packaging-guide/fixed-bug975026
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Needs Fixing
Review via email: mp+106950@code.launchpad.net

Description of the change

Made gpg section more clear

To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

A few small things I noticed.

 - can you add a space (" ") after each sentence and closed parentheses?
 - I'm not sure we'd want to recommend "testing emails" - if we do, we
   should also explain how to add new key IDs or remove ones you don't
   like any more
 - maybe we should link to some password advice instead of giving it
   ourselves - I immediately thought of http://xkcd.com/936/ :-)
 - personally I think I'd prefer "which" to "what" in the first
   sentence, but I'm not a native speaker, so somebody else might want
   to weigh in.

Thanks for your work on this, you made this easier to read! :)

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

I'll mark these merge proposals as 'work in progress' to get them off the review list.

Revision history for this message
Simon Quigley (tsimonq2) wrote :

Rejecting due to the fact that this has been rewritten and feedback was requested but nothing was done to fix it.

Unmerged revisions

100. By Joseph Mills

Bug Bug #975026 changed lines Changed lines 82 thou 93 in the file ubuntu-packaging-guide/getting-set-up.rst

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-03-12 22:28:59 +0000
+++ debian/changelog 2012-05-23 07:22:19 +0000
@@ -1,3 +1,9 @@
1ubuntu-packaging-guide (0.1ubuntu1) quantal; urgency=low
2
3 * ubuntu-packaging-guide/getting-set-up.rst: Changed lines 82 thou 93 and made backup called getting-set-up.rst~ (LP: #975026)
4
5 -- Joseph Mills <josephjamesmills@gmail.com> Sat, 19 May 2012 02:28:39 -0400
6
1ubuntu-packaging-guide (0.1) UNRELEASED; urgency=low7ubuntu-packaging-guide (0.1) UNRELEASED; urgency=low
28
3 * Initial release.9 * Initial release.
410
=== modified file 'ubuntu-packaging-guide/getting-set-up.rst'
--- ubuntu-packaging-guide/getting-set-up.rst 2012-03-20 22:02:49 +0000
+++ ubuntu-packaging-guide/getting-set-up.rst 2012-05-23 07:22:19 +0000
@@ -79,15 +79,18 @@
7979
80 $ gpg --gen-key80 $ gpg --gen-key
8181
82GPG will first ask you which kind of key you want to generate. Choosing the82GPG will first ask you what kind of key you would like to generate. Choosing
83default (RSA and DSA) is fine. Next it will ask you about the keysize. The83the default RSA and DSA (options 1)is fine.GPG will now ask you about the
84default (currently 2048) is fine, but 4096 is more secure. Afterwards, it will84keysize, that you would like to create. The default (currently 2048) is fine,
85ask you if you want it to expire the key at some stage. It is safe to say "0",85but 4096 is better because it is more secure. GPG will now ask how long you
86which means the key will never expire. The last questions will be about your86would like to keep this GPG key. It is safe to say “0”, which means the key
87name and email address. Just pick the ones you are going to use for Ubuntu87will never expire.GPG will now ask for your email address.It is best that you
88development here, you can add additional email addresses later on. Adding a88enter in the email address that you use to sign into Launchpad. Or if you
89comment is not necessary. Then you will have to set a passphrase, choose a89like you may pick a "testing" one for now, As you can add additional email
90safe one (a passphrase is just a password which is allowed to include spaces). 90addresses later on.Next GPG will ask if you would like too add a comment but
91this is not necessary.You will now have to set a passphrase. A passphrase is
92a fancy word for a password that can contain spaces and charecters like *#)&:.
93Please choose a strong password.
9194
92Now GPG will create a key for you, which can take a little bit of time; it 95Now GPG will create a key for you, which can take a little bit of time; it
93needs random bytes, so if you give the system some work to do it will be 96needs random bytes, so if you give the system some work to do it will be

Subscribers

People subscribed via source and target branches