Skip to content
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

Fix constructor failures when the repo/subchannel only has noarch packages #512

Merged
merged 1 commit into from
Jul 28, 2022
Merged

Fix constructor failures when the repo/subchannel only has noarch packages #512

merged 1 commit into from
Jul 28, 2022

Conversation

bryan-hunt
Copy link
Contributor

When the full_repodata is set to None it causes write_repodata to throw an AttributeError exception because it immediately tries to treat it as an initialized dictionary.

Instead of returning none this change returns an initialized repodata dictionary with the minimum expected data which allows constructor to continue properly.

@anaconda-issue-bot
Copy link

We require contributors to sign our Contributor License Agreement, and we don't have one on file for @bryan-hunt. In order for us to review and merge your code, please e-sign the PDF at https://conda.io/en/latest/contributing.html#conda-contributor-license-agreement. We then need to manually verify your signature. We will ping the bot to refresh the PR status when we have confirmed your signature.

@bryan-hunt
Copy link
Contributor Author

CLA has been signed

@jaimergp jaimergp closed this Jul 27, 2022
@jaimergp jaimergp reopened this Jul 27, 2022
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jul 27, 2022
@jaimergp
Copy link
Contributor

Just ran into this issue too, applied your patch and it worked! I am re-running the CI to make sure everything is alright. Merging later if green!

* Setting full_repodata to None causes write_repodata to throw an AttributeError exception
* Initializing to empty repodata minimizes the changes required throughout constructor.
@jaimergp jaimergp merged commit 01d021b into conda:main Jul 28, 2022
@github-actions github-actions bot added the locked [bot] locked due to inactivity label Jul 29, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed [bot] added once the contributor has signed the CLA locked [bot] locked due to inactivity
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants