Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
* [lttng-dev] [PATCH lttng-ust] Fix: java class check when uudecode is not present
@ 2015-06-30 20:59 Michael Jeanson
  2015-06-30 21:09 ` Mathieu Desnoyers
  0 siblings, 1 reply; 2+ messages in thread
From: Michael Jeanson @ 2015-06-30 20:59 UTC (permalink / raw)


On systems where 'uudecode' is present, the java class check will
run a precompiled class file in the local directory. If the CLASSPATH
variable doesn't contain "." the test will fail regardless of the
presence of the tested for class on the classpath.

This fix makes the behavior of the test consistant whether 'uudecode'
is present or not.

Signed-off-by: Michael Jeanson <mjeanson at efficios.com>
---
 config/ax_check_class.m4 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/ax_check_class.m4 b/config/ax_check_class.m4
index 098aa77..42b51d7 100644
--- a/config/ax_check_class.m4
+++ b/config/ax_check_class.m4
@@ -118,7 +118,7 @@ EOF
                         ac_cv_prog_uudecode_base64=no
                 fi
         rm -f Test.uue
-        if AC_TRY_COMMAND($JAVA $JAVAFLAGS Test $1) >/dev/null 2>&1; then
+        if AC_TRY_COMMAND($JAVA -classpath ".:$CLASSPATH" $JAVAFLAGS Test $1) >/dev/null 2>&1; then
                 eval "ac_cv_class_$ac_var_name=yes"
         else
                 eval "ac_cv_class_$ac_var_name=no"
-- 
1.9.1




^ permalink raw reply	[flat|nested] 2+ messages in thread

* [lttng-dev] [PATCH lttng-ust] Fix: java class check when uudecode is not present
  2015-06-30 20:59 [lttng-dev] [PATCH lttng-ust] Fix: java class check when uudecode is not present Michael Jeanson
@ 2015-06-30 21:09 ` Mathieu Desnoyers
  0 siblings, 0 replies; 2+ messages in thread
From: Mathieu Desnoyers @ 2015-06-30 21:09 UTC (permalink / raw)


merged into stable-2.6 and master, thanks!

Mathieu

----- On Jun 30, 2015, at 4:59 PM, Michael Jeanson mjeanson at efficios.com wrote:

> On systems where 'uudecode' is present, the java class check will
> run a precompiled class file in the local directory. If the CLASSPATH
> variable doesn't contain "." the test will fail regardless of the
> presence of the tested for class on the classpath.
> 
> This fix makes the behavior of the test consistant whether 'uudecode'
> is present or not.
> 
> Signed-off-by: Michael Jeanson <mjeanson at efficios.com>
> ---
> config/ax_check_class.m4 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/config/ax_check_class.m4 b/config/ax_check_class.m4
> index 098aa77..42b51d7 100644
> --- a/config/ax_check_class.m4
> +++ b/config/ax_check_class.m4
> @@ -118,7 +118,7 @@ EOF
>                         ac_cv_prog_uudecode_base64=no
>                 fi
>         rm -f Test.uue
> -        if AC_TRY_COMMAND($JAVA $JAVAFLAGS Test $1) >/dev/null 2>&1; then
> +        if AC_TRY_COMMAND($JAVA -classpath ".:$CLASSPATH" $JAVAFLAGS Test $1)
> >/dev/null 2>&1; then
>                 eval "ac_cv_class_$ac_var_name=yes"
>         else
>                 eval "ac_cv_class_$ac_var_name=no"
> --
> 1.9.1

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-06-30 21:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-30 20:59 [lttng-dev] [PATCH lttng-ust] Fix: java class check when uudecode is not present Michael Jeanson
2015-06-30 21:09 ` Mathieu Desnoyers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox