- User Since
- Sep 3 2017, 4:26 PM (66 w, 3 d)
Remove duplicate pragma.
Note that the build won't be totally warning free just yet because this patch doesn't suppress these warnings for the libraries.
Wed, Dec 5
Sorry, I was wrong. GHC does generate warnings because of this.
I looked at the stage1 and stage2 build logs, and GHC itself didn't generate any new warnings. I'll try to fix any warnings that other libraries in haskell/ might generate.
Thu, Nov 29
Wed, Nov 28
@RyanGlScott Oops, sorry. Fixed it :)
Bring back -ddump-tc.
Tue, Nov 27
Fixed some failing tests using 'traceTcRnForUser'.
(fixing the build)
Mon, Nov 26
@bgamari Should be ready for review soon :) I'm still working on rewriting the GCC parts.
@bgamari I started looking at DriverPipeline, and got distracted.
Also, even after this patch, '-ddump-mod-map' (used in Packages.mkPackageState) still prints the output to STDOUT,
because dumpPrefix is still Nothing. Does DriverPipeline.runPipeline not run before we get to mkPackageState?
On the other hand, it works ok if we use -ddump-file-prefix=blah by hand.
Fix other instances of when (dopt ...) print ....
As @RyanGlScott pointed out, there are a few other places where when (dopt ...) print ... is used directly. Fixing those now.
Fri, Nov 23
No, because those options will not have been processed by the RTS. Why would users need to do that?
@simonmar Oh, I see what you mean. Filtering these out would actually make it seem that they were in fact processed by the RTS,
which would be incorrect.
Start updating semantics per @simonmar's comment (TODO: remove GCC bits)
Nov 12 2018
Thanks @simonmar. I'll update this patch in a couple days.
Nov 9 2018
Oct 29 2018
This is still a WIP patch for option 2 mentioned in comment:2 in Trac #15732 (extending the RTS to parse response files). I submitted it here to get some early feedback. And there are a few things that must be dealt with:
Oct 22 2018
LGTM. Thanks @AndreasK :)
Sep 18 2018
The ticket mentions one more thing which I have not implemented:
consider making liftA2 known-key for Traversable deriving.
Sep 17 2018
Sep 6 2018
Aug 28 2018
(We should also merge this in the 8.6 branch)
Thanks @RyanGlScott. Fixed it.
Fix typo in the commit message.
Aug 24 2018
You can grep for grep, some tests declare what to look for the in the dump. But in this example, I think it is hard to write a reliable grep command.
Get rid of the test
Oh right, I should've used -dsuppress-uniques. Would a comment in the test file help ? Otherwise, I'll update this patch to not include it.
Aug 23 2018
It turns out that git rm PATH && git commit removes a sub-module from the git working tree but doesn't delete the relevant section from .gitmodules.
May 9 2018
Still WIP, and did not intend to submit a patch.
May 6 2018
In general I wish that :set weren't so destructive, but that is another issue altogether.
May 5 2018
May 4 2018
Does this look alright ?
May 3 2018
Oh no, it was my fault. I'm AFK right now, but I'll submit another patch to update this hint tomorrow.
Oops. Sorry, I should've changed the status to "Changes Planned" or something. Should I file a trac issue to fix this "hint" (@RyanGlScott's comment) ? Or is OK in it's current form ?
That's a good point. If we decide to add this hint, it could maybe include a note about this effect.
Apr 23 2018
Include an example that uses getArgsWithResponseFiles in the haddocks.
Apr 22 2018
Apr 21 2018
The build failure seems unrelated to the patch:
exception 'PhabricatorWorkerPermanentFailureException' with message 'Lease "PHID-DRYL-zbqyzktfang7i4ajamnt" never activated.'
Define getArgsWithResponseFiles in GHC.ResponseFile instead of System.Environment.
Mar 19 2018
Mar 17 2018
Mention the trac number in the comment.
Feb 26 2018
Fixed the instance for Lifetime
@RyanGlScott I think most of the annotations should be correct. I've tagged some of the places which I wasn't sure about.
Fixed deriving instance Traversable Identity