homepage

We have duplicate code to write a PEP 376 RECORD file in packaging.util and packaging.command.install_distinfo. The command should use the function from p7g.util. Tests would also be good: Currently, one function uses LF as end of line (change made by Tarek, so authoritative) but the other uses os.linesep.

Here’s the patch produced by Mike and I so far.

Mike, if you don’t have the time to finish this shortly, I will do it, as I need this fixed for another bug. Just let me know if you’d like to do it.

I am unable to work on this issue at this time.
On Thu, Feb 9, 2012 at 8:57 PM, Éric Araujo <report@bugs.python.org> wrote:
>
> Éric Araujo <merwok@netwok.org> added the comment:
>
> Mike, if you don’t have the time to finish this shortly, I will do it, as
> I need this fixed for another bug. Just let me know if you’d like to do it.
>
> ----------
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <http://bugs.python.org/issue13198>
> _______________________________________
>

OK. Thanks for your help!

Patch attached. Adding people from #13175 for review/testing on Windows.

Produces RECORD files with CR-CR-LF line endings (see issue13175). This breaks pysetup remove.
I can't tell if this patch is good otherwise - it's certainly no worse than the behaviour of the unpatched version, but that's not saying much :-)

Damn, bitten by the use of universal newlines! I guess I should split on a literal '\n' instead of calling splitlines. Could you make that change in your clone and tell me if it does the trick?

> Damn, bitten by the use of universal newlines! I guess I should split on a literal '\n' instead of calling splitlines. Could you make that change in your clone and tell me if it does the trick?
Not for a few days, but I'll check when I'm back home

Close as out of date. Packaging has been removed from stdlib.

Please don’t; distutils2 still exists and is mutating into one or more projects where these bug reports will be useful. If it bothers Python core devs to have the bugs in this tracker, they will be migrated, but for the moment please leave them open so they can be found.

Eric, if you want to keep distutils2 issues on the tracker for a while — I'm ok with that.

wheel/distlib/pip don’t have the same code, and if they had the same issue it would probably be found by a user. Closing.