snowleopard / hadrian

Hadrian: a new build system for the Glasgow Haskell Compiler. Now merged into the GHC tree!
https://gitlab.haskell.org/ghc/ghc/tree/master/hadrian
MIT License
208 stars 37 forks source link

Added includes to Bindist #632

Closed chitrak7 closed 6 years ago

chitrak7 commented 6 years ago

Adding includes directory to binary distribution.

We ship include folder in rts directory with binary distrbution.

@alpmestan Is this approach correct?

alpmestan commented 6 years ago

Can you show us the list of header files you end up shipping, with this patch?

chitrak7 commented 6 years ago

This is the tree generated hadrian bld/include/ ├── Cmm.h ├── DerivedConstants.h ├── ffi.h ├── ffitarget.h ├── ghcautoconf.h ├── ghcconfig.h ├── ghcplatform.h ├── ghcversion.h ├── HsFFI.h ├── MachDeps.h ├── rts │   ├── Adjustor.h │   ├── BlockSignals.h │   ├── Bytecodes.h │   ├── Config.h │   ├── Constants.h │   ├── EventLogFormat.h │   ├── EventLogWriter.h │   ├── FileLock.h │   ├── Flags.h │   ├── GetTime.h │   ├── Globals.h │   ├── Hpc.h │   ├── IOManager.h │   ├── Libdw.h │   ├── LibdwPool.h │   ├── Linker.h │   ├── Main.h │   ├── Messages.h │   ├── OSThreads.h │   ├── Parallel.h │   ├── PrimFloat.h │   ├── prof │   │   ├── CCS.h │   │   └── LDV.h │   ├── Profiling.h │   ├── Signals.h │   ├── SpinLock.h │   ├── Stable.h │   ├── StaticPtrTable.h │   ├── storage │   │   ├── Block.h │   │   ├── ClosureMacros.h │   │   ├── Closures.h │   │   ├── ClosureTypes.h │   │   ├── FunTypes.h │   │   ├── GC.h │   │   ├── Heap.h │   │   ├── InfoTables.h │   │   ├── MBlock.h │   │   └── TSO.h │   ├── Threads.h │   ├── Ticky.h │   ├── Time.h │   ├── Timer.h │   ├── TTY.h │   ├── Types.h │   └── Utils.h ├── RtsAPI.h ├── Rts.h ├── stg │   ├── DLL.h │   ├── HaskellMachRegs.h │   ├── MachRegs.h │   ├── MiscClosures.h │   ├── Prim.h │   ├── Regs.h │   ├── RtsMachRegs.h │   ├── SMP.h │   ├── Ticky.h │   └── Types.h └── Stg.h

chitrak7 commented 6 years ago

This is the tree for ghc-8.4.2 include directory ../ghc-8.2.1/bld/lib/ghc-8.4.2/include/ ├── Cmm.h ├── DerivedConstants.h ├── ffi.h ├── ffitarget.h ├── ghcautoconf.h ├── ghcconfig.h ├── ghcplatform.h ├── ghcversion.h ├── HsFFI.h ├── MachDeps.h ├── rts │   ├── Adjustor.h │   ├── BlockSignals.h │   ├── Bytecodes.h │   ├── Config.h │   ├── Constants.h │   ├── EventLogFormat.h │   ├── EventLogWriter.h │   ├── FileLock.h │   ├── Flags.h │   ├── GetTime.h │   ├── Globals.h │   ├── Hpc.h │   ├── IOManager.h │   ├── Libdw.h │   ├── LibdwPool.h │   ├── Linker.h │   ├── Main.h │   ├── Messages.h │   ├── OSThreads.h │   ├── Parallel.h │   ├── PrimFloat.h │   ├── prof │   │   ├── CCS.h │   │   └── LDV.h │   ├── Profiling.h │   ├── Signals.h │   ├── SpinLock.h │   ├── Stable.h │   ├── StaticPtrTable.h │   ├── storage │   │   ├── Block.h │   │   ├── ClosureMacros.h │   │   ├── Closures.h │   │   ├── ClosureTypes.h │   │   ├── FunTypes.h │   │   ├── GC.h │   │   ├── InfoTables.h │   │   ├── MBlock.h │   │   └── TSO.h │   ├── Threads.h │   ├── Ticky.h │   ├── Time.h │   ├── Timer.h │   ├── TTY.h │   ├── Types.h │   └── Utils.h ├── RtsAPI.h ├── Rts.h ├── stg │   ├── DLL.h │   ├── HaskellMachRegs.h │   ├── MachRegs.h │   ├── MiscClosures.h │   ├── Prim.h │   ├── Regs.h │   ├── RtsMachRegs.h │   ├── SMP.h │   ├── Ticky.h │   └── Types.h └── Stg.h

alpmestan commented 6 years ago

Looks like we've got it all? In which case: LGTM =)

chitrak7 commented 6 years ago

@alpmestan Added feature to ship ghci also.

snowleopard commented 6 years ago

Looks good to me too, so merging. Thanks @chitrak7 and @alpmestan!