| msg119550 - (view) |
Author: Atsushi Odagiri (Atsushi.Odagiri) |
Date: 2010-10-25 13:55 |
| I wrote setup.py includes scripts with distutils2, and ran `setup.py install`. scripts were installed in scripts directory, but not recorded in .dist-info/RECORD. I think that `distutils2._backport.shutil:copytree` should return copied filenames. |
|
|
| msg119845 - (view) |
Author: Éric Araujo (eric.araujo) *  |
Date: 2010-10-29 02:53 |
| Thanks for the report. This is indeed a bug with respect to PEP 376. It would be too difficult to modify shutil (and copy2) to make it return filenames, but thanks to the new copy_function hook it will be easy to store filenames in a list and use that. I’ll have to synchronize _backport.shutil with the 3.2 version first. |
|
|
| msg124092 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-12-15 23:09 |
| For the record, I have started work on this, then stopped due to lack of time. I’ll get back to it. |
|
|
| msg137975 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2011-06-09 14:42 |
| I wrote a test but it did not fail. Can you test it too? |
|
|
| msg138047 - (view) |
Author: Atsushi Odagiri (Atsushi.Odagiri) |
Date: 2011-06-10 04:44 |
| OK, I'll test it too. I looked sources of packaging. That use packaging.utils.copy_tree. When I reported that problem, install_scripts used _backport.shutil.copytree. Maybe problem is fixed, because it reterned copied file lists. Is that fixies applied to distutils2? |
|
|
| msg138085 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2011-06-10 15:43 |
| > OK, I'll test it too. Thanks. In case you’re not sure how to test my patch, see http://docs.python.org/devguide/faq#how-do-i-apply-a-patch and http://docs.python.org/devguide/runtests > I looked sources of packaging. That use packaging.utils.copy_tree. > When I reported that problem, install_scripts used backport.shutil.copytree. Ah, I see a comment in the code that copy_tree should be replaced by shutil.copytree, so we need to get the test anyway to make sure changing the code later will not cause a regression. > Maybe problem is fixed, because it reterned copied file lists. Indeed. > Is that fixies applied to distutils2? distutils2 will be recreated as a backport of packaging. IOW, the packaging module in the 3.3 stdlib is the distutils2 project. |
|
|
| msg143407 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2011-09-02 16:05 |
| I have added tests for scripts in RECORD and they pass. Closing, please reopen if you can reproduce the bug. |
|
|