bileto packaging diffs include anything that has 'debian' as a subpath
Bug #1651790 reported by
Steve Langasek
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bileto |
Fix Released
|
Undecided
|
Robert Bruce Park | ||
Bug Description
Reviewing a packaging diff here:
I saw that the diff included paths such as oxide-qt-
This is clearly not part of the packaging, but appears to have been included in the packaging diff because there is *a* /debian/ directory as part of the path.
The debian/ filtering should be anchored to only the first-level debian/ subdir. If using filterdiff, this would be '*/debian/**' vs. '**/debian/**'.
Revision history for this message
| Robert Bruce Park (robru) wrote | #1 |
Revision history for this message
| Robert Bruce Park (robru) wrote | #2 |
| Changed in bileto: | |
| status: | New → Fix Released |
| assignee: | nobody → Robert Bruce Park (robru) |
To post a comment you must log in.
We are using filterdiff but the current filter is '*debian/*' which suggests that filterdiff does not support ** vs * path-depth-aware globbing. I also can't find reference to any '**' in the manpage (though the manpage just says "--include=PATTERN" and never defines PATTERN).
I'll try your suggestion of '*/debian/**' and see what happens but I don't have much hope...