Re: [jamboost] Re: stage test breaking on Windows

From: Vladimir Prus (ghost_at_[hidden])
Date: 2003-04-16 05:13:32

Andre Hentz wrote:
> On Tue, 15 Apr 2003, Andre Hentz wrote:
> > I'll patch bjam and try again. I'll let you know the results.
>
> Well, I would if the patch weren't empty. 

Oh.. your are right :-(

> I assume the patch was going to make file_time look like this:
>
> int
> file_time(
> char *filename,
> time_t *time )
> {
> struct stat statbuf;
>
> if( stat( filename, &statbuf ) < 0 )
> return -1;
>
> *time = statbuf.st_mtime;
> if( *time == 0 ) /* HERE */
> *time = 1; /* HERE */
> return 0;
> }

Yep, that's why I meant.

> With those changes, stage.py fails with:
>
> mkdir: cannot create directory `/cygdrive/c/DOCUME~1': File exists
>
> which I think is a unrelated problem.

Right. I've committed the change to 'file_time'. The problem you see now is
the one we had to stage.py already. Jam has /cygdrive/c/DOCUME~1 target.
It looks at /cygdrive/c directory to see if it exists there, and finds only 
"Documents And Settings". So, it tries to create the directory.

Solutions are
1. If cygwin's gcc has access to winapi headers, extend the workaround in 
pwd.c, which applies only to NT now.
2. If cygwin has a command to convert short name to long one, use it.
3. Ignore this. Instruct to set TEMP for testing and always use long paths.

I'll try to find out the answers to the questions in (1) and (2) myself, but 
if you know, please tell.

> P.S.: BTW, stage.py used to pass for me because I had TEMP=/tmp and the
> default is TEMP=/cygdrive/c/DOCUME~1/LOCALS~1/ANDREH~1.

I see! Dave, I think you can use it to run cygwin tests, as well.

Thanks,
Volodya

>
>
>
>
> To unsubscribe from this group, send an email to:
> jamboost-unsubscribe_at_[hidden]
>
>
>
> Your use of Yahoo! Groups is subject to http://docs.yahoo.com/info/terms/