Open barracuda156 opened 1 year ago
There are assumptions about sizes within the "partition" code; this would be the cause of the failure. Thanks for picking this one up.
Probably affects Windows i386 (32-bit) too.
@stephematician Thank you! Once there is a fix, I will test it.
@barracuda156 - I've looked through the partition code; it's not clear to me why there's a bad alloc. I don't have a powerpc to debug on so looks like this one won't get fixed by me.
@stephematician Do tests pass fine on 32-bit Linux (i.e., i386
)?
They pass on Windows i386 (32-bit). I have not tested building i386 R on Linux.
Valgrind passes, and as far as i know, the ASAN tests on CRAN pass, too.
If I had a stacktrace that included the library calls then I'd know where to start looking.
@stephematician
If I had a stacktrace that included the library calls then I'd know where to start looking
How do I get it? Run a test binary via GDB?
I think so; that's roughly how I debug on Ubuntu - e.g. I'll run R -d gdb --vanilla
and try the tests (e.g. pkgbuild::compile_dll
/testthat::test_local
or devtools::test
probably will do). Any exception should bring up GDB prompt and you can get the stack info from there.
I'm not the most experienced dev, so if you have any suggestions, let me know!
@stephematician Sorry for a delay, I will return to this shortly.
@barracuda156 - just letting you know I've tested with linux i386 (via Docker, i386/ubuntu image - stock standard build-essential kit) and there were no failures during the test.
@stephematician Thank you! And thanks for reminding of the issue – it got lost in the queue of to-dos :(
I gonna re-run tests and try to get meaningful output.
UPD1. The error did not vanish with gcc13, just for the record. Will update on the issue further.
@stephematician I will test the updated version now.
Hi - thanks, I've moved the project to gitlab: https://gitlab.com/stephematician/literanger - so stay tuned there for updates.
P. S. This is not necessarily a problem with the code, since
malloc
errors show up once in a while when there are duplicatelibstdc++
(one belonging to the OS and another to GCC). However, forR
we have fixed that problem, and I do not see it now when testing packages. So possibly something assumes 64-bitness?