Re: make coverage

2017-05-19 Thread Bruno Haible
Tim Rühsen wrote: > 1000 works here. Thanks for the testing. I'm applying this workaround: 2017-05-19 Bruno Haible printf-posix tests: Avoid test failure with "gcc --coverage". Reported by Tim Rühsen . * tests/test-printf-posix2.c (main): Test a width of 1000

Re: make coverage

2017-05-19 Thread Tim Rühsen
On 05/18/2017 10:15 PM, Bruno Haible wrote: > Hi Tim, > >>> What do you get if you change the value 500 in >>> tests/test-fprintf-posix2.c to a larger or smaller value? >> >> Both succeed with 5000 (10x higher). > > Good, that's the way I'd like to see it fixed. But a malloc(50 MB) > is e

Re: make coverage

2017-05-18 Thread Bruno Haible
Hi Tim, > > What do you get if you change the value 500 in > > tests/test-fprintf-posix2.c to a larger or smaller value? > > Both succeed with 5000 (10x higher). Good, that's the way I'd like to see it fixed. But a malloc(50 MB) is extraordinary; some people are using embedded systems. C

Re: make coverage

2017-05-18 Thread Tim Rühsen
On Donnerstag, 18. Mai 2017 19:29:18 CEST Bruno Haible wrote: > Tim Rühsen wrote: > > $ ./gnulib-tool --create-testdir --dir=../testdir1 > > $ cd ../testdir > > $ ./configure CFLAGS="-g -coverage" > > $ make clean && make coverage > > I did >

Re: make coverage

2017-05-18 Thread Bruno Haible
Tim Rühsen wrote: > $ ./gnulib-tool --create-testdir --dir=../testdir1 > $ cd ../testdir > $ ./configure CFLAGS="-g -coverage" > $ make clean && make coverage I did $ ./gnulib-tool --create-testdir --dir=../testdir-all $ cd ../testdir-all $ ./configure CFLAGS=&quo

Re: make coverage

2017-05-18 Thread Tim Rühsen
On 05/18/2017 01:00 PM, Bruno Haible wrote: > Hi Tim, > >> Processing file gllib/gllib/unicase/special-casing-table.gperf >> genhtml: ERROR: cannot read >> /usr/oms/src/testdir1/gllib/gllib/unicase/special-casing-table.gperf >> maint.mk:1500: recipe for target 'gen-coverage' failed >> >> (Maybe th

Re: make coverage

2017-05-18 Thread Tim Rühsen
binISaaHroFt0.bin Description: PGP/MIME version identification encrypted.asc Description: OpenPGP encrypted message

Re: make coverage

2017-05-18 Thread Tim Rühsen
On 05/17/2017 06:40 PM, Bruno Haible wrote: > Hi Tim, > >> 'make coverage' fails here due to two tests failing, thus no coverage >> report is created. >> I couldn't find a better solution. > > In the comment, you write "possible crash when comp

Re: make coverage

2017-05-18 Thread Bruno Haible
Hi Tim, > Processing file gllib/gllib/unicase/special-casing-table.gperf > genhtml: ERROR: cannot read > /usr/oms/src/testdir1/gllib/gllib/unicase/special-casing-table.gperf > maint.mk:1500: recipe for target 'gen-coverage' failed > > (Maybe this is because I still use gperf 3.0.4 !?) The gperf-

Re: make coverage

2017-05-17 Thread Bruno Haible
Hi Tim, > 'make coverage' fails here due to two tests failing, thus no coverage > report is created. > I couldn't find a better solution. In the comment, you write "possible crash when compiled with -coverage". Our tests should not crash, with or without --c

make coverage

2017-05-17 Thread Tim Rühsen
Hi, 'make coverage' fails here due to two tests failing, thus no coverage report is created. Here is a patch to fix this by skipping those tests. I couldn't find a better solution. There still is Processing file gllib/gllib/unicase/special-casing-table.gperf genhtml: ERROR: c