Repository navigation
Commit 2494088
committed
Derive var_testurl from the testing issue
Setting var_testurl by hand means remembering to, for every script. This reads
it off the issue instead, when the issue is labelled Ready For Testing, and
inserts it into the ct/ script that issue belongs to.
Matching is APP= against the issue title with everything but letters and
digits removed. An exact comparison misses real pairs -- 'Rocky Linux' vs
'RockyLinux', 'llama-cpp' vs 'llama.cpp', 'Anki-Sync-Server' vs 'Anki Sync
Server' were four of them. Normalising found 69 of 77 with no collision in
either direction; the remaining 8 are addon and VM scripts, which create no
container and so have neither a MOTD nor a description to carry the request.
A title matching no script, or more than one, is reported and skipped rather
than guessed at.
The line is inserted after the last var_* declaration, commented ones
included, so a conventionally commented-out var_arm64 does not end up below it.
Scripts that already carry the variable are left alone, so the bulk run is
safe to repeat.
The issue title is passed through the environment rather than a ${{ }}
interpolation: it is attacker-controlled text and would otherwise be pasted
into the shell verbatim. Verified with titles containing quotes and command
substitution.1 parent 61855d8 commit 2494088
2 files changed
Lines changed: 167 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments