diff options
author | Edward Z. Yang <ezyang@cs.stanford.edu> | 2016-12-26 18:39:01 -0800 |
---|---|---|
committer | Edward Z. Yang <ezyang@cs.stanford.edu> | 2017-01-11 06:53:52 -0800 |
commit | 5f9c6d2a91ac710e7b75cfe50a7a8e84cc9ae796 (patch) | |
tree | 6127e630abd03e8f4106cb19b197976afcaa7f11 /testsuite/tests/backpack/should_compile/all.T | |
parent | 436aa7aaf1b30f19ece6c610e357cb678612de8a (diff) | |
download | haskell-5f9c6d2a91ac710e7b75cfe50a7a8e84cc9ae796.tar.gz |
Support for using only partial pieces of included signatures.
Summary:
Generally speaking, it's not possible to "hide" a requirement from a
package you include, because if there is some module relying on that
requirement, well, you can't just wish it out of existence.
However, some packages don't have any modules. For these, we can
validly thin out requirements; indeed, this is very convenient if
someone has published a large signature package but you only want
some of the definitions.
This patchset tweaks the interpretation of export lists in
signatures: in particular, they no longer need to refer to
entities that are defined locally; they range over both the current
signature as well as any signatures that were inherited from
signature packages (defined by having zero exposed modules.)
In the process of doing this, I cleaned up a number of other
things:
* rnModIface and rnModExports now report errors that occurred
during renaming and can propagate these to the TcM monad.
This is important because in the current semantics, you can
thin out a type which is referenced by a value you keep;
in this situation, we need to error (to ensure that all
types in signatures are rooted, so that we can determine
their identities).
* I ended up introducing a new construct 'dependency signature;
to bkp files, to make it easier to tell if we were depending
on a signature package. It's not difficult for Cabal to
figure this out (I already have a patch for it.)
Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
Test Plan: validate
Reviewers: simonpj, austin, bgamari
Subscribers: thomie, mpickering
Differential Revision: https://phabricator.haskell.org/D2904
GHC Trac Issues: #12994
Diffstat (limited to 'testsuite/tests/backpack/should_compile/all.T')
-rw-r--r-- | testsuite/tests/backpack/should_compile/all.T | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/testsuite/tests/backpack/should_compile/all.T b/testsuite/tests/backpack/should_compile/all.T index bb77278d26..299b28a7bc 100644 --- a/testsuite/tests/backpack/should_compile/all.T +++ b/testsuite/tests/backpack/should_compile/all.T @@ -34,3 +34,5 @@ test('bkp39', normal, backpack_compile, ['']) test('bkp40', normal, backpack_compile, ['']) test('bkp41', normal, backpack_compile, ['']) test('bkp42', normal, backpack_compile, ['']) +test('bkp43', normal, backpack_compile, ['']) +test('bkp44', normal, backpack_compile, ['']) |