[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-07-19 Thread Elvis Stansvik
Elvis Stansvik added the comment: So feel free to close this issue. The use case was quite marginal anyway, and we can always instruct developers to run `build_ext --inplace` when they intend to use the distribution out of the source directory (and not have inplace=1 in setup.cfg, to allow

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-07-19 Thread Elvis Stansvik
Elvis Stansvik added the comment: Okay, so it seems my "fix" is not complete at all. Not even `setup.py install` works with plain distutils, since it just copies the build dir. Back when I created this issue, I must have (accidentally) tested `setup.py install` using

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-07-19 Thread Elvis Stansvik
Elvis Stansvik added the comment: @inada.naoki: I was half afraid that note in the docs would be brought up :) But I should have brought it up myself, sorry about that. My take is that I see no reason why this limitation should exists. For projects that have extensions, and wish

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-07-19 Thread Elvis Stansvik
Changes by Elvis Stansvik <elvst...@gmail.com>: -- pull_requests: +2818 ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue29225> ___

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-02-15 Thread Elvis Stansvik
Elvis Stansvik added the comment: It's been over a month. It would be great if someone could at least acknowledge the bug. The current behavior is clearly wrong. I realize changes to distutils is perhaps not the most fun thing to review, but I'd like some kind of fix for this to go in. I have

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-01-10 Thread Elvis Stansvik
Elvis Stansvik added the comment: Not really related to this fix, but a colleague also prudently pointed out that perhaps an assert file[:prefix_len] == build_dir + os.sep before the outputs.append(os.path.join(self.install_dir, file[prefix_len:])) in the helper for pure modules

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-01-10 Thread Elvis Stansvik
Changes by Elvis Stansvik <elvst...@gmail.com>: -- versions: +Python 3.7 ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue29225> ___

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-01-10 Thread Elvis Stansvik
Changes by Elvis Stansvik <elvst...@gmail.com>: -- keywords: +patch Added file: http://bugs.python.org/file46241/fix-install-lib-with-inplace-ext.patch ___ Python tracker <rep...@bugs.python.org> <http://bugs.python

[issue29225] distutils.command.install_lib.get_outputs() wrong with extensions built inplace

2017-01-10 Thread Elvis Stansvik
New submission from Elvis Stansvik: I noticed the following strange behavior: estan@newton:~/testdist$ find . ./testext.c ./setup.cfg ./setup.py estan@newton:~/testdist$ cat setup.py from distutils.core import setup, Extension setup( name='testdist', version='1.0.0', ext_modules