Check file conflicts in x86_64 build job #36876
Conversation
common/scripts/lint-conflicts
Outdated
| rv=0 | ||
|
|
||
| template_exists() { | ||
| [ -e "srcpkgs/$1/template" ] |
There was a problem hiding this comment.
shouldn't this be -f (exists and is file)?
common/scripts/lint-conflicts
Outdated
| } | ||
|
|
||
| conflicts_of() { | ||
| # print list of packages that are _marked_ as confliting with given one |
There was a problem hiding this comment.
nit/typo: confliting -> conflicting
common/scripts/lint-conflicts
Outdated
| } | ||
|
|
||
| list_newly_built_files() { | ||
| # print one line per file in newly build packages |
common/scripts/lint-conflicts
Outdated
| # create associative array `owners` mapping file to list of packages | ||
| # for all packages in xlocate | ||
| local pkgname file pkgver | ||
| ## newly build packages |
common/travis/fetch-xtools.sh
Outdated
| @@ -4,7 +4,7 @@ | |||
|
|
|||
| TAR=tar | |||
| command -v bsdtar >/dev/null && TAR=bsdtar | |||
| URL="https://github.com/leahneukirchen/xtools/archive/master.tar.gz" | |||
| URL="https://github.com/Chocimier/xtools/archive/master.tar.gz" | |||
There was a problem hiding this comment.
this can be switched back now, right?
|
Changed usage so it takes list of packages from local repository index and not global file. Can be easily used locally without arguments. Test runs at https://github.com/Chocimier/void-packages-org/actions/runs/2272244355, https://github.com/Chocimier/void-packages-org/actions/runs/2272208791, https://github.com/Chocimier/void-packages-org/actions/runs/2272194034 |
|
those test runs look good to me, maybe annotations could be added à la #36950 (if they make sense to)? |
As an annotation without the |
|
Without |
It gets a special mention in the action run summary page, and arguably the special highlighting in the logs is worth it. The message in the logs could otherwise be printed in red with ANSI sequences since GH actions render those |
Report packages installing same file and not marked with conflicts or replaces.