Skip to content
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

boot error in commit 375fcf #2482

Closed
giuliolunati opened this issue Jun 26, 2016 · 8 comments
Closed

boot error in commit 375fcf #2482

giuliolunati opened this issue Jun 26, 2016 · 8 comments

Comments

@giuliolunati
Copy link

Commit 375fcf introduces this runtime error (natively compiled for Android5):
Rebol Internal Error
If you need to file a bug in the issue tracker, please give thorough
details on how to reproduce the problem:

http://github.com/rebol/rebol-issues/issues

Include the following information in the report:

C Source File ../src/core/b-init.c, Line 113
PANIC metaeducation/ren-c#111
** Boot Error: (string table not decompressed yet)

@hostilefork
Copy link
Member

Hm, I spotted at least one problem, and I don't know how it got past me. So let me know if this is enough to fix it:

metaeducation/ren-c@aba507c

@giuliolunati
Copy link
Author

Same error, in line 113

@hostilefork
Copy link
Member

hostilefork commented Jun 27, 2016

@giuliolunati Hm, well the error says what's wrong... the REBVAL is supposed to be a certain size and it's not. Something is throwing it off that's unique to Android, because the travis builds for other platforms are working:

https://travis-ci.org/metaeducation/ren-c

Could you try that SHOW_SIZEOFS define and see if you can figure out why it's not coming out to the right size?

For this structure:

https://github.com/metaeducation/ren-c/blob/master/src/include/sys-rebval.h#L701

On 32-bit platforms it expects a 32-bit header, a 32-bit "extra", and a 64-bit payload...

@giuliolunati
Copy link
Author

Hi, got this:
8 any_word
8 any_series
8 integer
8 decimal
2 char
8 datatype
8 typeset
8 symbol
8 time
8 tuple
8 function
8 any_context
8 pair
12 event
4 library
8 struct
8 gob
12 money
8 handle
8 all
Rebol Internal Error
If you need to file a bug in the issue tracker, please give thorough
details on how to reproduce the problem:

http://github.com/rebol/rebol-issues/issues

Include the following information in the report:

C Source File ../src/core/b-init.c, Line 113
PANIC metaeducation/ren-c#111
** Boot Error: (string table not decompressed yet)

@hostilefork
Copy link
Member

Ok... well, darn. Why are none of the other builds getting this error? :-/

Thanks, I am on it...!

@giuliolunati
Copy link
Author

Note:
sizeof(void *) == 4
sizeof(REBVAL) == 20

@hostilefork
Copy link
Member

hostilefork commented Jun 27, 2016

Ok, well this involves two of my least favorite types... the money type and the event type...and I'm not overly concerned about them over the long term. But here's an attempt to workaround the issue for now:

metaeducation/ren-c@01699d7

@giuliolunati
Copy link
Author

It works! :-)

@hostilefork hostilefork transferred this issue from metaeducation/ren-c Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants