CFEP: Unixy layout for python on windows#65
Conversation
| We are intentionally not changing the layout here because there's no way to | ||
| tell conda to install things here for `noarch: python` packages. | ||
| I propose adding a `python_scripts_path` as a CEP and use that in a future | ||
| release, but is out of scope for this CFEP. |
There was a problem hiding this comment.
I wish conda clients would query sysconfig for this instead of hardcoding it, but I don't think there's a static JSON like in PEP 739, is there?
There was a problem hiding this comment.
conda clients shouldn't run the package IMO. That's why python_site_packages_path was added for purelib/platlib.
There was a problem hiding this comment.
Yes, I agree no code should be run. I just wish we had a static file shipped with the package, perhaps the output of python -m sysconfig or something. I'm mostly thinking out loud here, not requesting anything
Co-authored-by: jaimergp <jaimergp@users.noreply.github.com>
pb01ka
left a comment
There was a problem hiding this comment.
I have left some questions and plausible corrects that can be made. Please feel free to correct me if you feel I mis-interpreted anything.
|
@conda-forge/core @conda-forge/emeritus-core This PR falls under the CFEP Approval policy, please vote and/or comment on this PR. This vote will end on in two weeks on June 17, 2026. /vote |
Vote created@isuruf has called for a vote on The members of the following teams have binding votes:
Non-binding votes are also appreciated as a sign of support! How to voteYou can cast your vote by reacting to
Please note that voting for multiple options is not allowed and those votes won't be counted. The vote will be open for |
|
Glad to see this change proposal, thank you! Just throwing this out here (but explicitly not meant as an objection to the change which I very much welcome): |
Most tools use sysconfig to get these values. For eg: CMake and setuptools |
|
@baszalmstra, @chenghlee, @dopplershift, @hmaarrfk, @jakirkham, @jezdez, @kkraus14, @loriab, @mariusvniekerk, @minrk, @soapy1, @SylvainCorlay, @synapticarbors, @wolfv: gentle reminder to vote! |
I've seen a fair amount of C++ code internally that has these paths semi-hardcoded as well. I don't think this is something we can easily patch / PR our way out of. This is effectively a breaking change and we'd need to treat it as such. Is there any way for us to use symlinks or some kind of links to transition this change more smoothly? |
|
How is the C++ code using these paths? Would need to know more to understand how to fix it.
Yes, that's why I'm suggesting using python 3.15 for this change and not porting it to <3.15 for now. |
|
Really happy about this. I hope that we will eventually get rid of the LIBRARY_PREFIX / CONDA_PREFIX distinction eventually, this is a great move in this direction. |
I've seen both build code and runtime code that specifically checks for the |
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)