Skip to content

Fix ENOENT error during harvesting Conda components#575

Merged
elrayle merged 1 commit intoclearlydefined:masterfrom
qtomlinson:qt/fix_conda_package_map_location
May 8, 2024
Merged

Fix ENOENT error during harvesting Conda components#575
elrayle merged 1 commit intoclearlydefined:masterfrom
qtomlinson:qt/fix_conda_package_map_location

Conversation

@qtomlinson
Copy link
Copy Markdown
Collaborator

@qtomlinson qtomlinson commented May 7, 2024

This is to fix ENONET exception encountered during testing on dev deployment. It follows the same approach taken in debianFetch.js.

@qtomlinson qtomlinson marked this pull request as ready for review May 7, 2024 17:27
@qtomlinson qtomlinson requested a review from lumaxis May 7, 2024 17:27
@qtomlinson qtomlinson requested a review from elrayle May 7, 2024 17:31
@qtomlinson qtomlinson marked this pull request as draft May 7, 2024 20:19
@qtomlinson qtomlinson marked this pull request as ready for review May 7, 2024 21:42
Copy link
Copy Markdown
Contributor

@lumaxis lumaxis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I wonder if we can do anything about either the env setup or the code to make this mechanism more resilient in the future so other contributors don't run into this discrepancy as easily 🤔

@elrayle
Copy link
Copy Markdown
Collaborator

elrayle commented May 8, 2024

@lumaxis That refactor sounds like a great idea!

@elrayle
Copy link
Copy Markdown
Collaborator

elrayle commented May 8, 2024

Thanks @qtomlinson for making this happen.

@elrayle elrayle merged commit cdd518c into clearlydefined:master May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test Conda components harvest

3 participants