Re: recomp013
I couldn't repro on a devel2 build. Edward Excerpts from Simon Peyton Jones's message of 2015-10-26 08:24:00 -0700: > driver/recomp013 is failing, on Linux, on HEAD for me: > > Actual stdout output differs from expected: > > --- ./driver/recomp013/recomp013.stdout.normalised2015-10-26 > 14:26:51.920153558 + > > +++ ./driver/recomp013/recomp013.run.stdout.normalised 2015-10-26 > 14:26:51.920153558 + > > @@ -3,5 +3,3 @@ > > [2 of 3] Compiling B( B.hs, B.o ) > > [3 of 3] Compiling C( C.hs, C.o ) > > second run > > -[2 of 3] Compiling B( B.hs, B.o ) > > -[3 of 3] Compiling C( C.hs, C.o ) [B changed] ___ ghc-devs mailing list ghc-devs@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs
Re: recomp013
The test as in the repo also fails on my Mac (18 out of 20 times in a row). Adding a 'sleep 1' before the second set of echo commands makes it succeed (20 out of 20 times in a row). On Mon, Sep 23, 2013 at 7:43 AM, Simon Peyton-Jones simo...@microsoft.comwrote: No, it fails on Linux. Mostly but not invariably. Running it by hand (ie with TEST=recomp013) fails 9 times out of 10 and succeeds once. Weird. Simon | -Original Message- | From: Simon Marlow [mailto:marlo...@gmail.com] | Sent: 23 September 2013 12:08 | To: Gabor Greif; Simon Peyton-Jones | Cc: Joachim Breitner; ghc-devs@haskell.org | Subject: Re: recomp013 | | Is it failing on Windows only? It could be an issue with the | granularity of timestamps. I think we use more accurate timestamps on | Linux nowadays, whereas Windows is still using 1 second granularity. If | that's the case, it probably needs a sleep 1 somewhere. If the sleep | 1 didn't help, perhaps it wasn't in the right place - it needs to be | before the second set of echo commands. | | On the subject of naming the test, we want two pieces of information in | the name: |- what it tests |- the ticket number, if it's a regression test | So rather than just renaming it to the ticket number (which would lose | the information that it's a recomp test), I suggest moving all the | recomp tests into their own directory first. It's really useful to have | all the tests for a component together, so you can run them quickly when | you're working on that component. | | Cheers, | Simon | | On 19/09/2013 08:57, Gabor Greif wrote: | On 9/19/13, Simon Peyton-Jones simo...@microsoft.com wrote: | I lied. Whether or not the 'sleep' is there, the test usually fails, | but | occasionally (one time in ten or so) succeeds. | | I am the originator of the bug report, and could consistently | reproduce it in a bigger project. Unfortunately I have no time | currently to dive into the issue, but I have it earmarked for later. | | Cheers, | | Gabor | | | | SImon | | | -Original Message- | | From: ghc-devs [mailto:ghc-devs-boun...@haskell.org] On Behalf Of | Simon | | Peyton-Jones | | Sent: 19 September 2013 12:50 | | To: Joachim Breitner; ghc-devs@haskell.org | | Subject: RE: recomp013 | | | | that makes it work.! | | | | | -Original Message- | | | From: Joachim Breitner [mailto:m...@joachim-breitner.de] | | | Sent: 19 September 2013 12:41 | | | To: Simon Peyton-Jones; ghc-devs@haskell.org | | | Subject: Re: recomp013 | | | | | | Dear Simon, | | | | | | Am Donnerstag, den 19.09.2013, 11:33 + schrieb Simon Peyton- | Jones: | | | = recomp013(normal) 89 of 114 [0, 0, 0] | | | cd ./recomp013 $MAKE -s --no-print-directory recomp013 | | | /dev/null recomp013.run.stdout 2recomp013.run.stderr | | | Actual stdout output differs from expected: | | | --- ./recomp013/recomp013.stdout 2013-09-18 12:38:22.0 | | +0100 | | | +++ ./recomp013/recomp013.run.stdout 2013-09-19 | 12:15:22.611120935 | | | +0100 | | | @@ -3,5 +3,3 @@ | | | [2 of 3] Compiling B( B.hs, B.o ) | | | [3 of 3] Compiling C( C.hs, C.o ) | | | second run | | | -[2 of 3] Compiling B( B.hs, B.o ) | | | -[3 of 3] Compiling C( C.hs, C.o ) [B changed] | | | *** unexpected failure for recomp013(normal) | | | | | | It works here. So that means either two things: | | | * that the bug described in #8247 is actually present in HEAD, | but | | |only under certain circumstances, or | | | * that there is a timing issue with the update of B.hs. | | | | | | The latter seems to be more likely. I notice that other recomp- | tests | | | have a sleep 1 before updating the file: If you insert that in | | | recomp013/Makefile, does it go through for you? | | | | | | Greetings, | | | Joachim | | | | | | -- | | | Joachim nomeata Breitner | | | m...@joachim-breitner.de * http://www.joachim-breitner.de/ | | | Jabber: nome...@joachim-breitner.de * GPG-Key: 0x4743206C | | | Debian Developer: nome...@debian.org | | ___ | | ghc-devs mailing list | | ghc-devs@haskell.org | | http://www.haskell.org/mailman/listinfo/ghc-devs | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs | | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs | ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo
Re: recomp013
Is it failing on Windows only? It could be an issue with the granularity of timestamps. I think we use more accurate timestamps on Linux nowadays, whereas Windows is still using 1 second granularity. If that's the case, it probably needs a sleep 1 somewhere. If the sleep 1 didn't help, perhaps it wasn't in the right place - it needs to be before the second set of echo commands. On the subject of naming the test, we want two pieces of information in the name: - what it tests - the ticket number, if it's a regression test So rather than just renaming it to the ticket number (which would lose the information that it's a recomp test), I suggest moving all the recomp tests into their own directory first. It's really useful to have all the tests for a component together, so you can run them quickly when you're working on that component. Cheers, Simon On 19/09/2013 08:57, Gabor Greif wrote: On 9/19/13, Simon Peyton-Jones simo...@microsoft.com wrote: I lied. Whether or not the 'sleep' is there, the test usually fails, but occasionally (one time in ten or so) succeeds. I am the originator of the bug report, and could consistently reproduce it in a bigger project. Unfortunately I have no time currently to dive into the issue, but I have it earmarked for later. Cheers, Gabor SImon | -Original Message- | From: ghc-devs [mailto:ghc-devs-boun...@haskell.org] On Behalf Of Simon | Peyton-Jones | Sent: 19 September 2013 12:50 | To: Joachim Breitner; ghc-devs@haskell.org | Subject: RE: recomp013 | | that makes it work.! | | | -Original Message- | | From: Joachim Breitner [mailto:m...@joachim-breitner.de] | | Sent: 19 September 2013 12:41 | | To: Simon Peyton-Jones; ghc-devs@haskell.org | | Subject: Re: recomp013 | | | | Dear Simon, | | | | Am Donnerstag, den 19.09.2013, 11:33 + schrieb Simon Peyton-Jones: | | = recomp013(normal) 89 of 114 [0, 0, 0] | | cd ./recomp013 $MAKE -s --no-print-directory recomp013 | | /dev/null recomp013.run.stdout 2recomp013.run.stderr | | Actual stdout output differs from expected: | | --- ./recomp013/recomp013.stdout 2013-09-18 12:38:22.0 | +0100 | | +++ ./recomp013/recomp013.run.stdout 2013-09-19 12:15:22.611120935 | | +0100 | | @@ -3,5 +3,3 @@ | | [2 of 3] Compiling B( B.hs, B.o ) | | [3 of 3] Compiling C( C.hs, C.o ) | | second run | | -[2 of 3] Compiling B( B.hs, B.o ) | | -[3 of 3] Compiling C( C.hs, C.o ) [B changed] | | *** unexpected failure for recomp013(normal) | | | | It works here. So that means either two things: | | * that the bug described in #8247 is actually present in HEAD, but | |only under certain circumstances, or | | * that there is a timing issue with the update of B.hs. | | | | The latter seems to be more likely. I notice that other recomp-tests | | have a sleep 1 before updating the file: If you insert that in | | recomp013/Makefile, does it go through for you? | | | | Greetings, | | Joachim | | | | -- | | Joachim “nomeata” Breitner | | m...@joachim-breitner.de • http://www.joachim-breitner.de/ | | Jabber: nome...@joachim-breitner.de • GPG-Key: 0x4743206C | | Debian Developer: nome...@debian.org | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs
RE: recomp013
No, it fails on Linux. Mostly but not invariably. Running it by hand (ie with TEST=recomp013) fails 9 times out of 10 and succeeds once. Weird. Simon | -Original Message- | From: Simon Marlow [mailto:marlo...@gmail.com] | Sent: 23 September 2013 12:08 | To: Gabor Greif; Simon Peyton-Jones | Cc: Joachim Breitner; ghc-devs@haskell.org | Subject: Re: recomp013 | | Is it failing on Windows only? It could be an issue with the | granularity of timestamps. I think we use more accurate timestamps on | Linux nowadays, whereas Windows is still using 1 second granularity. If | that's the case, it probably needs a sleep 1 somewhere. If the sleep | 1 didn't help, perhaps it wasn't in the right place - it needs to be | before the second set of echo commands. | | On the subject of naming the test, we want two pieces of information in | the name: |- what it tests |- the ticket number, if it's a regression test | So rather than just renaming it to the ticket number (which would lose | the information that it's a recomp test), I suggest moving all the | recomp tests into their own directory first. It's really useful to have | all the tests for a component together, so you can run them quickly when | you're working on that component. | | Cheers, | Simon | | On 19/09/2013 08:57, Gabor Greif wrote: | On 9/19/13, Simon Peyton-Jones simo...@microsoft.com wrote: | I lied. Whether or not the 'sleep' is there, the test usually fails, | but | occasionally (one time in ten or so) succeeds. | | I am the originator of the bug report, and could consistently | reproduce it in a bigger project. Unfortunately I have no time | currently to dive into the issue, but I have it earmarked for later. | | Cheers, | | Gabor | | | | SImon | | | -Original Message- | | From: ghc-devs [mailto:ghc-devs-boun...@haskell.org] On Behalf Of | Simon | | Peyton-Jones | | Sent: 19 September 2013 12:50 | | To: Joachim Breitner; ghc-devs@haskell.org | | Subject: RE: recomp013 | | | | that makes it work.! | | | | | -Original Message- | | | From: Joachim Breitner [mailto:m...@joachim-breitner.de] | | | Sent: 19 September 2013 12:41 | | | To: Simon Peyton-Jones; ghc-devs@haskell.org | | | Subject: Re: recomp013 | | | | | | Dear Simon, | | | | | | Am Donnerstag, den 19.09.2013, 11:33 + schrieb Simon Peyton- | Jones: | | | = recomp013(normal) 89 of 114 [0, 0, 0] | | | cd ./recomp013 $MAKE -s --no-print-directory recomp013 | | | /dev/null recomp013.run.stdout 2recomp013.run.stderr | | | Actual stdout output differs from expected: | | | --- ./recomp013/recomp013.stdout 2013-09-18 12:38:22.0 | | +0100 | | | +++ ./recomp013/recomp013.run.stdout 2013-09-19 | 12:15:22.611120935 | | | +0100 | | | @@ -3,5 +3,3 @@ | | | [2 of 3] Compiling B( B.hs, B.o ) | | | [3 of 3] Compiling C( C.hs, C.o ) | | | second run | | | -[2 of 3] Compiling B( B.hs, B.o ) | | | -[3 of 3] Compiling C( C.hs, C.o ) [B changed] | | | *** unexpected failure for recomp013(normal) | | | | | | It works here. So that means either two things: | | | * that the bug described in #8247 is actually present in HEAD, | but | | |only under certain circumstances, or | | | * that there is a timing issue with the update of B.hs. | | | | | | The latter seems to be more likely. I notice that other recomp- | tests | | | have a sleep 1 before updating the file: If you insert that in | | | recomp013/Makefile, does it go through for you? | | | | | | Greetings, | | | Joachim | | | | | | -- | | | Joachim nomeata Breitner | | | m...@joachim-breitner.de * http://www.joachim-breitner.de/ | | | Jabber: nome...@joachim-breitner.de * GPG-Key: 0x4743206C | | | Debian Developer: nome...@debian.org | | ___ | | ghc-devs mailing list | | ghc-devs@haskell.org | | http://www.haskell.org/mailman/listinfo/ghc-devs | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs | | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs | ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs
Re: recomp013
Simon, I think you didn't paste the failure. Janek - Oryginalna wiadomość - Od: Simon Peyton-Jones simo...@microsoft.com Do: ghc-devs@haskell.org, Joachim Breitner m...@joachim-breitner.de Wysłane: czwartek, 19 wrzesień 2013 12:17:29 Temat: recomp013 I’m getting the failure below for recomp013. Any ideas? It was added by Joachim: commit b1f6cab360faf79d586eda8ee3c1c486e53d0066 Author: Joachim Breitner m...@joachim-breitner.de Date: Tue Sep 17 22:37:51 2013 +0200 Recompilation test case for #8247 Joachim: when you have a test for a ticket, please use “T8247” as the test name. Much easier to track back! Thanks. Simon Microsoft Research Limited (company number 03369488) is registered in England and Wales Registered office is at 21 Station Road, Cambridge, CB1 2FB ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs
RE: recomp013
Sorry, here = recomp013(normal) 89 of 114 [0, 0, 0] cd ./recomp013 $MAKE -s --no-print-directory recomp013/dev/null recomp013.run.stdout 2recomp013.run.stderr Actual stdout output differs from expected: --- ./recomp013/recomp013.stdout2013-09-18 12:38:22.0 +0100 +++ ./recomp013/recomp013.run.stdout2013-09-19 12:15:22.611120935 +0100 @@ -3,5 +3,3 @@ [2 of 3] Compiling B( B.hs, B.o ) [3 of 3] Compiling C( C.hs, C.o ) second run -[2 of 3] Compiling B( B.hs, B.o ) -[3 of 3] Compiling C( C.hs, C.o ) [B changed] *** unexpected failure for recomp013(normal) | -Original Message- | From: Jan Stolarek [mailto:jan.stola...@p.lodz.pl] | Sent: 19 September 2013 12:22 | To: Simon Peyton-Jones | Cc: ghc-devs@haskell.org; Joachim Breitner | Subject: Re: recomp013 | | Simon, I think you didn't paste the failure. | | Janek | | - Oryginalna wiadomość - | Od: Simon Peyton-Jones simo...@microsoft.com | Do: ghc-devs@haskell.org, Joachim Breitner m...@joachim-breitner.de | Wysłane: czwartek, 19 wrzesień 2013 12:17:29 | Temat: recomp013 | | | | I’m getting the failure below for recomp013. Any ideas? It was added by | Joachim: | | | | commit b1f6cab360faf79d586eda8ee3c1c486e53d0066 | | Author: Joachim Breitner m...@joachim-breitner.de | | Date: Tue Sep 17 22:37:51 2013 +0200 | | | | Recompilation test case for #8247 | | | | Joachim: when you have a test for a ticket, please use “T8247” as the | test name. Much easier to track back! Thanks. | | | | Simon | | | | | | Microsoft Research Limited (company number 03369488) is registered in | England and Wales | | Registered office is at 21 Station Road, Cambridge, CB1 2FB | | | | ___ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs
Re: recomp013
Dear Simon, Am Donnerstag, den 19.09.2013, 11:33 + schrieb Simon Peyton-Jones: = recomp013(normal) 89 of 114 [0, 0, 0] cd ./recomp013 $MAKE -s --no-print-directory recomp013/dev/null recomp013.run.stdout 2recomp013.run.stderr Actual stdout output differs from expected: --- ./recomp013/recomp013.stdout 2013-09-18 12:38:22.0 +0100 +++ ./recomp013/recomp013.run.stdout 2013-09-19 12:15:22.611120935 +0100 @@ -3,5 +3,3 @@ [2 of 3] Compiling B( B.hs, B.o ) [3 of 3] Compiling C( C.hs, C.o ) second run -[2 of 3] Compiling B( B.hs, B.o ) -[3 of 3] Compiling C( C.hs, C.o ) [B changed] *** unexpected failure for recomp013(normal) It works here. So that means either two things: * that the bug described in #8247 is actually present in HEAD, but only under certain circumstances, or * that there is a timing issue with the update of B.hs. The latter seems to be more likely. I notice that other recomp-tests have a sleep 1 before updating the file: If you insert that in recomp013/Makefile, does it go through for you? Greetings, Joachim -- Joachim “nomeata” Breitner m...@joachim-breitner.de • http://www.joachim-breitner.de/ Jabber: nome...@joachim-breitner.de • GPG-Key: 0x4743206C Debian Developer: nome...@debian.org signature.asc Description: This is a digitally signed message part ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs
RE: recomp013
that makes it work.! | -Original Message- | From: Joachim Breitner [mailto:m...@joachim-breitner.de] | Sent: 19 September 2013 12:41 | To: Simon Peyton-Jones; ghc-devs@haskell.org | Subject: Re: recomp013 | | Dear Simon, | | Am Donnerstag, den 19.09.2013, 11:33 + schrieb Simon Peyton-Jones: | = recomp013(normal) 89 of 114 [0, 0, 0] | cd ./recomp013 $MAKE -s --no-print-directory recomp013 | /dev/null recomp013.run.stdout 2recomp013.run.stderr | Actual stdout output differs from expected: | --- ./recomp013/recomp013.stdout2013-09-18 12:38:22.0 +0100 | +++ ./recomp013/recomp013.run.stdout2013-09-19 12:15:22.611120935 | +0100 | @@ -3,5 +3,3 @@ | [2 of 3] Compiling B( B.hs, B.o ) | [3 of 3] Compiling C( C.hs, C.o ) | second run | -[2 of 3] Compiling B( B.hs, B.o ) | -[3 of 3] Compiling C( C.hs, C.o ) [B changed] | *** unexpected failure for recomp013(normal) | | It works here. So that means either two things: | * that the bug described in #8247 is actually present in HEAD, but |only under certain circumstances, or | * that there is a timing issue with the update of B.hs. | | The latter seems to be more likely. I notice that other recomp-tests | have a sleep 1 before updating the file: If you insert that in | recomp013/Makefile, does it go through for you? | | Greetings, | Joachim | | -- | Joachim “nomeata” Breitner | m...@joachim-breitner.de • http://www.joachim-breitner.de/ | Jabber: nome...@joachim-breitner.de • GPG-Key: 0x4743206C | Debian Developer: nome...@debian.org ___ ghc-devs mailing list ghc-devs@haskell.org http://www.haskell.org/mailman/listinfo/ghc-devs