git-import-dsc merges to master despite --debian-branch

Bug #637286 reported by Rolf Leggewie
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
git-buildpackage (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: git-buildpackage

git-import-dsc will merge to master even when --debian-branch is explicitly set. Run the attached script as a test-case.

Tags: patch
Revision history for this message
Rolf Leggewie (r0lf) wrote :
Revision history for this message
Guido Günther (agx) wrote : Re: [Bug 637286] [NEW] git-import-dsc merges to master despite --debian-branch

On Mon, Sep 13, 2010 at 03:41:13PM -0000, Launchpad Bug Tracker wrote:
> You have been subscribed to a public bug by Rolf Leggewie (r0lf):
>
> Binary package hint: git-buildpackage
>
> git-import-dsc will merge to master even when --debian-branch is
> explicitly set. Run the attached script as a test-case.

Attached patch should fix this.
 -- Guido

Revision history for this message
Guido Günther (agx) wrote :

On Mon, Sep 13, 2010 at 03:41:13PM -0000, Launchpad Bug Tracker wrote:
> You have been subscribed to a public bug by Rolf Leggewie (r0lf):
>
> Binary package hint: git-buildpackage
>
> git-import-dsc will merge to master even when --debian-branch is
> explicitly set. Run the attached script as a test-case.
I've pushed the fix to git. Thanks for the testcase.
 -- Guido

Revision history for this message
Rolf Leggewie (r0lf) wrote :

Thank you very much.

Changed in git-buildpackage (Ubuntu):
importance: Undecided → Medium
status: New → Fix Committed
Revision history for this message
Rolf Leggewie (r0lf) wrote :

Are you planning to make another release to experimental anytime soon? I guess rather than backporting the fix to maverick I'd prefer to sync. Once Maverick has the fix, I'll seek an SRU exception for lucid.

Revision history for this message
Guido Günther (agx) wrote : Re: [Bug 637286] Re: git-import-dsc merges to master despite --debian-branch

On Tue, Sep 14, 2010 at 02:52:09AM -0000, Rolf Leggewie wrote:
> Are you planning to make another release to experimental anytime soon?
> I guess rather than backporting the fix to maverick I'd prefer to sync.
> Once Maverick has the fix, I'll seek an SRU exception for lucid.
No, not that soon. I have some other stuff pending which I need to
finish first and Squeeze eats up the little free time I have a at the
momene.

BTW. I've comitted a better fix for this issue which avoids the
unnecessary branch switch.
 -- Guido

Revision history for this message
Guido Günther (agx) wrote :

This is fixed in 0.5.9

tags: added: patch
Revision history for this message
Rolf Leggewie (r0lf) wrote :

Guido, that's great news. Thank you very much. I shall request syncing the latest release for Maverick.

Revision history for this message
Rolf Leggewie (r0lf) wrote :

0.5.9 has been in Maverick for about three weeks, it seems. I shall try to verify the fix in the next days.

Revision history for this message
Rolf Leggewie (r0lf) wrote :

Is this the same problem as

"cd /tmp && mkdir capifax && wget wget http://capifax.v3v.de/dl/capifax-0.7.3.tar.bz2 && cd capifax && git init && git-import-orig --pristine-tar --no-merge ../capifax-0.7.3.tar.bz2" which still creates the master branch and merges to it despite the call not to do so?

Revision history for this message
Guido Günther (agx) wrote :

On Sat, Oct 30, 2010 at 09:19:23PM -0000, Rolf Leggewie wrote:
> Is this the same problem as
>
> "cd /tmp && mkdir capifax && wget wget
> http://capifax.v3v.de/dl/capifax-0.7.3.tar.bz2 && cd capifax && git init
> && git-import-orig --pristine-tar --no-merge ../capifax-0.7.3.tar.bz2"
> which still creates the master branch and merges to it despite the call
> not to do so?
Nope. This is unrelated. In fact the issue from above is not a bug since
the _inital_ import always goes to master (the upstream branch is then
created from there). Everything else would pretty much break what's
expected from a git repo.
Cheers,
 -- Guido

Iain Lane (laney)
Changed in git-buildpackage (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.