merge finish misses removing ":" from an LP bug
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| git-ubuntu |
New
|
Undecided
|
Unassigned | ||
Bug Description
I have this series of commits applied on top of new/debian. I abbreviated the text a bit for clarity. Please observe the three commits that I highlighted which have LP bug references and are all part of a "drop" section:
commit 796af0741bbecc2
- d/p/CVE-
commit 2162b4bb6d3da0a
- d/p/CVE-
commit f589bbb8f5622b3
- d/control, d/rules: add json support for the statistics channels.
(LP: #1669193)
commit ebbc87a504757a2
* Drop:
- rules: Fix path to libsofthsm2.so. (LP: #1685780)
commit 7772ca09166f633
- Add RemainAfterExit to bind9-resolvconf unit configuration file
(LP: #1536181).
commit abbc90262a20671
(...)
When I run merge finish pkg/ubuntu/devel, this is the changelog that it produces:
* Merge with Debian unstable. Remaining changes:
- Add RemainAfterExit to bind9-resolvconf unit configuration file
(LP #1536181).
* Drop:
- rules: Fix path to libsofthsm2.so. (LP: #1685780)
[adopted in 1:9.10.6+dfsg-5]
- d/control, d/rules: add json support for the statistics channels.
(LP #1669193)
[adopted in 1:9.10.6+dfsg-5]
- d/p/CVE-
regression (RT #44318) introduced with the CVE-2016-8864.patch
and fixed in CVE-2016-
[applied upstream]
- d/p/CVE-
introduced with the CVE-2016-8864.patch and fixed in
CVE-
[applied upstream]
It correctly removed the ":" from the LP bugs in all but the "Fix path to libsofthsm2" entry.
| Nish Aravamudan (nacc) wrote Re: [Bug 1745374] [NEW] merge finish misses removing ":" from an LP bug | #1 |
| tags: | added: merge |
Ah funny, it's because we think it's a top-level entry, which we don't
modify :) I'll give it a think.
On Thu, Jan 25, 2018 at 5:44 AM, Andreas Hasenack <email address hidden> wrote:
> Public bug reported:
>
> I have this series of commits applied on top of new/debian. I
> abbreviated the text a bit for clarity. Please observe the three commits
> that I highlighted which have LP bug references and are all part of a
> "drop" section:
>
> commit 796af0741bbecc2
> - d/p/CVE-
>
> commit 2162b4bb6d3da0a
> - d/p/CVE-
>
> commit f589bbb8f5622b3
> - d/control, d/rules: add json support for the statistics channels.
> (LP: #1669193)
>
> commit ebbc87a504757a2
> * Drop:
> - rules: Fix path to libsofthsm2.so. (LP: #1685780)
>
> commit 7772ca09166f633
> - Add RemainAfterExit to bind9-resolvconf unit configuration file
> (LP: #1536181).
>
> commit abbc90262a20671
> (...)
>
> When I run merge finish pkg/ubuntu/devel, this is the changelog that it produces:
> * Merge with Debian unstable. Remaining changes:
> - Add RemainAfterExit to bind9-resolvconf unit configuration file
> (LP #1536181).
> * Drop:
> - rules: Fix path to libsofthsm2.so. (LP: #1685780)
> [adopted in 1:9.10.6+dfsg-5]
> - d/control, d/rules: add json support for the statistics channels.
> (LP #1669193)
> [adopted in 1:9.10.6+dfsg-5]
> - d/p/CVE-
> regression (RT #44318) introduced with the CVE-2016-8864.patch
> and fixed in CVE-2016-
> [applied upstream]
> - d/p/CVE-
> introduced with the CVE-2016-8864.patch and fixed in
> CVE-2016-
> [applied upstream]
>
> It correctly removed the ":" from the LP bugs in all but the "Fix path
> to libsofthsm2" entry.
>
> ** Affects: usd-importer
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are a member of Ubuntu
> Server Dev import team, which is subscribed to usd-importer.
> https:/
>
> Title:
> merge finish misses removing ":" from an LP bug
>
> To manage notifications about this bug go to:
> https:/