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

Test strange deserialzation bug on 32bit #23330

Closed
wants to merge 1 commit into from
Closed

Conversation

timholy
Copy link
Member

@timholy timholy commented Aug 18, 2017

Since I don't have easy access to a 32bit machine, this seems to be the best way to find out whether the strange error in #23194 (comment) is a consequence of code changes or simply revealing a latent bug. (This passes locally on 64bit.)

@ararslan ararslan added the system:32-bit Affects only 32-bit systems label Aug 18, 2017
@tkelman
Copy link
Contributor

tkelman commented Aug 19, 2017

You don't need a 32 bit machine to test 32 bit Julia, at least on any distribution that has multilib compilers available it's pretty easy to build 32 bit Julia on 64 bit Linux (this is what we do on both Travis and Circle) - set override ARCH = i686 in a Make.user

@timholy
Copy link
Member Author

timholy commented Aug 19, 2017

I found that I got errors when I tried this with an existing directory (even after make cleanall), but starting from a fresh clone seems to be successfully underway (so far).

@tkelman
Copy link
Contributor

tkelman commented Aug 19, 2017

yeah you'll at least need to do a new out of tree build at a minimum, changing arch in place isn't something our makefiles can handle well (distcleanall could work but that's basically equivalent to a new clone)

@timholy
Copy link
Member Author

timholy commented Aug 19, 2017

Very useful tip, thanks. Fixed in #23348 and #23194

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system:32-bit Affects only 32-bit systems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants