[Patch] improvement for forrest script under cygwin

[Antoine Levy-Lambert <le...@tiscali-dsl.de>]

The environment variable TERM is not always set to "cygwin" under cygwin.

There are actually reasons to set it to other values, such as "linux" or
"ansi"
litterature about TERM under cygwin :
http://www.cygwin.com/ml/cygwin/2001-11/msg01417.html


|---------------------------------------------------------------------------
-------------------|
|So, forrest should use the same test to tell that one is under cygwin as
ant  :              |
|case "`uname`" in
|
|  CYGWIN*) cygwin=true ;;
|
|---------------------------------------------------------------------------
-------------------|

This works better and should not harm anybody working under cygwin or under
UNIX.

Yours

Antoine Lévy-Lambert

Re: [Patch] improvement for forrest script under cygwin

[Antoine Levy-Lambert <le...@tiscali-dsl.de>]

Steven Noels wrote :
>
> The validate-xdocs target however still fails when using cygwin in a
> dir-with-spaces:
>

Stevenn, I have tried something similar :

/cygdrive/c/Dokumente und Einstellungen/Administrator/Eigene
Dateien/antoine/testforrest>
$ forrest validate
Apache Forrest.  Run 'forrest -projecthelp' to list options

Buildfile: c:\gnu\forrest\forrest.build.xml

init-props:
Loading project specific properties from c:\Dokumente und
Einstellungen\Administrator\Eigene
Dateien\antoine\testforrest/forrest.properties

echo-settings:

examine-proj:

validation-props:

validate-xdocs:
6 file(s) have been successfully validated.
...validated xdocs

validate-skinconf:
...validated skinconf

validate-sitemap:

validate-skins-stylesheets:

validate-skins:

validate-skinchoice:
...validated existence of skin 'forrest-site'

validate-stylesheets:

validate:

BUILD SUCCESSFUL
Total time: 3 seconds
/cygdrive/c/Dokumente und Einstellungen/Administrator/Eigene
Dateien/antoine/testforrest

It works for me with a similar directory structure containing spaces. Maybe
my project (something I copied from the ant site proposal) is too simplistic
for the error to show up. Maybe you should try to do a "forrest -verbose
validate" to see where it is really failing. Also, did you check that the
"forrest" script which you are actually running is the actual last version ?
Yours
Antoine

Re: [Patch] improvement for forrest script under cygwin

[Steven Noels <st...@outerthought.org>]

Antoine Levy-Lambert wrote:

> |---------------------------------------------------------------------------
> -------------------|
> |So, forrest should use the same test to tell that one is under cygwin as
> ant  :              |
> |case "`uname`" in
> |
> |  CYGWIN*) cygwin=true ;;
> |
> |---------------------------------------------------------------------------
> -------------------|
> 
> This works better and should not harm anybody working under cygwin or under
> UNIX.

Applied, thank you!!

The validate-xdocs target however still fails when using cygwin in a 
dir-with-spaces:

$ forrest validate
Apache Forrest.  Run 'forrest -projecthelp' to list options

Buildfile: c:\Documents and Settings\stevenn\My 
Documents\Outerthought\Projects\xml-forrest\build\di
st\shbat\forrest.build.xml

init-props:
Loading project specific properties from c:\Documents and 
Settings\stevenn\My Documents\Outerthought
\Projects\outerthought.org/forrest.properties

echo-settings:

examine-proj:

validation-props:

validate-xdocs:
dropping c:\Documents from path as it doesn't exist
dropping and from path as it doesn't exist
dropping Settings\stevenn\My from path as it doesn't exist
dropping 
Documents\Outerthought\Projects\xml-forrest\build\dist\shbat\lib\endorsed\*.jar 
from path as it doesn't exist

</Steven>
-- 
Steven Noels                            http://outerthought.org/
Outerthought - Open Source, Java & XML Competence Support Center
Read my weblog at            http://blogs.cocoondev.org/stevenn/
stevenn at outerthought.org                stevenn at apache.org