Move library modules to src/

maoe commented 7 years ago

The motivation behind this change is to

avoid unnecessary recompilation for lib, exe, and tests
- This travis log is an example of unnecessary recompilation. It built the modules for library and the same modules got recompiled for the ghc-events executable.
- With this change, cabal no longer recompiles the modules
fix the warning stack prints like below

Warning: The following modules should be added to exposed-modules
or other-modules in /Users/maoe/src/github.com/haskell/ghc-events/ghc-events.cabal:
    - In ghc-events component:
        GHC.RTS.EventParserUtils
        GHC.RTS.EventTypes
        GHC.RTS.Events
        GHC.RTS.Events.Analysis
        GHC.RTS.Events.Analysis.Capability
        GHC.RTS.Events.Analysis.SparkThread
        GHC.RTS.Events.Analysis.Thread
        GHC.RTS.Events.Merge
        GHC.RTS.EventsIncremental- In test-versions component:
        GHC.RTS.EventParserUtils
        GHC.RTS.EventTypes
        GHC.RTS.Events
        GHC.RTS.EventsIncremental
        Utils- In write-merge component:
        GHC.RTS.EventParserUtils
        GHC.RTS.EventTypes
        GHC.RTS.Events
        GHC.RTS.EventsIncremental
        Utils

Mikolaj commented 7 years ago

Well spotted. LGTM.

kvelicka commented 7 years ago

Good job!

maoe commented 7 years ago

I believe this is not controversial and I've confirmed threadscope builds fine with this. I'm going to merge this now.

haskell / ghc-events

Move library modules to src/ #20