Return-Path: 
 <dev-return-56834-apmail-ant-dev-archive=ant.apache.org@ant.apache.org>
Delivered-To: apmail-ant-dev-archive@www.apache.org
Received: (qmail 53968 invoked from network); 5 Apr 2004 16:50:29 -0000
Received: from daedalus.apache.org (HELO mail.apache.org) (208.185.179.12)
  by minotaur-2.apache.org with SMTP; 5 Apr 2004 16:50:29 -0000
Received: (qmail 10349 invoked by uid 500); 5 Apr 2004 16:50:20 -0000
Delivered-To: apmail-ant-dev-archive@ant.apache.org
Received: (qmail 10316 invoked by uid 500); 5 Apr 2004 16:50:19 -0000
Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:dev-unsubscribe@ant.apache.org>
List-Subscribe: <mailto:dev-subscribe@ant.apache.org>
List-Help: <mailto:dev-help@ant.apache.org>
List-Post: <mailto:dev@ant.apache.org>
List-Id: "Ant Developers List" <dev.ant.apache.org>
Reply-To: "Ant Developers List" <dev@ant.apache.org>
Delivered-To: mailing list dev@ant.apache.org
Received: (qmail 10303 invoked from network); 5 Apr 2004 16:50:19 -0000
Received: from unknown (HELO exchange.sun.com) (192.18.33.10)
  by daedalus.apache.org with SMTP; 5 Apr 2004 16:50:19 -0000
Received: (qmail 9529 invoked by uid 50); 5 Apr 2004 16:51:03 -0000
Date: 5 Apr 2004 16:51:03 -0000
Message-ID: <20040405165103.9528.qmail@nagoya.betaversion.org>
From: bugzilla@apache.org
To: dev@ant.apache.org
Cc: 
Subject: DO NOT REPLY [Bug 28096]  -
    jjtree outputdirectory now required
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=28096>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28096

jjtree outputdirectory now required





------- Additional Comments From javadotant@shadowland.demon.nl  2004-04-05 16:51 -------
Are you referring to this dicussion ?

Stefan,

See javadoc for method validateOutputFile:
    * When running JJTree from an Ant taskdesk the -OUTPUT_DIRECTORY must
    * always be set. But when -OUTPUT_DIRECTORY is set, -OUTPUT_FILE is
    * handled as if relative of this -OUTPUT_DIRECTORY. Thus when the
    * -OUTPUT_FILE is absolute or contains a drive letter we have a problem.

Thus when the outputdirectory is supplied the outputfile is no longer treated as
an absolute path but relative to that outputdirectory, and therefore starting
the outputfile with an / or \ is kind of ignored.
At the moment I'm working on a patch that transforms the absolute outputfile to
an relative outputfile for the default outputdirectory (project basedirectory)
when the outputdirectory is not supplied.
Leaving a drive letter as the only problem. But the because when JavaCC is run
from the commandline with an outputdirectory and and outputfile (containing a
drive letter) this will result in a file of the form <outputdirectory>/<drive
letter>: which is empty. Thus always blocking a drive letter in the outputfile
is justified imho.

I will attach the patch in bugzilla today or tomorrow,

         Jene.

Stefan Bodewig wrote:

>
> ------------------------------------------------------------------------
>
> Subject:
> DO NOT REPLY [Bug 21526] - Enhancement JavaCC, JJTree and JJDoc [update]
> From:
> bugzilla@apache.org
> Date:
> 29 Jul 2003 14:48:05 -0000
> To:
> dev@ant.apache.org
>
>
> DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH
THE WEB INTERFACE AVAILABLE AT
> <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21526>.
> ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG
DATABASE.
>
> http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21526
>
> Enhancement JavaCC, JJTree and JJDoc [update]
>
>
>
>
>
> ------- Additional Comments From bodewig@apache.org  2003-07-29 14:48 -------
> Some questions
>
> +        if ((outputDirectory == null)
> +            && (outputFile.startsWith("/") || outputFile.startsWith("\\")))
> +        {
> +            throw new BuildException("Absolute path in 'outputfile' not
> supported: " + outputFile);
> +        }
>
> _*only if outputdirectory has not been set?  I.e. absolute paths are supported
> if it has been set?
> *_
> +            throw new BuildException("Drive letter in 'outputfile' not
> supported: " + outputFile);
>
> Is this a general limitation of jjtree?
>
>  
>
>> Maybe this table could be attached to jjtree.html ?
>>   
>
>
> Of course it can, if you send a patch
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org