-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to v24.9.2 #22
Conversation
recipe/.condarc
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should all of our recipes have this? I think I'm missing context on how this deprecation affects our repos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is a conda-standalone specific problem: conda/conda-standalone#99
folder: constructor_src # [win] | ||
|
||
build: | ||
number: 0 | ||
ignore_run_exports: | ||
- '*' | ||
script_env: | ||
- PYINSTALLER_CONDARC_DIR={{ RECIPE_DIR }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- PYINSTALLER_CONDARC_DIR={{ RECIPE_DIR }} | |
- PYINSTALLER_CONDARC_DIR={{ RECIPE_DIR }} |
This logic is a little confusing to me. Is this pointing to the current recipe dir to retrieve the .condarc to pull from defaults or is it intended to point to the pyinstaller recipe dir?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{{ RECIPE_DIR }}
renders to the recipe directory of this feedstock. pyinstaller is not involved with this.
folder: conda_src | ||
patches: | ||
- "{{ SRC_DIR | replace('\\', '/') }}/src/conda_patches/0001-Rename-and-replace-entrypoint-stub-exe.patch" | ||
- "{{ SRC_DIR | replace('\\', '/') }}/src/conda_patches/0002-Manipulate-PATH-directly-instead-of-_call_ing-conda.patch" | ||
- "{{ SRC_DIR | replace('\\', '/') }}/src/conda_patches/0003-Restrict-search-paths.patch" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine, but I am curious why testing is skipped for windows and why the paths do not follow those used for Windows (i.e. '' instead of '/')?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing is performed on Windows.
Using forward slashes in recipe files is the correct format, but on Windows, {{ SRC_DIR }}
uses Windows-style paths.
This reverts commit b08cbc0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
conda-standalone 24.9.2
Destination channel: defaults
Links
Explanation of changes:
.condarc
file to point todefaults
since automatically pointing todefaults
is deprecatedlibmambapy
was pinned to 1.5.8 because 1.5.10 is not available on defaults.