Closed daviddavis closed 5 months ago
@daviddavis Can you add a regression test for this? Something like:
@quba42 just any binary package or do you mean a source package?
@quba42 just any binary package or do you mean a source package?
I meant source package of course!
I found for example this one in our current test data set to work with: https://github.com/ATIX-AG/pulp_deb/tree/main/pulp_deb/tests/functional/data/debian/pool/asgard/m/mimir
I reviewed the changes and the test. The test will fail as expected when the changes are not applied. Everything looks good to me. If we can change the test to run in parallel I'm happy to merge it.
❌ Failed to cleanly apply 9e2a1967ace4bb4f07ffb99226b24480b86bd7dc on top of patchback/backports/3.1/9e2a1967ace4bb4f07ffb99226b24480b86bd7dc/pr-1054
origin
remote.upstream
. If you don't
have it, here's how you can add it:
$ git remote add upstream https://github.com/pulp/pulp_deb.git
$ git fetch upstream
$ git checkout -b patchback/backports/3.1/9e2a1967ace4bb4f07ffb99226b24480b86bd7dc/pr-1054 upstream/3.1
$ git cherry-pick -x 9e2a1967ace4bb4f07ffb99226b24480b86bd7dc
If it'll yell at you with something like fatal: Commit 9e2a1967ace4bb4f07ffb99226b24480b86bd7dc is a merge but no -m option was given.
, add -m 1
as follows instead:
$ git cherry-pick -m1 -x 9e2a1967ace4bb4f07ffb99226b24480b86bd7dc
$ git push origin patchback/backports/3.1/9e2a1967ace4bb4f07ffb99226b24480b86bd7dc/pr-1054
🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.
✅ Backport PR branch: patchback/backports/3.2/9e2a1967ace4bb4f07ffb99226b24480b86bd7dc/pr-1054
Backported as https://github.com/pulp/pulp_deb/pull/1067
🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.
Handle a source package publish IntegrityError when a source package belongs to multiple dists.
fixes #1053