[GitHub] matthiasblaesing commented on issue #828: [NETBEANS-1074] Module Review libs.glassfish_logging

[GitBox <gi...@apache.org>]

matthiasblaesing commented on issue #828: [NETBEANS-1074] Module Review libs.glassfish_logging
URL: https://github.com/apache/incubator-netbeans/pull/828#issuecomment-417913099
 
 
   The ALv2 license looks correct. If you look at the package structure, this is a shaded version of commons logging. I checked [glassfish 2.1.1](http://www.oracle.com/technetwork/java/javaee/downloads/java-archive-downloads-glassfish-419424.html#sges-2.1.1-oth-JPR)  and indeed, it contains the jar, but that jar correctly holds a LICENSE ant NOTICE file.
   
   That file can also be found on maven central:
   https://search.maven.org/artifact/com.sun.commons/logging-api/1.0.4/jar
   
   From an API perspective they are identical to the `glassfish-logging-2.0.jar`. I suspect, that the version number in the jar refers to the glassfish version that it was contained in.
   
   I suggest we switch to the maven central version, fix the version number and are done. I don't think this artifact is used often, as glassfish is AFAIK currently using java.util.logging.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists