Unvanquished / validate-release

MIT License
1 stars 1 forks source link

deduplicate expected binary lists the hardcoded way #7

Closed illwieckz closed 1 year ago

illwieckz commented 1 year ago

This is an alternate way to deduplicate things (see #6) that is still done the hardcoded way, one can for example do this without adding or editing any logic to the code:

diff --git a/validate_release.py b/validate_release.py
index 0d73688..42bfbef 100755
--- a/validate_release.py
+++ b/validate_release.py
@@ -232,6 +232,7 @@ def CheckSymbols(z, symids):
         ('Linux', 'amd64', linux_binaries),
         ('Linux', 'armhf', linux_binaries),
         ('Linux', 'arm64', linux_binaries),
+        ('FreeBSD', 'riscv64', ['daemonded']),
         ('windows', 'i686', windows_binaries),
         ('windows', 'amd64', windows_binaries),
         ('NaCl', 'armhf', game_binaries),
illwieckz commented 1 year ago

I noticed we already had a ENGINE_BINARIES list so I reused it. I added a list comprehension for the windows binary list.

I don't mind this one being “computed”, the purpose of the PR is that the proposed design allows anyone to hardcode the list in an obvious and easy way, something the proposed code doesn't prevent. The code also not assumes all platforms have the same amount of things done the same way (unlike my original proposition in #6).

slipher commented 1 year ago

LGTM