Merge lp:~jjesse/ubuntu-packaging-guide/fixes-bug978493 into lp:ubuntu-packaging-guide

Proposed by Jonathan Jesse
Status: Merged
Merged at revision: 110
Proposed branch: lp:~jjesse/ubuntu-packaging-guide/fixes-bug978493
Merge into: lp:ubuntu-packaging-guide
Diff against target: 12 lines (+1/-1)
1 file modified
ubuntu-packaging-guide/fixing-a-bug.rst (+1/-1)
To merge this branch: bzr merge lp:~jjesse/ubuntu-packaging-guide/fixes-bug978493
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Approve
Review via email: mp+107574@code.launchpad.net

Description of the change

Proposing a fix to 978493 replaced bzr commit with debcommit

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

Thanks for your work on this.

I just checked and there's a number of other occurences, where we might want to make sure that 'debcommit' is used. Generally wherever debian/changelog already contains the documentation of the change, 'debcommit' can be uesd safely.

Would you mind reviewing the guide again and see if we need to update other places as well?

daniel@daydream:~/bzr/ubuntu-packaging-guide$ grep -r "bzr commit" ubuntu-packaging-guide/ | grep -v -- "-m"
ubuntu-packaging-guide/patches-to-packages.rst: $ bzr commit
ubuntu-packaging-guide/udd-sponsorship.rst:<link-via-changelog>` using ``dch`` and ``bzr commit``. However, the branch
ubuntu-packaging-guide/udd-sponsorship.rst: $ bzr commit
ubuntu-packaging-guide/udd-uploading.rst: $ bzr commit --author "Bob Dobbs <email address hidden>"
ubuntu-packaging-guide/udd-working.rst: bzr commit
ubuntu-packaging-guide/udd-latest.rst:revert``. Once you are happy with the changes then use ``bzr commit``.
daniel@daydream:~/bzr/ubuntu-packaging-guide$

review: Needs Information
Revision history for this message
Jonathan Jesse (jjesse) wrote :

Yes I will make sure all references are right
On May 29, 2012 3:35 AM, "Daniel Holbach" <email address hidden> wrote:

> Review: Needs Information
>
> Thanks for your work on this.
>
> I just checked and there's a number of other occurences, where we might
> want to make sure that 'debcommit' is used. Generally wherever
> debian/changelog already contains the documentation of the change,
> 'debcommit' can be uesd safely.
>
> Would you mind reviewing the guide again and see if we need to update
> other places as well?
>
> daniel@daydream:~/bzr/ubuntu-packaging-guide$ grep -r "bzr commit"
> ubuntu-packaging-guide/ | grep -v -- "-m"
> ubuntu-packaging-guide/patches-to-packages.rst: $ bzr commit
> ubuntu-packaging-guide/udd-sponsorship.rst:<link-via-changelog>` using
> ``dch`` and ``bzr commit``. However, the branch
> ubuntu-packaging-guide/udd-sponsorship.rst: $ bzr commit
> ubuntu-packaging-guide/udd-uploading.rst: $ bzr commit --author "Bob
> Dobbs <email address hidden>"
> ubuntu-packaging-guide/udd-working.rst: bzr commit
> ubuntu-packaging-guide/udd-latest.rst:revert``. Once you are happy with
> the changes then use ``bzr commit``.
> daniel@daydream:~/bzr/ubuntu-packaging-guide$
>
> --
>
> https://code.launchpad.net/~jjesse/ubuntu-packaging-guide/fixes-bug978493/+merge/107574
> You are the owner of lp:~jjesse/ubuntu-packaging-guide/fixes-bug978493.
>

Revision history for this message
Jonathan Jesse (jjesse) wrote :

In patches-to-packages it is dealing w/ a branch and is ok, doesn't need to be changed to debcomit

In udd-sponsorship it is talking about bazaar and is correct

In udd-working it is talking about bazar and is correct

In udd-uploading this is correct for bzr commit

So it looks like it should be good to

Let me know

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

Thanks a lot, merged!

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/fixing-a-bug.rst'
2--- ubuntu-packaging-guide/fixing-a-bug.rst 2012-05-21 10:31:14 +0000
3+++ ubuntu-packaging-guide/fixing-a-bug.rst 2012-05-28 02:22:19 +0000
4@@ -181,7 +181,7 @@
5
6 With the changelog entry written and saved, you can just run::
7
8- bzr commit
9+ debcommit
10
11 and the change will be committed (locally) with your changelog entry as a
12 commit message.

Subscribers

People subscribed via source and target branches