Skip to content

Commit 968ad2e

Browse files
committed
Expose Copilot.Compile.Bluespec.External. Refs #36.
The non-public `Copilot.Compiler.Bluespec.External` module lives under a `shared/` directory so that it be imported by both the library code and the test suite code. This is a questionable design, as if the test suite needs to use the code, then it should be exposed by the library. This commit moves `Copilot.Compiler.Bluespec.External` from `shared/` to `src/`, exposes the module as part of the public API, and removes the `shared/` directory. As a result, the test suite can now import `Copilot.Compiler.Bluespec.External` like it does other parts of the `copilot-bluespec` API.
1 parent 8ed8429 commit 968ad2e

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

copilot-bluespec.cabal

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ source-repository head
3636

3737
library
3838
default-language : Haskell2010
39-
hs-source-dirs : src, shared
39+
hs-source-dirs : src
4040

4141
ghc-options : -Wall
4242
build-depends : base >= 4.9 && < 5
@@ -48,12 +48,12 @@ library
4848
, language-bluespec >= 0.1 && < 0.2
4949

5050
exposed-modules : Copilot.Compile.Bluespec
51+
, Copilot.Compile.Bluespec.External
5152

5253
other-modules : Copilot.Compile.Bluespec.CodeGen
5354
, Copilot.Compile.Bluespec.Compile
5455
, Copilot.Compile.Bluespec.Error
5556
, Copilot.Compile.Bluespec.Expr
56-
, Copilot.Compile.Bluespec.External
5757
, Copilot.Compile.Bluespec.FloatingPoint
5858
, Copilot.Compile.Bluespec.Name
5959
, Copilot.Compile.Bluespec.Representation
@@ -70,8 +70,6 @@ test-suite tests
7070
other-modules:
7171
Test.Copilot.Compile.Bluespec
7272

73-
Copilot.Compile.Bluespec.External
74-
7573
build-depends:
7674
base
7775
, directory
@@ -91,7 +89,7 @@ test-suite tests
9189
, copilot-bluespec
9290

9391
hs-source-dirs:
94-
tests, shared
92+
tests
9593

9694
default-language:
9795
Haskell2010
File renamed without changes.

0 commit comments

Comments
 (0)