darcs

Issue 1063 cygwin pwd confuses darcs shell tests

Title cygwin pwd confuses darcs shell tests
Priority bug Status resolved
Milestone 2.1.x Resolved in
Superseder Nosy List E.Y.Kow, darcs-devel, dmitry.kurochkin, eivuokko, jaredj, kirby, kowey, thorkilnaur, wglozer
Assigned To
Topics Windows

Created on 2008-09-07.09:18:39 by kowey, last changed 2010-06-15.21:48:24 by admin.

Messages
msg5924 (view) Author: kowey Date: 2008-09-07.09:18:34
This isn't a bug in darcs, but in our test suite.  Darcs is a native Windows
program and doesn't support cygwin paths (i.e. those that start with /cygdrive/)

I think MSYS does something else, maybe it just returns Windows friendly paths?
msg5936 (view) Author: droundy Date: 2008-09-08.12:44:13
I'd call this a cygwin bug, not a bug in our test scripts.  There's no reason
that running the test scripts in an unsupported operating system (cygwin) should
work.
msg5946 (view) Author: dagit Date: 2008-09-08.16:18:13
On Mon, Sep 8, 2008 at 5:44 AM, David Roundy <bugs@darcs.net> wrote:
>
> David Roundy <droundy@darcs.net> added the comment:
>
> I'd call this a cygwin bug, not a bug in our test scripts.  There's no reason
> that running the test scripts in an unsupported operating system (cygwin) should
> work.

I really wish we could drop cygwin completely, but it seems that we're
required to use cygwin because of our dependence on autotools.  Are
you saying that windows testers should build with cygwin but test with
cmd.exe?
msg5951 (view) Author: droundy Date: 2008-09-08.17:32:02
On Mon, Sep 08, 2008 at 09:18:04AM -0700, Jason Dagit wrote:
> On Mon, Sep 8, 2008 at 5:44 AM, David Roundy <bugs@darcs.net> wrote:
> >
> > David Roundy <droundy@darcs.net> added the comment:
> >
> > I'd call this a cygwin bug, not a bug in our test scripts.  There's no reason
> > that running the test scripts in an unsupported operating system (cygwin) should
> > work.
> 
> I really wish we could drop cygwin completely, but it seems that we're
> required to use cygwin because of our dependence on autotools.  Are
> you saying that windows testers should build with cygwin but test with
> cmd.exe?

We only depend on bash.  Isn't there a bash available with msys?

David
msg5954 (view) Author: E.Y.Kow Date: 2008-09-08.23:20:19
The following patch updated the status of issue1063 to be resolved:

* Resolve issue1063: Wrap pwd in cygpath in shell tests. 
This is so that we can run the test suite under Cygwin.
Solution by Zooko.
History
Date User Action Args
2008-09-07 09:18:39koweycreate
2008-09-08 12:44:15droundysetstatus: unread -> unknown
nosy: + droundy
messages: + msg5936
2008-09-08 16:18:15dagitsetnosy: droundy, kowey, wglozer, eivuokko, dagit, jaredj
messages: + msg5946
2008-09-08 17:32:04droundysetnosy: droundy, kowey, wglozer, eivuokko, dagit, jaredj
messages: + msg5951
2008-09-08 23:20:21E.Y.Kowsetstatus: unknown -> resolved-in-unstable
nosy: + E.Y.Kow
messages: + msg5954
2009-04-22 03:35:03twbsetstatus: resolved-in-unstable -> resolved
nosy: + dmitry.kurochkin, simon, thorkilnaur
2009-08-06 18:00:12adminsetnosy: + markstos, jast, Serware, darcs-devel, zooko, mornfall, tommy, beschmi, - droundy, wglozer, eivuokko, jaredj, E.Y.Kow
2009-08-06 21:12:26adminsetnosy: - beschmi
2009-08-10 21:48:21adminsetnosy: + wglozer, E.Y.Kow, eivuokko, jaredj, - tommy, markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:45:06adminsetnosy: - dagit
2009-08-25 17:44:37adminsetnosy: + darcs-devel, - simon
2009-08-27 14:19:15adminsetnosy: kowey, wglozer, darcs-devel, eivuokko, thorkilnaur, jaredj, dmitry.kurochkin, E.Y.Kow
2010-06-15 21:48:23adminsetmilestone: 2.1.x
2010-06-15 21:48:24adminsettopic: - Target-2.1
nosy: + kirby