Bug#489722: git patches to migrate to PEP 8 conformant names

Ben Finney ben+debian at benfinney.id.au
Thu Sep 11 14:19:45 UTC 2008


On 11-Sep-2008, Stefano Zacchiroli wrote:
> Thanks a lot: I've investigated with the other maintainers and no 
> voices against your request were raised, we will be happy to 
> integrate your patch (and thanks for the deprecation module, which 
> will probably be useful also for other stuff in the future).

Great news, thank you.

> Still, your patch at the moment is a bit messy and suboptimal to review.
> Take for example the patch about the example
> examples/debtags/tagsByRelevance; your patch first remove all the code
> and then read it, probably using the right calls.

As far as I can tell, that's an artefact of the way Git handles file 
metadata: it can't tell the difference between "file X was renamed to 
file Y and had some changes" and "file X lost all 100 lines and file Y 
gained all 100 very similar lines".

I may be wrong on this (I'm not an expert at using Git) but Git, as 
far as I can tell, is just representing the change as best it can.

You'll notice that Git has produced more readable output for the other 
method-renaming patch (the one that changes many more files).

> Can you please:
> - provide 2 patches instead of 3: one for the deprecation module, the
>   other (instead of 2) for all the method renamings.

In fact the patch to 'examples/debtags/tagsbyrelevance' is only 
separate because I missed it on the first pass. Combining the two 
patches won't help the readability, as it will still contain a whole 
lot of removed lines in one file and a whole lot of added lines in the 
other.

I was a little dismayed to see that the project decided to move away 
from Bazaar, which handles this case more elegantly :-/

> - avoid stuff like the example above and keep your patches minimal.
> 
> Once that is ready, I'll be more than happy to review, apply, and
> upload.

I hope you can reconsider based on the above; I don't see how to 
improve my patches while still using Git.

-- 
 \                              “Holy rising hemlines, Batman!” —Robin |
  `\                                                                   |
_o__)                                                                  |
Ben Finney <ben at benfinney.id.au>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-python-debian-maint/attachments/20080912/64f068e5/attachment.pgp 


More information about the pkg-python-debian-maint mailing list