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 squashfs packaging when using --dest-prefix #210

Merged
merged 5 commits into from
Aug 10, 2023
Merged

Conversation

rafiyr
Copy link
Contributor

@rafiyr rafiyr commented Feb 10, 2022

Bracket just the unpack script generation with "has_dest" in Packer.finish instead of returning.

In the current state it doesn't actually pack anything into the squashfs images when used with --dest-prefix.

@anaconda-issue-bot
Copy link

We require contributors to sign our Contributor License Agreement, and we don't have one on file for @rafiyr. 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.

@dbast
Copy link
Member

dbast commented Feb 15, 2022

@anaconda-issue-bot check

@anaconda-issue-bot anaconda-issue-bot added the cla-signed [bot] added once the contributor has signed the CLA label Feb 15, 2022
@mcg1969 mcg1969 requested a review from a team as a code owner September 2, 2022 20:16
Copy link
Contributor

@mcg1969 mcg1969 left a comment

Choose a reason for hiding this comment

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

LGTM!

@rafiyr rafiyr requested a review from xhochy as a code owner April 27, 2023 22:26
@rafiyr
Copy link
Contributor Author

rafiyr commented May 12, 2023

@xhochy please take a look at this and #267

Bracket just the unpack generation with has_dest in Packer.finish
instead of returning.
@xhochy xhochy merged commit 6d5eec3 into conda:main Aug 10, 2023
36 checks passed
@jaimergp
Copy link
Collaborator

This could have used a news file given that the PR description is not too detailed. Alternatively we can edit the PR title now so GH picks it up in the new release. Thanks! 🙏

@xhochy xhochy changed the title Fix dest + squashfs Fix squashfs packaging when using --dest-prefix Aug 10, 2023
@xhochy
Copy link
Collaborator

xhochy commented Aug 10, 2023

Updated the title. I would suspect that we will also be using the PR titles for the changelog of the next release.

@github-actions github-actions bot added the locked [bot] locked due to inactivity label Aug 10, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 2024
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.

6 participants