Copy backend does not detect file conflicts
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Steve Kowalik |
Bug Description
The current copy infrastructure checks for version conflicts but does not catch file content clashes.
Since DSCs and DIFFs and native TARBALLs are unique across source versions as binary files, they are indirectly checked by operations on versions.
This problem affects specifically ORIG TARBALLs files, because they are allowed to be different in different archives, but if the source is copied to another PPA a clash may happen and the publisher will refuse to override the previously published in the destination pool/.
We should check for ORIG filename/content clashes and deny the copy if it's the case.
This problem seems to occur more frequently in daily-build/
[1] ppa-script is a tool widely used for daily-builds in PPAs, https:/
Related branches
- Michael Nelson (community): Approve (code)
- Jelmer Vernooij (community): Approve (code*)
-
Diff: 75 lines (+54/-0)2 files modifiedlib/lp/soyuz/scripts/packagecopier.py (+15/-0)
lib/lp/soyuz/scripts/tests/test_copypackage.py (+39/-0)
- Brad Crittenden (community): Approve (code)
-
Diff: 100 lines (+48/-7)2 files modifiedlib/lp/soyuz/scripts/packagecopier.py (+0/-4)
lib/lp/soyuz/scripts/tests/test_copypackage.py (+48/-3)
Changed in soyuz: | |
assignee: | Celso Providelo (cprov) → nobody |
milestone: | pending → 10.02 |
Changed in soyuz: | |
milestone: | 10.02 → 10.03 |
Changed in soyuz: | |
milestone: | 10.03 → pending |
Changed in soyuz: | |
assignee: | nobody → Steve Kowalik (stevenk) |
status: | Triaged → In Progress |
Changed in soyuz: | |
status: | Fix Committed → In Progress |
status: | In Progress → Fix Released |
Note that COMPONENT_ORIG files now need to be checked too -- why not just check all of them to be safe?