Changes by Berker Peksag : ---------- components: +Distutils nosy: +berker.peksag, dstufft, eric.araujo stage: -> patch review type: -> behavior versions: +Python 3.5, Python 3.6 _______________________________________ Python tracker http://bugs.python.org/issue25592 _______________________________________

Change by Jeroen Demeyer : Removed file: https://bugs.python.org/file40993/data_files_doc.patch _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Jeroen Demeyer added the comment:

If you’re not sure about the reason for that sentence, I think you should not remove it from the docs

If the docs are wrong, their history doesn't matter that much: the docs should be fixed regardless.

test the conditions (package with extension module)

I did that before posting this bug report. I also looked at the distutils sources. I couldn't find any evidence that data_files are ever installed in sys.exec_prefix. ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Jeroen Demeyer added the comment: There is also commit fa2f4b6d8e297eda09d8ee52dc4a3600b7d458e7 Author: Greg Ward Date: Sat Jun 24 17:22:39 2000 +0000 Changed the default installation directory for data files (used by the "install_data" command to the installation base, which is usually just sys.prefix. (Any setup scripts out there that specify data files will have to set the installation directory, relative to the base, explicitly.) with commit message stating that data_files are installed in sys.prefix. ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Jeroen Demeyer added the comment: Well, I did try it on a minimal Python project. I also read the distutils sources and understood why it installs data_files in sys.prefix by default. So what more do you need to be convinced? ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Jeroen Demeyer added the comment:

Did you try with a minimal project containing a C extension? Did you install in a system where sys.prefix != sys.exec_prefix?

Yes to both questions. ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Antoine Pitrou added the comment: Though it's difficult to say for certain (distutils' option selection logic is obscure and difficult to follow), it seems that Jeroen's analysis is right. ---------- nosy: +pitrou _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Antoine Pitrou added the comment: New changeset 598e15d4feaee3849a91d92c9ca51f17baafe19c by Antoine Pitrou (jdemeyer) in branch 'master': bpo-25592: Improve documentation of distutils data_files (GH-9767) https://github.com/python/cpython/commit/598e15d4feaee3849a91d92c9ca51f17baa... ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Change by miss-islington : ---------- pull_requests: +11551, 11552 _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Éric Araujo added the comment: Thanks for the analysis and PR! Sorry that the slow process here was frustrating. ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Antoine Pitrou added the comment: Fixing this on 2.7 would require additional investigation (distutils might have diverged), so I'm closing now. ---------- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: -Python 2.7 _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Change by Antoine Pitrou : ---------- status: closed -> open _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Jeroen Demeyer added the comment:

Could you still give it a quick check?

I did just that. For reference, these are the steps: - Checkout the "2.7" branch of the cpython git repo - ./configure --prefix=/tmp/prefix --exec-prefix=/tmp/eprefix && make && make install - Install pip (https://pip.pypa.io/en/stable/installing/#installing-with-get-pip-py) - /tmp/prefix/pip install --no-deps --verbose pari_jupyter # This requires the PARI/GP library to be installed - Note that all Jupyter stuff is installed in /tmp/prefix/share/jupyter while the directory /tmp/eprefix/share does not exist ---------- _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Change by miss-islington : ---------- pull_requests: +11633 stage: resolved -> patch review _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Change by miss-islington : ---------- pull_requests: +11633, 11634, 11635 stage: resolved -> patch review _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________

Change by Antoine Pitrou : ---------- stage: patch review -> resolved status: open -> closed versions: +Python 2.7 _______________________________________ Python tracker https://bugs.python.org/issue25592 _______________________________________