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 |
|
@hmaarrfk, @jakirkham, @jezdez, @kkraus14, @loriab, @mariusvniekerk, @minrk, @synapticarbors: final reminder, vote ends in ~3.5h! |
Vote closedThe vote did not pass.
Summary
Binding votes (21)
|
|
Vote bot doesn't know our bespoke voting rules. There are 24 current core members, and 21 total people voted. Of those who did not abstain, the vote was 17 in favor, 1 against. This vote has passed the 60% threshold. Congrats @conda-forge/core for minting a new CFEP! 🎉 |
|
Thanks for checking that @beckermr ! I'm glad we reached quorum and that the vote passed!! |
Unfortunately for windows, no. Symlinks on windows are for admin users only and conda does not know how to package them AFAIR.
If the code is open source, I'll be happy to have a look and make some recommendations. |
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)