commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LANG-1145) 64 bit check in SystemUtils
Date Tue, 21 Feb 2017 00:26:44 GMT

    [ https://issues.apache.org/jira/browse/LANG-1145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15875169#comment-15875169
] 

ASF GitHub Bot commented on LANG-1145:
--------------------------------------

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.


> 64 bit check in SystemUtils
> ---------------------------
>
>                 Key: LANG-1145
>                 URL: https://issues.apache.org/jira/browse/LANG-1145
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.*
>            Reporter: Gabor Liptak
>            Priority: Minor
>
> Add is64bit method to SystemUtils.java
> http://stackoverflow.com/a/2269242/304690 for Windows snippet and
> https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/SizeEstimator.scala
for zOS snippet



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message