[Mondrian] Updated buildOnJdk.sh

Julian Hyde jhyde at pentaho.com
Thu Mar 8 18:35:45 EST 2012


Thanks for the patch. However, even though that change works, I'd rather not incorporate it. My goal is that buildOnJdk is simple and not frightening to developers who just want to build the code. Anyone who is technical enough to be installing multiple JDKs and doing releases is technical enough to be editing that file. Or creating links in /usr/lib/jvm.

(In retrospect, I should not have added the "Darwin" case either. I build on MacOS, but use some JDKs that are not blessed by apple, so the /usr/lib/jvm approach actually works better.)

When you say "the defaults fail". Hopefully it uses the JAVA_HOME passed in, and successfully builds for that one JDK version. If that's not the case, I'd consider that a bug.


On Mar 8, 2012, at 2:15 PM, Patrick Leckey wrote:


Saw your changes to buildOnJdk.sh and tried them out - works on Mac OS X Lion and Ubuntu 11.10, but not on CentOS 5/6 (with defaults).  Settings JAVA_HOME_16 works, but with that unset the defaults fail on CentOS.

Attached updated buildOnJdk.sh (as text) that checks the distro and architecture and uses those for the CentOS defaults.  Behaviour on other platforms (Mac, Ubuntu) is unchanged.

Also now provides the $distroCommonName (which defaults to "Linux" if none found) and $processorType (which is just uname -p) variables.

Mondrian mailing list
Mondrian at pentaho.org<mailto:Mondrian at pentaho.org>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.pentaho.org/pipermail/mondrian/attachments/20120308/4dfb01c3/attachment-0001.html 

More information about the Mondrian mailing list