You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-dev@xmlgraphics.apache.org by bu...@apache.org on 2009/02/26 13:48:23 UTC

DO NOT REPLY [Bug 46771] New: [PATCH] fop fails if JAVACMD_OPTS is set

https://issues.apache.org/bugzilla/show_bug.cgi?id=46771

           Summary: [PATCH] fop fails if JAVACMD_OPTS is set
           Product: Fop
           Version: 0.95
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: fop-dev@xmlgraphics.apache.org
        ReportedBy: mvyskocil@suse.cz


Created an attachment (id=23313)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23313)
Allow to set a JAVACMD_OPTS on jpackage.org compatible systems

Moved from https://bugzilla.novell.com/show_bug.cgi?id=467866

The jpackage-utils contatenace the JAVACMD_OPTS with JAVACMD, so on rpm
distributions with something in JAVACMD_OPTS in /etc/java/java.conf the
following test in /usr/share/fop/fop fails:

if [ ! -x "$JAVACMD" ] ; then
  echo "Error: JAVA_HOME is not defined correctly."
  echo "  We cannot execute $JAVACMD"
  exit 1
fi

Attached patch splits JAVACMD variable in executable test and remove the \"\"
from exec.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46771] [PATCH] fop fails if JAVACMD_OPTS is set

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46771

--- Comment #2 from Glenn Adams <gl...@skynav.com> 2012-04-07 01:43:06 UTC ---
resetting P2 open bugs to P3 pending further review

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46771] [PATCH] fop fails if JAVACMD_OPTS is set

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46771

Glenn Adams <gl...@skynav.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P2                          |P3

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46771] [PATCH] fop fails if JAVACMD_OPTS is set

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46771





--- Comment #1 from Vincent Hennebert <vh...@gmail.com>  2009-02-27 03:43:58 PST ---
Hi,

It looks like it is a jpackage.org-specific problem. Basically the
java-functions script referred to in the upstream bug should probably not
concatenate $JAVACMD_OPTS to JAVACMD. As its name indicates JAVACMD refers to
the executable, not to possible command arguments.

The fop script is based on the script shipped with ant, so that problem that's
occurring with fop will most probably occur with ant as well. The removal of
quotes in fop_exec_command worries me: there's certainly a good reason why they
were added in the first place, probably to handle paths with spaces in them.
I'm not keen to modify the script without a good reason, as it's designed to
work on all types of Unix platforms, even those with ancient tools and shells.
If we change it then it needs to be re-tested on all those platforms, which we
don't have access to.

I'm happy to hear comments or opinions about this, but I would close the bug as
invalid.

Thanks,
Vincent

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46771] [PATCH] fop fails if JAVACMD_OPTS is set

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46771

Glenn Adams <ga...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2

--- Comment #3 from Glenn Adams <ga...@apache.org> 2012-04-11 03:20:15 UTC ---
increase priority for bugs with a patch

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.