[
https://issues.apache.org/jira/browse/MESOS-127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13192012#comment-13192012
]
[email protected] commented on MESOS-127:
-----------------------------------------------------
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > configure.ac, line 118
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70253#file70253line118>
bq. >
bq. > Can we change the comment to "linker flags for JNI, when building
Python egg" please?
Done.
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > configure.ac, lines 226-227
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70253#file70253line226>
bq. >
bq. > Is there any clean way to wrap these two so they don't go beyond 80
characters? The rest of the file should meet this constraint (except for maybe
one line), and I'd like to keep it as much as possible.
Refactored (using temp var).
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > configure.ac, line 242
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70253#file70253line242>
bq. >
bq. > Mind fixing this indentation?
Done.
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > configure.ac, lines 276-278
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70253#file70253line276>
bq. >
bq. > So if these three are all AC_ARG_VAR then they don't actually need
AC_SUBST ... not sure why I left the JAVA_LDFLAGS one around.
Removed all.
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > src/Makefile.am, lines 403-404
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70254#file70254line403>
bq. >
bq. > Move these two lines to line 408 (after the
libmesos_no_third_party_la_LIBADD += libjava.la).
Done.
bq. On 2012-01-24 02:46:34, Benjamin Hindman wrote:
bq. > src/Makefile.am, line 493
bq. > <https://reviews.apache.org/r/3580/diff/3/?file=70254#file70254line493>
bq. >
bq. > Wrap this line (under 80 characters) please.
Done.
- Charles
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3580/#review4546
-----------------------------------------------------------
On 2012-01-24 08:33:36, Charles Reiss wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/3580/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-01-24 08:33:36)
bq.
bq.
bq. Review request for mesos.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. On Linux, I've needed to build with -Wl,-rpath to nicely link against
-ljvm. We should do something like this automatically.
bq.
bq. This patch (on Linux only) uses -Wl,-ljvm instead of -ljvm to link -ljvm
so libtool doesn't add it to dependency_libs [*]. Besides hopefully being
unnecessary, it fails to link things linked against that libtool library
because libtool doesn't add the rpath for the java libraries (even if I specify
it with -R). This also requires that the flags be added on libmesos.la and uses
the assumption that libmesos.la is linked dynamically.
bq.
bq. [*] Things added to dependency_libs are automatically linked into any
executables linking against the library; this would be fine (and would make
sense if our header files might contain inline functions that call JNI_* calls)
if the -R settings were properly passed through, but libtool does not appear to
do this.
bq.
bq.
bq. This addresses bug MESOS-127.
bq. https://issues.apache.org/jira/browse/MESOS-127
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. configure.ac bda6769
bq. src/Makefile.am b8fc58e
bq.
bq. Diff: https://reviews.apache.org/r/3580/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Charles
bq.
bq.
> configure JAVA_LDFLAGS detection doesn't set rpath on Linux
> -----------------------------------------------------------
>
> Key: MESOS-127
> URL: https://issues.apache.org/jira/browse/MESOS-127
> Project: Mesos
> Issue Type: Bug
> Components: build
> Environment: Linux (only some JVM versions or distros?)
> Reporter: Charles Reiss
> Assignee: Charles Reiss
> Priority: Minor
>
> On Linux, it is usually necessary to use something like -rpath to link
> against a library like the libjvm.so that is typically off the default
> library search path. Our current build scripts don't set this; if one doesn't
> set LDFLAGS to include -Wl,-rpath or similar, this can result in errors like:
> /scratch/charles/mesos2/build-linux/src/.libs/lt-mesos-tests: error while
> loading shared libraries: libjvm.so: cannot open shared object file: No such
> file or directory
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
