Issue429

Title Can't generate patch for issue 11816
Priority bug Status resolved
Superseder Nosy List loewis, ncoghlan
Assigned To Topics patch_generator

Created on 2011-11-26.08:31:22 by ncoghlan, last changed 2011-12-05.00:07:05 by ncoghlan.

Messages
msg2313 (view) Author: ncoghlan Date: 2011-11-26.08:31:22
Every time I hit "Create Patch" on issue 11816 for my get_opinfo branch, I get the error message: "9512712044a6.diff is already available"

It first started happening when I hit 'Create Patch' after unlinking the previous diff (which used the filename that Roundup is apparently trying to reuse). However, relinking that file didn't make the problem go away.
msg2334 (view) Author: ncoghlan Date: 2011-12-04.06:00:46
OK, the problem really is that it keeps trying to diff against the same changeset, ignoring all subsequent commits to that branch:
https://bitbucket.org/ncoghlan/cpython_sandbox/changesets/9512712044a6
msg2335 (view) Author: loewis Date: 2011-12-04.10:18:34
AFAICT, 9512712044a6 *is* the most recent change on the get_opinfo branch.

Which changeset do you think should it have used instead?
msg2336 (view) Author: ncoghlan Date: 2011-12-04.10:24:15
Any of the changesets that appear above it here: https://bitbucket.org/ncoghlan/cpython_sandbox/changesets/tip/branch%28%22get_opinfo%22%29

There have been at least 4 meaningful updates since then, and 7 more recent merges from default.
msg2337 (view) Author: ncoghlan Date: 2011-12-04.10:26:42
The current latest changeset on that branch is this merge from default: https://bitbucket.org/ncoghlan/cpython_sandbox/changeset/5ce60675e572
msg2339 (view) Author: loewis Date: 2011-12-04.18:32:57
> The current latest changeset on that branch is this merge from default: https://bitbucket.org/ncoghlan/cpython_sandbox/changeset/5ce60675e572

The repository you specified on issue 11816 is
http://hg.python.org/sandbox/ncoghlan, not
https://bitbucket.org/ncoghlan/cpython_sandbox.
In that repository, those changes (including 5ce60675e572) do not
exist.
msg2341 (view) Author: ncoghlan Date: 2011-12-05.00:07:05
Ah, you're right - I added it before I moved my sandbox over to BitBucket, and then my brain was doing the "fill in what you expect to see rather than reading what is actually there" trick.

Sorry for the noise.
History
Date User Action Args
2011-12-05 00:07:05ncoghlansetstatus: chatting -> resolved
messages: + msg2341
2011-12-04 18:32:57loewissetmessages: + msg2339
2011-12-04 10:26:42ncoghlansetmessages: + msg2337
2011-12-04 10:24:15ncoghlansetmessages: + msg2336
2011-12-04 10:18:34loewissetmessages: + msg2335
2011-12-04 06:00:46ncoghlansetstatus: unread -> chatting
messages: + msg2334
2011-11-26 08:31:22ncoghlancreate