Hi Justin,

Thank your for voting and advices, We will try to improve the README.md about 
compiling on MacOSX and Yes it is not optional by default, I guess the reason 
why cannot include module mnemonic-pmalloc-service that probably stems from 
compiling or installation of pmalloc project, Thanks.

As you mentioned, the artifact "org.beanshell:bsh" is licensed in LGPL that 
should be introduced by testng and only be used by test process. The mnemonic 
does not depend on "bsh" elsewhere, I appreciate your reminding and letting me 
beware of that. 

Best Regards
+Gary

-----Original Message-----
From: Justin Mclean [mailto:jus...@classsoftware.com] 
Sent: Friday, July 29, 2016 6:48 PM
To: general@incubator.apache.org
Subject: Re: [VOTE] Release of Apache Mnemonic-0.2.0-incubating [rc3]

Hi,

+1 (binding)

I checked:
- release contains incubating
- signature and hash good
- LICENSE and NOTICE good
- No binaries in source
- All files have ASF headers
- Could compile from source but some minor issues (on OSX)

I think the README could do with some clearer information on how to compile on 
OSX. It seem the optional parts are not optional by default?

I manage to get it to compile by skipping the optional bits:
mvn -pl '!mnemonic-memory-services/mnemonic-nvml-vmem-service' -pl 
'!mnemonic-memory-services/mnemonic-pmalloc-service' clean package install

You may want to double check your dependancies as I’m seeing a LGPL dependancy 
but it seems to be for testing?

[INFO] org.apache.mnemonic:mnemonic-utilities-service:jar:0.2.0-incubating
[INFO] +- org.testng:testng:jar:6.8.17:test [INFO] |  +- 
org.beanshell:bsh:jar:2.0b4:test [INFO] |  \- com.beust:jcommander:jar:1.27:test
[INFO] +- org.apache.mnemonic:mnemonic-core:jar:0.2.0-incubating:compile
[INFO] |  +- org.flowcomputing.commons:commons-resgc:jar:0.8.9:compile
[INFO] |  \- com.squareup:javapoet:jar:1.6.1:compile
[INFO] +- org.apache.mnemonic:mnemonic-collections:jar:0.2.0-incubating:compile
[INFO] \- org.flowcomputing.commons:commons-primitives:jar:0.6.0:compile


Thanks,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to