-
Notifications
You must be signed in to change notification settings - Fork 890
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
tests: run-params issue #1668
Comments
@wythe ^^ I just noticed you had a recent commit to |
run-params doesn't test anything vital to the current build, so you can safely remove it during testing for now. I'm about to run out the door. I can take a closer look at it later. We do a lot of neat redirection of SIGABRT in that test which may be the issue on arch. |
Are you still seeing this when you configure with |
@wythe so far I am running my tests with https://gist.github.com/jsarenik/6923be59beb524b75a154fd11b47263a#file-full-log-txt-L40 |
Just run |
Will do, with current master (i.e. 14c6310). |
#In the meantime I am widening the scope to also UPDATE: See #1693
|
It does not happen on Ubuntu (glibc). Travis would notice... |
And here is strace from Debian aarch64:
|
And yet another from Alpine running in chroot on that Debian aarch64 box:
|
Fixes ElementsProject#1668 Reported-by: @jsarenik Signed-off-by: Mark Beckwith <wythe@intrig.com>
@wythe thanks for wythe@49e3a45 on https://github.com/wythe/lightning/tree/fix-param-arg-size. It fixes the original issue (ending with SIGABRT). Now, there is still an outstanding issue with Segmentation fault on musl (both aarm64 and x86_64, I have not tested more). I will do a separate issue for this, with easy steps on how to reproduce on any x86_64 Linux by setting up a small Alpine Linux chroot. UPDATE: see #1693 |
Reopening because of #1684 (comment) (as far as I understood it's failing on 32-bit architecture). UPDATE: No, it's not failing, see #1684 (comment) |
Issue and Steps to Reproduce
Testing on Debian buster/sid on aarch64 (DragonBoard 410c with latest build 359)
UPDATE: Also Alpine Linux on
x86_64
see later commentI can give root access to this box, just send me your public SSH key via email.
And the strace of
run-params
is here:See full log
The text was updated successfully, but these errors were encountered: