Issue Details (XML | Word | Printable)

Key: FOR-905
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Ross Gardler
Reporter: Mark Eggers
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Forrest

Recent change in cygwin appears to prevent the classpath for loop in the forrest command from working

Created: 06/Jul/06 10:36 PM   Updated: 07/Jan/08 11:48 AM
Return to search
Component/s: Core operations
Affects Version/s: 0.8
Fix Version/s: 0.9-dev

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works forrest-sh.patch 2006-07-06 10:47 PM Mark Eggers 0.6 kB

Urgency: Normal
Other Info: Patch available


 Description  « Hide
I recently upgraded Cygwin on a Windows/2000
Professional machine to:

bash-3.1$ uname -a
CYGWIN_NT-5.0 phoenix 1.5.20(0.156/4/2) 2006-07-01
02:22 i686 Cygwin

bash-3.1$ cygpath --version
cygpath (cygwin) 1.42
Path Conversion Utility
Copyright 1998, 1999, 2000, 2001, 2002, 2003, 2004,
2005 Red Hat, Inc.
Compiled on Jul 1 2006

bash-3.1$ bash --version
GNU bash, version 3.1.17(6)-release (i686-pc-cygwin)
Copyright (C) 2005 Free Software Foundation, Inc.

Forrest stopped working and gave the following error.

bash-3.1$ pwd
/cygdrive/c/src/forrest/site-author
bash-3.1$ echo $FORREST_HOME
C:\src\forrest
bash-3.1$ forrest
Apache Forrest. Run 'forrest -projecthelp' to list
options

Buildfile: C:\src\forrest\main\forrest.build.xml

BUILD FAILED
Target
`c:/src/forrest/lib/endorsed/jakarta-regexp-1.4.jar'
does not exist in this project.

Total time: 1 second

After investigating, it seems that the loop populating
the CLASSPATH was not being executed.

It appears that the bash shell no longer understands

C:\src\forrest/lib/endorsed/*.jar

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Mark Eggers added a comment - 06/Jul/06 10:47 PM
Patch to handle apparent change in Cygwin's behavior

Mark Eggers made changes - 06/Jul/06 10:47 PM
Field Original Value New Value
Attachment forrest-sh.patch [ 12336446 ]
Gavin added a comment - 28/Feb/07 09:48 PM
Someone who uses Forrest with Cygwin should try this.

In addition, may not be relevent but I got this in an email :-

<quote>...but as I understand it, Cygwin recently changed the default for how it handles line endings. This caused a lot of scripts that I had which contained DOS line endings to break (even blank lines caused "syntax error:
unexpected end of file"), as well as other problems that didn't seem to go away even after converting my line endings to unix style. I got the problem to disappear (without ever really understanding it) by adding a system variable called "SHELLOPTS" with the value "igncr"...</quote>

Quote was by Steve Strobel.

Gav...

Doron Cohen added a comment - 28/Dec/07 10:11 PM
Thanks for this patch - just wanted to let you know that it was helpful!

After upgrading from Forrest 0.7 to 0.8, running Forrest for building Lucene Java docs failed with:
   Target `../apache-forrest-0.8/lib/endorsed/jakarta-regexp-1.4.jar' does not exist in this project.

Applying this patch solved the problem.

Thanks.
Doron

Repository Revision Date User Message
ASF #609578 Mon Jan 07 11:47:13 UTC 2008 rgardler Make things work in Cygwin (FOR-905, due to Mark Eggers)
Files Changed
MODIFY /forrest/trunk/bin/forrest

Ross Gardler added a comment - 07/Jan/08 11:48 AM
Applied with thanks

Ross Gardler made changes - 07/Jan/08 11:48 AM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Fix Version/s 0.9-dev [ 12310041 ]
Assignee Ross Gardler [ rgardler ]