This repository has been archived by the owner on Feb 18, 2023. It is now read-only.
workflows: add run-on-arch action #134
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Looks like the IPv6 part fails in |
|
the best thing would be to have a generic API to check if a certain AF is supported, If you have one or more IPv6 address, then net_af_supported(AF_INET6) would return true. |
|
should we merge this to main ? The advantage is that we get extra coverage and testing for ARM64 platforms, Another alternative is to run this in a special branch, or with a crontab (e.g. once a week). |
|
Looks good to me, maybe we can extend this with matrix, so we have at least one 32 bit platform test? |
|
thanks, I added the script with the following combinations:
S390x is Big Endian 64-bits If the script becomes very slow or annoying, please feel free to disable it. |
alfredh
added a commit
that referenced
this pull request
Nov 1, 2022
* async: add new tests (#107) * cmake: add path to windows build * mock: fix warning * dns/client: add async getaddrinfo usage (#112) * dns: refactor IP checks and add system test * use dnsc_getaddrinfo * fmt: workaround for windows compiler * sys: fix unlink and WIN32 * test: cast values to fix WIN32 warnings * trace: fix unlink and WIN32 * main: only run regular tests if GETOPT is missing * test: various fixes for WIN32 * async: fix multithreading tests (#115) * main: if GETOPT is missing run regular tests * cmake: add LINKLIBS variable * test cmake on Windows (#114) * test cmake on Windows * You have an error in your yaml syntax on line 14 * fix yaml syntax error * try to find re * try to find rem * mock: fix win32 warning * fix unlink warnings * fix warnings * fix write * fix size_t * write needs unsigned int * link to win32 libs * add qwave * execute retest.exe in build/debug * set verbose * set verbose * disable all * update flags * ci: fix compiler list and choco * LINKLIBS and flags * test: fix c11 err handling * cmake: use re config (#118) * base64: Encoding/Decoding with URL and Filename Safe Alphabet (#113) * cmake: bump minimum to 3.10 (#119) * Don't fail on compilation when there no OpenSSL on target platform (#120) * trice: fix win32 with explicit error tests (#121) * http: test http/https requests with large body * http: validate body of http requests for get method * http: add test for http conn requests * dns: test dnsc_getaddrinfo_enabled * rtp: Update tests for the changed rtp_send signature. This updates RTP tests for compatibility with this PR: baresip/re#418 * crc32: add re wrapper * sipreg: use TEST_ERR and remove static port test Tests should run without interfere with each other. Looks like on Github Actions runners share the same port range sometimes? So for tests we have to use dynamic ports. * cmake: no need to link to libz * sa: init err * cmake: add optional zlib linking * cmake/FindREM: add rem-static target name * sys: remove old sys div test (#130) * aubuf: refactor aubuf_auframe test (#70) * rtcp: use udp_send() (#132) * rtcp: use udp_send() * rtcp: add workaround for OOM test * leb128: remove debug * http: fix http content length format (#136) * Support run specific integration test and add http with local dnssrv (#135) * support run specific integration test * http: add integration test for http using local dns server * odict: check return value of compare() * odict: use correct union field (boolean) (#137) Fixes a bug on Alpine/S390x which is 64-bit Big Endian architecture * test: use fs_stdio_restore() (#138) * oom: align testcase names * align testcase names * tmr: increase upper bound * remove leb128 printf * sys: add todo * net: add ipv6_supported() * sys: unique filename in test_sys_fs_fopen (#142) * net: remove logging * Enable valgrind for Linux build (#140) * integration: make IPv6 tests optional * bfcp: check for handler errors after re_main * fmt: add test_fmt_str_itoa (#143) * git: ignore cmake generated files * workflows: add run-on-arch action (#134) * test_multithread: increase timeout to 15 seconds * tls: remove some warnings (#144) * ci/run-on-arch: add prepare (needed for dependency) (#145) * ci: use actions/checkout@v3 (#148) * list: add test_list_flush (#151) * list: fix flushl global (#152) Co-authored-by: Sebastian Reimers <[email protected]> Co-authored-by: Dmitry Ilyin <[email protected]> Co-authored-by: Franz Auernigg <[email protected]> Co-authored-by: Andrey Semashev <[email protected]> Co-authored-by: Franz <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.