Thread overview
[dmd-internals] Failed dmd tests
Jan 30, 2011
Michel Fortin
Jan 30, 2011
Jason House
Jan 30, 2011
Michel Fortin
Jan 30, 2011
Michel Fortin
Jan 30, 2011
Brad Roberts
January 30, 2011
Running the test suite in the DMD repository, there's 3 tests that always fails for me on Mac OS X, all related to coverage analysis. I'm not too sure what to do with them.

The tests are:

	runnable/a20.d
	runnable/cov2.d
	runnable/sieve.d

The problem comes from this in runnable/extra-files/a20-postscript.sh:

	# trim off the last line which contains the path of the file which differs between windows and non-windows
	head -n-1 ${RESULTS_DIR}/runnable/runnable-a20.lst > ${RESULTS_DIR}/runnable/runnable-a20.lst2

It fails with this:

	head: illegal line count -- -1

Looks like 'head' does not support negative numbers on Mac OS X, which is why those tests fail. I'd rather fix them than having to disable them. Any idea?


-- 
Michel Fortin
michel.fortin at michelf.com
http://michelf.com/



January 30, 2011
Try "tail -n 1"

Sent from my iPhone

On Jan 30, 2011, at 10:13 AM, Michel Fortin <michel.fortin at michelf.com> wrote:

> Running the test suite in the DMD repository, there's 3 tests that always fails for me on Mac OS X, all related to coverage analysis. I'm not too sure what to do with them.
> 
> The tests are:
> 
>    runnable/a20.d
>    runnable/cov2.d
>    runnable/sieve.d
> 
> The problem comes from this in runnable/extra-files/a20-postscript.sh:
> 
>    # trim off the last line which contains the path of the file which differs between windows and non-windows
>    head -n-1 ${RESULTS_DIR}/runnable/runnable-a20.lst > ${RESULTS_DIR}/runnable/runnable-a20.lst2
> 
> It fails with this:
> 
>    head: illegal line count -- -1
> 
> Looks like 'head' does not support negative numbers on Mac OS X, which is why those tests fail. I'd rather fix them than having to disable them. Any idea?
> 
> 
> -- 
> Michel Fortin
> michel.fortin at michelf.com
> http://michelf.com/
> 
> 
> 
> _______________________________________________
> dmd-internals mailing list
> dmd-internals at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/dmd-internals
January 30, 2011
Le 2011-01-30 ? 11:28, Jason House a ?crit :

> Try "tail -n 1"

That'll give me the last line of the file. What "head -n-1" should do is give me the whole file minus the last line.


-- 
Michel Fortin
michel.fortin at michelf.com
http://michelf.com/



January 30, 2011
On 01/30/2011 10:51 AM, Michel Fortin wrote:
> Le 2011-01-30 ? 11:28, Jason House a ?crit :
>
>> Try "tail -n 1"
>
> That'll give me the last line of the file. What "head -n-1" should do is give me the whole file minus the last line.

I recall that is a nonstandard gnu extension of head. It could be simulated with

head -n $(( `wc -l < file` - 1 )) file


Andrei
January 30, 2011
Le 2011-01-30 ? 12:01, Andrei Alexandrescu a ?crit :

> I recall that is a nonstandard gnu extension of head. It could be simulated with
> 
> head -n $(( `wc -l < file` - 1 )) file

Thanks Andrei. I've fixed the tests in my own clone of the DMD repository. <https://github.com/michelf/dmd/commit/31205c13800b452b68b02382e80206f601438382>

Now it'd be nice if someone with write access to the main repo could merge it so I'm not the only one to benefit from it. :-)

-- 
Michel Fortin
michel.fortin at michelf.com
http://michelf.com/



January 30, 2011
On 1/30/2011 10:22 AM, Michel Fortin wrote:
> Le 2011-01-30 ? 12:01, Andrei Alexandrescu a ?crit :
> 
>> I recall that is a nonstandard gnu extension of head. It could be simulated with
>>
>> head -n $(( `wc -l < file` - 1 )) file
> 
> Thanks Andrei. I've fixed the tests in my own clone of the DMD repository. <https://github.com/michelf/dmd/commit/31205c13800b452b68b02382e80206f601438382>
> 
> Now it'd be nice if someone with write access to the main repo could merge it so I'm not the only one to benefit from it. :-)
> 

These look very familiar.  I thought I'd made the same changes when Sean and I were working on getting the tests to pass on OSX, but I guess they're still sitting on his box waiting to finish getting it running the auto-tester.  Oops.

Sorry you had to re-find and fix those.  I've pushed them into the main repo.