-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add S390X architecture as a Travis job. #5382
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
Conversation
The FFI one has been reported already at: https://bugs.php.net/bug.php?id=79472 Intl failures are due to the fact that the constuctor is failing e.g:
The EXIF ones seems to related to overflow behaviour (and some cast with NaN which seem highly surprising)
I'm not super sure about the MySQL ones as they are maybe caused because I didn't think about how to set it up. |
Regarding EXIF issues, see https://bugs.php.net/79336 |
@Girgias The above error is thrown as it fails to create a new Resource bundle Object. The resource file that the TC refers while calling the constructor is generated specifically for little endian arch. Generate .res files for s390x using following commands:
Will it be possible to add the above changes in travis/compile.sh ? Such that it generates .res files for the respective arch. |
a1475fd
to
639de9f
Compare
33d1299
to
3c4030b
Compare
@Girgias wrt the above EXIF failures, Following observations were made on Big endian System(s390x): |
I don't really know EXIF, maybe @KalleZ has an idea? |
3c4030b
to
4cbfe91
Compare
This gives us a way to compile and test a big endian architecture.
I've fixed the exif issue and applied this change. |
To reduce queue times for Travis, we might want to kill one of the x86_64 jobs, as we don't really need them. |
@nikic Thanks for the fix. |
This gives us a way to compile and test a big endian architecture.