-
Notifications
You must be signed in to change notification settings - Fork 253
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rewrite tests #515
rewrite tests #515
Conversation
williamh
commented
Apr 10, 2022
- rewrite tests to work with meson
- add tests to ci
This will fix #513 once it is merged. |
I need some folks from Debian/Ubuntu/Alpine to take a look and tell me what I'm missing. |
Not a user of any of those, however I may have some suggestions. It's a weird error message, but that being said -- this script uses /bin/bash instead of /bin/sh and Alpine prefers not to have bash in the base setup, while the BSDs typically put bash in /usr/local/bin due to hysterical raisins. |
Hi @eli-schwartz , That was a typo actually, all of the test scripts are meant to be sh scripts. I made that change, and Alpine is the only one that is failing now. Can you take a look? Thanks much, William |
It appears this test already failed at 0efc1b1 in Alpine. --- a/test/check-trailing-newlines.sh
+++ b/test/check-trailing-newlines.sh
@@ -6,7 +6,7 @@ top_srcdir=${SOURCE_ROOT:-..}
ebegin "Checking trailing newlines in code"
out=$(cd ${top_srcdir};
for f in `find */ -name '*.[ch]'` ; do
- sed -n -e :a -e '/^\n*$/{$q1;N;ba' -e '}' $f || echo $f
+ awk '/^$/ {ret=1; next} 1 {ret=0} END {exit ret}' $f || echo $f
done)
[ -z "${out}" ]
eend $? "Trailing newlines need to be deleted:"$'\n'"${out}"
(EDIT:) Or alternatively using |
This ports our tests to meson and makes them able to be run in parallel.
This test was using a GNU sed command which does not work on Alpine Linux.
99e9177
to
1ae6ec8
Compare