Commit
31db96c312b953afc26ae4b16b266e367a0fdcef
by Oliver Smith
lint/lint_diff: run clang-format if config was found
The osmo-asf4-dfu and osmo-ccid-firmware now have clang-format configs. Adjust the lint script to run clang-format if such a config is found in the repository for which the lint script is running.
Commit
3dffb34587dc0bba9ce46c7ef0d7e07c8167a8d4
by Oliver Smith
lint/lint_diff: allow skipping with OSMO_LINT=0
When this script is set as pre-commit hook, it can be useful to skip the linter with this env var to make a commit that intentionally does not pass the linter (e.g. to test if it will be caught in gerrit + jenkins).
There is also "git commit --no-verify", but with that the hook that applies the Change-Id does not run.
Commit
9a18ebaa78fc255a0aea42186a2743829d69162e
by Oliver Smith
obs: gerrit_binpkgs: set --allow-unknown-package
The gerrit_binpkgs.sh script can be used to quickly test building of debian packages in a container even for packages that we don't have in the OBS osmocom repositories. For example I'm currently using it to test-build kamailio packages. Use the --allow-unknown-package argument so build_srcpkg.py doesn't complain if this gets called with a package that isn't a known Osmocom package.
Commit
e28454ea4b120a7ab96e133589a7b7b3ee8c0846
by Oliver Smith
*.py: ignore: E722 Do not use bare `except`
Let the ruff linter not complain about the existing cases of "Do not use bare `except`" so we get the linter to pass without changing the existing logic here. With this we don't disable the check globally, so we can consider making stricter excepts in new code.
Commit
c8ec5d58507718165b863c36b51e29e9afd1a4a1
by Oliver Smith
scripts/verify_log_statements: fix undefined name
Append it to "errors_found" instead of undefined variable "ignored". This makes it work similarly to further above:
# Skip 'LOG("bla" fmt )' strings that typically appear as #defines. if fmt_re.match(quoted): if debug: errors_found.append(error_found(f, log.start(), 'Skipping define', log.group(0))) continue
Fix the following bug:
scripts/verify_log_statements.py:92:11: F821 Undefined name `ignored` | 90 | # hopefully eval broke because of some '## args' macro def 91 | if debug: 92 | ignored.append(error_found(f, log.start(), 'Ignoring', log.group(0))) | ^^^^^^^ F821 93 | continue |
Commit
b97a04f431503a5aa0a0f07a68910153c13721be
by Oliver Smith
jobs/osmocom-build-tags…: set seccomp profile
Set the seccomp profile to make io_uring work in docker, so the regression test in libosmocore using io_uring does not fail. This happened to work previously on the build4 jenkins node because of an older docker version.