darcs

Patch 1112 resolve issue2345: solution using cabal's checkForeign...

Title resolve issue2345: solution using cabal's checkForeign...
Superseder Nosy List ganesh
Related Issues
Status accepted Assigned To
Milestone

Created on 2014-01-23.23:50:14 by ganesh, last changed 2014-04-23.17:45:35 by gh.

Files
File name Status Uploaded Type Edit Remove
patch-preview.txt ganesh, 2014-01-23.23:50:14 text/x-darcs-patch
resolve-issue2345_-solution-using-cabal_s-checkforeigndeps.dpatch ganesh, 2014-01-23.23:50:14 application/x-darcs-patch
unnamed ganesh, 2014-01-23.23:50:14
See mailing list archives for discussion on individual patches.
Messages
msg17142 (view) Author: ganesh Date: 2014-01-23.23:50:14
Adding the attachment from issue2345 as a patch in the tracker.

I'm going to screen this now and then mark it as reviewed once
people have had a chance to try it out a bit. It seems like
the right fix even if overriding the postConf hook may prove
dangerous in future.

1 patch for repository darcs-unstable@darcs.net:screened:

Mon Jan 20 22:05:52 GMT 2014  Dan Frumin <difrumin@gmail.com>
  * resolve issue2345: solution using cabal's checkForeignDeps
  
  Usually, Cabal performs the external dependencies check after the
  configuration phase, in the `postConf` hook. This patch moves this
  check into the configure section.
Attachments
msg17385 (view) Author: gh Date: 2014-04-23.17:45:34
Accepting this patch given its age and Ganesh's comment.
History
Date User Action Args
2014-01-23 23:50:14ganeshcreate
2014-02-01 15:44:44ganeshsetstatus: needs-screening -> needs-review
2014-04-23 17:45:35ghsetstatus: needs-review -> accepted
messages: + msg17385