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

deb: Fix patches when patch is in subdir #543

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

cpuguy83
Copy link
Member

@cpuguy83 cpuguy83 commented Feb 25, 2025

When a patch is in a subdir, the wrong path was used to extract the
patch from the source from.

Additionally, the call to mountSources was causing issues afer this
fix.
mountSources is just in this case is an optimization so to prevent
dpkg-source from having to re-extract data we already have available.
Since this is causing an issue I decided to just remove it for now.

I believe the two changes included in this PR are playing off each other (ie, one change requires a change in the other).
We'll need to look into this before including this optimization again.

When a patch is in a subdir, the wrong path was used to extract the
patch from the source from.

Additionally, the call to `mountSources` was causing issues afer this
fix.
`mountSources` is just in this case is an optimization so to prevent
dpkg-source from having to re-extract data we already have available.
Since this is causing an issue I decided to just remove it for now.

Signed-off-by: Brian Goff <[email protected]>
@cpuguy83 cpuguy83 changed the title WIP: Fix jammy patching Fix jammy patching Feb 27, 2025
@cpuguy83 cpuguy83 changed the title Fix jammy patching deb: Fix patches when patch is in subdir Feb 27, 2025
@cpuguy83 cpuguy83 marked this pull request as ready for review February 27, 2025 00:22
@cpuguy83 cpuguy83 requested a review from a team as a code owner February 27, 2025 00:22
@cpuguy83 cpuguy83 merged commit 4f3e336 into Azure:main Feb 28, 2025
18 checks passed
@cpuguy83 cpuguy83 deleted the fix_deb_patches branch February 28, 2025 19:19
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.

2 participants