Éric Araujo <mer...@netwok.org> added the comment:

I did some more work on the patch and committed.  Thanks again to both of you.

I also thought again about this remark:

> Changeset r83996 was introduced to prevent sdist from overwriting a
> project maintained manifest by testing for a comment at the head of
> the manifest file. It's not clear to me this was necessary because the
> write_manifest() should never have been called if the template was
> absent but a manifest existed.

I also wonder if that was necessary; I did not fully understand the bug at the 
time.  Unfortunately, now that we have a handful of releases out there with 
this change, we have to support the magic comment.  For distutils2 however 
(a.k.a. “packaging” module in 3.3), I’ll open another bug report and if the 
same bugs apply (MANIFEST handling is a bit different there), I’ll try to fix 
the bug without using a magic comment.

----------
components:  -Distutils2
resolution:  -> fixed
stage: commit review -> committed/rejected
status: open -> closed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11104>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to