breakpoint breakage

2007-08-28 Thread Manuel M T Chakravarty
I just ran validate on the current head. Unexpected failures: break003(ghci) break006(ghci) break012(ghci) break013(ghci) break018(ghci) break021(ghci) I guess this is due to simultaneous changes in both the type families patch and ghci debugger code. (Ie, both changes have b

darcs-all breakage

2007-08-28 Thread Manuel M T Chakravarty
Unfortunately, a change to darcs-all snug inadvertently into the type families patch that breaks it for ssh repos at least with Perl 5.8.8 (on MacOS). A patch that reverts the change is in the repo now. So, if you experience problems just do a plain "darcs pull" first, then darcs-all should b

patch applied (ghc): FIX: Remove accidential change to darcs-all in type families patch

2007-08-28 Thread chak
Tue Aug 28 18:00:11 PDT 2007 Manuel M T Chakravarty <[EMAIL PROTECTED]> * FIX: Remove accidential change to darcs-all in type families patch - The type families patch includes a change to darcs-all that breaks it for ssh repos at least for Perl 5.8.8 (on MacOS). - My Perl-fu is not suffi

patch applied (ghc): Remove INSTALL_INCLUDES; no longer used

2007-08-28 Thread Ian Lynagh
Tue Aug 28 13:56:36 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Remove INSTALL_INCLUDES; no longer used M ./mk/install.mk -8 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): Use DESTDIR when installing

2007-08-28 Thread Ian Lynagh
Tue Aug 28 13:51:19 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Use DESTDIR when installing M ./Makefile -3 +2 M ./driver/ghc/Makefile -1 +1 M ./driver/ghci/Makefile -1 +1 M ./libraries/Makefile -1 +1 M ./libraries/installPackage.hs -8 +20 M ./mk/install.mk -60 +61 M .

patch applied (ghc): Copy LICENSE files into the bindist, as Cabal now installs them

2007-08-28 Thread Ian Lynagh
Tue Aug 28 06:04:28 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Copy LICENSE files into the bindist, as Cabal now installs them M ./libraries/Makefile +1 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): TAG 2007-08-28

2007-08-28 Thread Ian Lynagh
Tue Aug 28 14:54:45 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> tagged 2007-08-28 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

[nightly] 28-Aug-2007 build of HEAD on i386-unknown-linux (cam-02-unx.europe.corp.microsoft.com)

2007-08-28 Thread GHC Build Reports
Build description = HEAD on i386-unknown-linux (cam-02-unx.europe.corp.microsoft.com) Build location= /playpen/ghc/nightly/HEAD-cam-02-unx Build config file = /home/simonmar/nightly/site/msrc/conf-HEAD-cam-02-unx Nightly build started on cam-02-unx at Tue Aug 28 19:30:01 BST 2007. checki

patch applied (ghc): Better handling of join points in spill cleaner

2007-08-28 Thread Ben Lippmeier
Tue Aug 28 06:30:50 PDT 2007 [EMAIL PROTECTED] * Better handling of join points in spill cleaner M ./compiler/nativeGen/RegSpillClean.hs -24 +256 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): comment wibble

2007-08-28 Thread Ben Lippmeier
Fri Aug 24 09:03:14 PDT 2007 [EMAIL PROTECTED] * comment wibble M ./compiler/nativeGen/PprMach.hs -11 M ./compiler/nativeGen/RegAllocInfo.hs -8 +4 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): Add count of reg-reg moves remaining for linear allocator stats

2007-08-28 Thread Ben Lippmeier
Fri Aug 24 08:57:32 PDT 2007 [EMAIL PROTECTED] * Add count of reg-reg moves remaining for linear allocator stats M ./compiler/nativeGen/AsmCodeGen.lhs -1 +1 M ./compiler/nativeGen/RegAllocLinear.hs -5 +28 ___ Cvs-ghc mailing list Cvs-ghc@hask

patch applied (ghc): Add a count of how many spill/reloads/reg-reg-moves remain to dump-asm-stats

2007-08-28 Thread Ben Lippmeier
Fri Aug 24 08:21:42 PDT 2007 [EMAIL PROTECTED] * Add a count of how many spill/reloads/reg-reg-moves remain to dump-asm-stats M ./compiler/nativeGen/RegAllocColor.hs -1 +2 M ./compiler/nativeGen/RegAllocStats.hs -26 +52 ___ Cvs-ghc mailing li

Re: Warnings

2007-08-28 Thread Simon Marlow
Ian Lynagh wrote: I'd go for plain old -Wall, personally. How much is this style of name shadowing used in GHC? Do people think it is desirable? Personally I like to use shadowing and I think -fwarn-name-shadowing is overkill, but there are cases where it can catch real bugs. Take this as a

Re: windows head build problems: Setup.exe: errorreading dist\setup-config;

2007-08-28 Thread Claus Reinke
Incidentally, I've just realised that if it did work then it would also tell you that you have added all the files in all the libraries, testsuite, etc. indeed, and one has to be careful when filtering those out, because mixed in between are real left-over files that need removing.. that's one r

Re: Warnings

2007-08-28 Thread Ian Lynagh
On Tue, Aug 28, 2007 at 03:22:33PM +0100, Simon Peyton-Jones wrote: > Norman's suggestion for warnings when building GHC. I was actually thinking about warnings the other day. The problem is that to keep code warning free you really need -Werror, or people won't notice and fix them, but that's a p

Warnings

2007-08-28 Thread Simon Peyton-Jones
Norman's suggestion for warnings when building GHC. If we agree this we should add it to guidance for developers: 1. Switch warnings off by default in build.mk.sample. Reason: too many are generated. 2. Whenever a developer does non-trivial work on a module, ask them to a) add {-# O

patch applied (ghc): Use INSTALL_HEADERS in includes/ rather than abusing INSTALL_DATAS

2007-08-28 Thread Ian Lynagh
Tue Aug 28 05:49:18 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Use INSTALL_HEADERS in includes/ rather than abusing INSTALL_DATAS M ./includes/Makefile -7 +4 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-

patch applied (ghc): Make some header-installing machinery, and use it to install gmp.h

2007-08-28 Thread Ian Lynagh
Tue Aug 28 05:47:59 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Make some header-installing machinery, and use it to install gmp.h M ./gmp/Makefile -2 +9 M ./mk/config.mk.in +3 M ./mk/install.mk +8 ___ Cvs-ghc mailing list Cvs-ghc@haskell.o

patch applied (ghc): Tidy up the gmp/ Makefile a bit

2007-08-28 Thread Ian Lynagh
Tue Aug 28 05:32:18 PDT 2007 Ian Lynagh <[EMAIL PROTECTED]> * Tidy up the gmp/ Makefile a bit M ./gmp/Makefile -17 +13 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (testsuite): Added tests for :steplocal and :stepmodule

2007-08-28 Thread Pepe Iborra
Tue Aug 28 04:54:03 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Added tests for :steplocal and :stepmodule A ./tests/ghc-regress/ghci.debugger/scripts/Break020b.hs M ./tests/ghc-regress/ghci.debugger/scripts/all.T +2 A ./tests/ghc-regress/ghci.debugger/scripts/break020.hs A ./tes

distclean ghosts/darcsclean (was: windows head build problems)

2007-08-28 Thread Claus Reinke
btw, Ian pointed out that compiler/stage1, being boring, won't even appear in 'darcs whatsnew -l', but his suggestion to add --boring doesn't work for me. firstly, darcs gets really memory-hungry when i add that flag, and secondly, it gives less, rather than more output. am i doing anything wro

patch applied (testsuite): Accept output

2007-08-28 Thread Pepe Iborra
Tue Aug 28 03:32:40 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Accept output M ./tests/ghc-regress/ghci.debugger/scripts/break001.stdout -6 +6 M ./tests/ghc-regress/ghci.debugger/scripts/break003.stdout -4 +4 M ./tests/ghc-regress/ghci.debugger/scripts/break005.stdout -5 +5 M ./

patch applied (testsuite): Aad test break020 for :stepover

2007-08-28 Thread Pepe Iborra
Tue Aug 14 09:44:06 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> UNDO: Aad test break020 for :stepover M ./tests/ghc-regress/ghci.debugger/scripts/all.T -1 R ./tests/ghc-regress/ghci.debugger/scripts/break020.hs R ./tests/ghc-regress/ghci.debugger/scripts/break020.script R ./tests/g

patch applied (ghc): :stepover ---> :steplocal, :stepmodule

2007-08-28 Thread Pepe Iborra
Mon Aug 27 11:09:49 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * :stepover ---> :steplocal, :stepmodule :stepover is declared a failed experiment. :steplocal steps only on ticks contained in the current top level declaration. :stepmodule steps only on ticks contained on the current

patch applied (ghc): Use a version of obtainTerm that takes a max depth bound

2007-08-28 Thread Pepe Iborra
Mon Aug 27 10:23:15 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Use a version of obtainTerm that takes a max depth bound when printing the contents of binding at a breakpoint M ./compiler/ghci/InteractiveUI.hs -1 +1 M ./compiler/ghci/RtClosureInspect.hs -15 +18 M ./compiler/main/GH

patch applied (ghc): Be more careful when calculating the enclosing top level declaration of a srcspan

2007-08-28 Thread Pepe Iborra
Mon Aug 27 04:44:37 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Be more careful when calculating the enclosing top level declaration of a srcspan M ./compiler/main/InteractiveEval.hs -7 +10 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.

patch applied (ghc): Style: remove trailing spaces

2007-08-28 Thread Pepe Iborra
Sun Aug 26 14:33:57 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Style: remove trailing spaces M ./compiler/ghci/Debugger.hs -23 +23 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): Print contents of bindings when stopping at a breakpoint

2007-08-28 Thread Pepe Iborra
Sun Aug 26 14:33:39 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> * Print contents of bindings when stopping at a breakpoint M ./compiler/ghci/Debugger.hs -23 +28 M ./compiler/ghci/InteractiveUI.hs -5 +25 M ./compiler/main/GHC.hs -1 +9 M ./compiler/main/InteractiveEval.hs -1 +1

patch applied (ghc): A partial attempt to improve :stepover

2007-08-28 Thread Pepe Iborra
Fri Aug 24 06:40:22 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> UNDO: A partial attempt to improve :stepover With this patch, :stepover can effectively appear to step over recursive calls and calls to locally bound functions (in a where clause). However, when we run out of t

patch applied (ghc): Better document :stepover and its limitations

2007-08-28 Thread Pepe Iborra
Wed Aug 22 01:42:54 PDT 2007 Pepe Iborra <[EMAIL PROTECTED]> UNDO: Better document :stepover and its limitations :stepover only works lexically locally, in the context of the current expression. I have tried to make this point clear in the users guide with an example. M ./comp

Re: windows head build problems: Setup.exe: errorreading dist\setup-config;

2007-08-28 Thread Ian Lynagh
On Tue, Aug 28, 2007 at 12:08:05PM +0100, Claus Reinke wrote: > >but tried to clean most of distclean's left-overs as well, and that Ian > >and i > >agreed that a darcs-clean target would be helpful to automate that. > > btw, Ian pointed out that compiler/stage1, being boring, won't even > appea

Re: windows head build problems: Setup.exe: error reading dist\setup-config;

2007-08-28 Thread Simon Marlow
Claus Reinke wrote: Claus Reinke wrote: $ ls compiler/stage1/ghc-inplace* compiler/stage1/ghc-inplace compiler/stage1/ghc-inplace.bat compiler/stage1/ghc-inplace.exe Ah, this could well be the source of the problem; you should only have ghc-inplace.exe. Does removing the others fix i

Re: windows head build problems: Setup.exe: errorreading dist\setup-config;

2007-08-28 Thread Claus Reinke
but tried to clean most of distclean's left-overs as well, and that Ian and i agreed that a darcs-clean target would be helpful to automate that. btw, Ian pointed out that compiler/stage1, being boring, won't even appear in 'darcs whatsnew -l', but his suggestion to add --boring doesn't work f

patch applied (testsuite): add test for #1593

2007-08-28 Thread Simon Marlow
Tue Aug 28 03:25:08 PDT 2007 Simon Marlow <[EMAIL PROTECTED]> * add test for #1593 M ./tests/ghc-regress/lib/Process/all.T +1 A ./tests/ghc-regress/lib/Process/process004.hs A ./tests/ghc-regress/lib/Process/process004.stdout ___ Cvs-ghc

patch applied (testsuite): cc013 works now

2007-08-28 Thread Simon Marlow
Tue Aug 28 03:24:33 PDT 2007 Simon Marlow <[EMAIL PROTECTED]> * cc013 works now M ./tests/ghc-regress/ccall/should_compile/all.T -1 +1 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

Re: windows head build problems: Setup.exe: error reading dist\setup-config;

2007-08-28 Thread Claus Reinke
Claus Reinke wrote: $ ls compiler/stage1/ghc-inplace* compiler/stage1/ghc-inplace compiler/stage1/ghc-inplace.bat compiler/stage1/ghc-inplace.exe Ah, this could well be the source of the problem; you should only have ghc-inplace.exe. Does removing the others fix it? i changed the Ma

patch applied (ghc): FIX #1533: foreign exporing the same identifier multiple times gave a link error

2007-08-28 Thread Simon Marlow
Tue Aug 28 02:15:50 PDT 2007 Simon Marlow <[EMAIL PROTECTED]> * FIX #1533: foreign exporing the same identifier multiple times gave a link error We were generating a new top-level binding derived from the name of the existing top-level name, and making the name external. Multiple instanc

patch applied (ghc): convert to use System.FilePath

2007-08-28 Thread Simon Marlow
Sun Aug 26 08:19:03 PDT 2007 Simon Marlow <[EMAIL PROTECTED]> * convert to use System.FilePath M ./utils/ghc-pkg/Main.hs -85 +8 ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (ghc): Refactoring only: remove [Id] field from ForeignStubs

2007-08-28 Thread Simon Marlow
Sun Aug 26 00:33:22 PDT 2007 Simon Marlow <[EMAIL PROTECTED]> * Refactoring only: remove [Id] field from ForeignStubs We used to pass the list of top-level foreign exported bindings to the code generator so that it could create StablePtrs for them in the stginit code. Now we don't use stg

Re: patch applied (ghc): Type checking for type synonym families

2007-08-28 Thread Manuel M T Chakravarty
* Type checking for type synonym families This patch introduces type checking for type families of which associated type synonyms are a special case. Please note that this is very experimental code. We have tried hard to ensure that it does not break existing code (scream if it still

Re: windows head build problems: Setup.exe: error reading dist\setup-config;

2007-08-28 Thread Simon Marlow
Claus Reinke wrote: $ ls compiler/stage1/ghc-inplace* compiler/stage1/ghc-inplace compiler/stage1/ghc-inplace.bat compiler/stage1/ghc-inplace.exe Ah, this could well be the source of the problem; you should only have ghc-inplace.exe. Does removing the others fix it? i changed the Ma

Daily report for head

2007-08-28 Thread BuildBot Collator
Build results: x86-64 Linux head: pass x86 Windows head: pass x86 Windows head fast: pass pass pass pass fail (failed darcs) pass gbesh Intel x86_64 Linux head: pass kahl G5 Gentoo Linux head: pass macgyver PPC OSX head: fail (failed stage1) mnemosyne x8

patch applied (ghc): Type checking for type synonym families

2007-08-28 Thread chak
Mon Aug 27 23:18:51 PDT 2007 Manuel M T Chakravarty <[EMAIL PROTECTED]> * Type checking for type synonym families This patch introduces type checking for type families of which associated type synonyms are a special case. E.g. type family Sum n m type instance Su

patch applied (ghc): TAG Before type family merge

2007-08-28 Thread chak
Mon Aug 27 18:11:43 PDT 2007 Manuel M T Chakravarty <[EMAIL PROTECTED]> tagged Before type family merge ___ Cvs-ghc mailing list Cvs-ghc@haskell.org http://www.haskell.org/mailman/listinfo/cvs-ghc

patch applied (testsuite): TypeFamilies: check arity of type instances

2007-08-28 Thread chak
Mon Aug 27 23:55:51 PDT 2007 Manuel M T Chakravarty <[EMAIL PROTECTED]> * TypeFamilies: check arity of type instances A ./tests/ghc-regress/indexed-types/should_fail/TyFamArity1.hs A ./tests/ghc-regress/indexed-types/should_fail/TyFamArity1.stderr A ./tests/ghc-regress/indexed-types

patch applied (testsuite): Changes necessary for type family patch

2007-08-28 Thread chak
Mon Aug 27 22:51:40 PDT 2007 Manuel M T Chakravarty <[EMAIL PROTECTED]> * Changes necessary for type family patch - Mostly changed error messages - 'while' needs another type annotation due to changed rules as to what annotations are required for GADTs. M ./tests/ghc-regress/gadt/rw