-
Notifications
You must be signed in to change notification settings - Fork 167
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
use channel_remap for envs.txt too #656
Conversation
We require contributors to sign our Contributor License Agreement and we don't have one on file for @EisW. In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature. We will ping the bot to refresh the PR status when we have confirmed your signature. |
conda is required to hav an activate script
The added example, (with test instructions as comment in it) generates "myinstaller-0.0.1.sh" Without the provided fix the installer fails with assertion:
With the fix it rans perfectly and installation is usable (tested under Windows and Linux) |
remove trailing whitespace
# prepare private channel first: | ||
# (1) create folder for channel: mkdir -p /tmp/mychannel/linux-64 | ||
# (2) copy a private package into channel: | ||
# (2.1) simulate with package from conda-forge: wget https://anaconda.org/conda-forge/bitvector/3.5.0/download/noarch/bitvector-3.5.0-pyhd8ed1ab_0.conda | ||
# (2.2) cp bitvector-3.5.0-pyhd8ed1ab_0.conda /tmp/mychannel/linux-64/ | ||
# (2.3) Create index (required for usage): conda index /tmp/mychannel | ||
# (3) Now create installer: constructor examples/use_channel_remap/ |
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.
Can we provide this as a little script under scripts/
so we test this in CI? 🙏
> Can we provide this as a little script under scripts/ so we test this in CI?
I will try it, makes sense
Gesendet: Montag, 13. März 2023 um 12:45 Uhr
Von: "jaimergp" ***@***.***>
An: "conda/constructor" ***@***.***>
Cc: "Tino Winkler" ***@***.***>, "Mention" ***@***.***>
Betreff: Re: [conda/constructor] use channel_remap for envs.txt too (PR #656)
@jaimergp commented on this pull request.
In examples/use_channel_remap/construct.yaml:
+# prepare private channel first:
+# (1) create folder for channel: mkdir -p /tmp/mychannel/linux-64
+# (2) copy a private package into channel:
+# (2.1) simulate with package from conda-forge: wget https://anaconda.org/conda-forge/bitvector/3.5.0/download/noarch/bitvector-3.5.0-pyhd8ed1ab_0.conda
+# (2.2) cp bitvector-3.5.0-pyhd8ed1ab_0.conda /tmp/mychannel/linux-64/
+# (2.3) Create index (required for usage): conda index /tmp/mychannel
+# (3) Now create installer: constructor examples/use_channel_remap/
Can we provide this as a little script under scripts/ so we test this in CI? 🙏
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Co-authored-by: jaimergp <jaimergp@users.noreply.github.com>
Interesting: The created installer was valid though, because URL is not used during install, only for verify cache, which works fine
CLA Check failed again: must I sign again? Or is this just a barrier to prevent automatic merge? Btw: with a little hack the channel remap is now tested within CI. |
This was merged earlier today in conda/infrastructure#729 |
@conda-bot check |
Sorry, do not understand the failed test: may some temporary condition on CI? |
Let's re-run it. Maybe there's a repodata issue in defaults for macos, but that'd be weird. We could use conda-forge for the example and see if that provides a passing test. |
What confuses me: it must be related to remap, because the very same construct.yaml, only without remap from examples/miniconda did work only seconds before.
But why only for Python=3.9, this is strange. Unfortunately I have no OSX available.
Suggestion:
(1) rerun the test without modifications (just in case)
(1a) if (1) fails again, I would like to see the complete contents of local var repodatas preconda.py:write_index_cache() at the given line
(by adding a "print(repodatas)" and rerun (means chckin the print and rerun I think)
(2) your suggestion: modify source channel to conda-forge instead main
Message ID: ***@***.***>
|
Well it worked :) |
Description
Fix #654
Created installer with private channels does not find the package(s) from private channel during install.
This occured on Windows and Linux, but should be alike for OSX
Checklist - did you ...
news
directory (using the template) for the next release's release notes?