| > 1. If you have lots of libraries in your tree, 'sh validate' will | try to compile them. But they are not -Wall clean, so the -Werror | kills the validation. | > | > Solution: we should say what libraries validate will validate; and | validate should make build.mk compile only them. | > | > The easiest way to do this looks to be: make GhcBootLibs=YES work, | and specify that those are the libs that are validated. | | Is there a reason you're building all (?) the extra libs by default? | | I find it quite convenient to be able to just symlink them into a tree | when I want to test them and have it Just Work, but to have a faster | build by default.
That may be fine for you, but we are asking *everyone* to run validate, and we can't predict what is fine for them. Can we simply make validate do the right thing? It already checks that you have *enough* libraries; can't it simply arrange to use only them? | You could always put "GhcBootLibs=YES" in mk/validate.mk. On the other | hand, I could put "GhcBootLibs=NO" in mine; I don't really mind | whichever way. I urge _strongly_ that it's validate's responsibility. So if GhcBootLibs=YES is exactly what is needed for validation, then can you do just that, please? Incidentally, just to confirm, I never have to even look in validate.mk do I? It's 100% validate's responsibility. | > Incidentally are GhcBootLibs the same as "core libs"? | | Yes, I think so. | | > Do we have this terminology Wiki'd somewhere? | | We should probably just rename the variable. Excellent thank you. And is the "core lib" and "extra lib" terminology described anywhere on the Wiki? | OK, it now uses mk/validate-settings.mk when validating. mk/build.mk is | ignored, with mk/validate.mk taking on its role. Wonderful thank you. That completely solves that problem. | > 3. On Windows I'm getting warnings from utils/genprimopcode/Lexer.hs | which kill the validation. There's no -w at the top, and Simon and I | aren't sure if there should be. But it's getting compiled with -Werror | nevertheless. Dump below. | | I've added a -w pragma; strange that it worked for me, though. I have | alex 2.1.0; what version do you have? I believe I had 2.0.1. Perhaps I should upgrade! thanks Ian Simon _______________________________________________ Cvs-ghc mailing list [email protected] http://www.haskell.org/mailman/listinfo/cvs-ghc
