Github user kinow commented on the issue:
https://github.com/apache/commons-lang/pull/231
@michael-o
>What is the real pupose for this actually? The client should not care
about the arch at all.
I think @Tomschi use case is valid, where a client could need to know the
arch before loading a certain library, and we had another issue submitted
LANG-1145 with similar requirement.
>The regex match is brittle. This will likely fail on ARM and it fails here
on Itanium with HP-UX for os.arch IA64N which is a 32 bit VM.
Note taken, perhaps before merging we can try to cover more archs, like
this list:
* http://lopica.sourceforge.net/os.html
There is another place where arch is used within Commons too:
*
https://github.com/apache/commons-crypto/blob/158be0644c353a617427ab190a4f09082cda42ac/src/main/java/org/apache/commons/crypto/OsInfo.java#L28
We can possibly look at how crypto is using it, and if we could maybe use a
similar approach here.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
