Skip to contentSkip to conversation

stgit: cannot push a patch - Python trace dump

DISCUSSION Auke Kok 3 participants 4 messages Jun 15, 2016

lore
Share & tools
lore

Auke Kok

author
#0 lore raw

Hi all,

I'm preparing patches for upstream and am haunted by an apparent patch breaking stg. The problem appears to be one or more of my patches breaking a push:

$ stg push
Traceback (most recent call last):
  File "/usr/bin/stg", line 43, in ?
    main()
  File "/usr/lib/python2.4/site-packages/stgit/main.py", line 255, in main
    command.func(parser, options, args)
  File "/usr/lib/python2.4/site-packages/stgit/commands/push.py", line 101, in
func
    push_patches(patches, options.merged)
  File "/usr/lib/python2.4/site-packages/stgit/commands/common.py", line 160,
in push_patches
    forwarded = crt_series.forward_patches(patches)
  File "/usr/lib/python2.4/site-packages/stgit/stack.py", line 789, in
forward_patches
    committer_email = committer_email)
  File "/usr/lib/python2.4/site-packages/stgit/git.py", line 439, in commit
    if message[-1:] != '\n':
TypeError: unsubscriptable object

I've been abusing stg by leaving the commit messages empty so I assume that that's the cause here, or related to the problem.

This happens with today's stg as well as stg-0.10

After scraping all my patches out manually and adding meaningfull (non-empty) commit messages I seem to be able to push and pop them all again.

Cheers,

Auke

#1 lore raw
Quoted 3 lines · Auke Kok

I'm preparing patches for upstream and am haunted by an apparent patch breaking stg. The problem appears to be one or more of my patches breaking a push:

[...]

Quoted 2 lines

I've been abusing stg by leaving the commit messages empty so I assume that that's the cause here, or related to the problem.

Thanks, I fixed StGIT to cope with this and I'll update the repository and snapshot tonight (which, BTW, will contain the initial support for patch history tracking).

Signature · 1 line
Catalin
author
#2 lore raw
Quoted 11 lines · Catalin Marinas
Quoted 3 lines · Auke Kok

I'm preparing patches for upstream and am haunted by an apparent patch breaking stg. The problem appears to be one or more of my patches breaking a push:

[...]

Quoted 2 lines

I've been abusing stg by leaving the commit messages empty so I assume that that's the cause here, or related to the problem.

Thanks, I fixed StGIT to cope with this and I'll update the repository and snapshot tonight (which, BTW, will contain the initial support for patch history tracking).

awesome and awesome, allthough I have no idea what that (patch history tracking) is about :)

Auke

#3 lore raw
Quoted 7 lines · Auke Kok
Quoted 3 lines · Catalin Marinas

Thanks, I fixed StGIT to cope with this and I'll update the repository and snapshot tonight (which, BTW, will contain the initial support for patch history tracking).

awesome and awesome, allthough I have no idea what that (patch history tracking) is about :)

It keeps the history of all the patch modifications (like refresh, push) so that you can get previous versions if something went wrong. It's like having many small commits reflecting the changes to a patch. These commits are totally separate from the main patch commit (so that an StGIT repository is still compatible with GIT) but you can access them with "stg log [--graphical] <patch>". The --graphical option will automatically invoke gitk.

Signature · 1 line
Catalin