-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
#12103 - CI for FreeBSD #12179
#12103 - CI for FreeBSD #12179
Conversation
This PR is complete, the build fails due to other FreeBSD errors that I've opened issues for. |
This seems to be hanging at I've tested it locally on FreeBSD 12 RELEASE and it runs through all the tests without a problem, so I'm not sure what's going on. |
How can we see what is failing? For you openBSD PR there are errors printed at the end of the log, but I don't see the same here. |
@narimiran What's failing here is a timeout (see my previous comment), which I can't replicate on VMs or physical hardware. I'll push an empty commit to this PR later to try trigger another test run and see if it still times out. |
Even better: you can rebase it onto the latest devel and solve the existing merge conflict :) |
@narimiran Sure, that should be easy enough. Will do so after work. |
5e96ca4
to
7c3fe16
Compare
Looks like it's still stalling at the same point - it reaches I have no idea why, or what it's actually meant to be doing at that point. |
Could you disable that test so we can see if something else fails too? The top of discard """
disabled: true
output: "Done"
cmd: "nim $target --debuginfo --hints:on --define:useNimRtl $options $file"
""" |
Yep, will give it a go.
…On Tue, 12 Nov 2019, at 06:32, Miran wrote:
Could you disable that test so we can see if something else fails too?
The top of `tests/dll/client.nim` should read:
discard """
disabled: true
output: "Done"
cmd: "nim $target --debuginfo --hints:on --define:useNimRtl $options $file"
"""
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#12179?email_source=notifications&email_token=AAFW24JWNQB2DYUBR7TPA6DQTJEXNA5CNFSM4IVW7IXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDZF7SA#issuecomment-552755144>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFW24J4AJLFKNZ4ATDLNXTQTJEXNANCNFSM4IVW7IXA>.
|
So I disabled
It now seems to be stuck again, so whichever test is running after this one seems to be the problem. |
Oh wow, the CI actually passed! 😮 |
Build failures seem unrelated as far as I can see - the current |
…r byte, so minus 1 for result length
- use GNU make to build csources - set PATH variable using the environment key - remove modification of config to set CC as this is already set
e4fbb16
to
844c960
Compare
* Ref nim-lang#12103 - adds FreeBSD CI * Fix getApplFreebsd - length of the string includes the null terminator byte, so minus 1 for result length * Show last commit in setup task. * Remove .git from repository URL * Don't include noisy details showing last commit. * Add FreeBSD build status badge * Fix nim-lang#12182 - disable tconsole on FreeBSD * Disable tgetaddrinfo on FreebSD as getaddrinfo doesn't support the ICMP protocol. * Install boehm-gc-threaded * Use libgc-threaded.so on FreeBSD rather than libgc.so. * Simplify build failure handling. Update alt text for CI badge. * Disable test on FreeBSD * Simplify build config - use GNU make to build csources - set PATH variable using the environment key - remove modification of config to set CC as this is already set * Install git which seems to be missing from current freebsd images * Revert change to how path is set * Add a comment explaining why the length is truncated * Fix tconsole.
Fixes #12113
Fixes #12103
Linked to #12105 - this adds FreeBSD CI support where the other adds OpenBSD.
There are a few tests currently failing on FreeBSD. I've opened some issues for some of them and will open more after the current test run completes.