On 09/09/2013 01:40 PM, Omair Majid wrote: > On 09/05/2013 05:10 AM, Magnus Ihse Bursie wrote: >> On 2013-09-04 21:26, Omair Majid wrote: >>> Over in Fedora-land, we are trying to build OpenJDK (using zero) on >>> s390(x) [1]. One of the changes needed is updating the platform.m4 file >>> to teach it about s390(x). The following webrev does that: >>> http://cr.openjdk.java.net/~omajid/webrevs/s390-platform/00/ >> >> My only concern here is about >> >> VAR_CPU_ARCH=s390x >> >> While I'm not 100% familiar about the specifics of the s390 family, nor >> the s390 port, the general idea behind the VAR_CPU_ARCH variable, as >> opposed to the VAR_CPU variable, is to unify 32 and 64 bit versions of >> the same CPU. > > Thanks for spotting this. Fixed: > http://cr.openjdk.java.net/~omajid/webrevs/s390-platform/01/
If there are no further concerns, shall I go ahead and push? Thanks, Omair -- PGP Key: 66484681 (http://pgp.mit.edu/) Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681