search

digital-asset / ghc-lib

The GHC API, decoupled from GHC versions
Other
161 stars 9 forks source link

ghc-lib-parser-9.8.2.20240223 failed to build with ghc-9.8.3 #563

Open juhp opened 13 hours ago

juhp commented 13 hours ago

For Stackage Nightly with ghc-9.8.3:

       Building library for ghc-lib-parser-9.8.2.20240223..                                                                         
       [155 of 433] Compiling GHC.Unit.Types                                                                                        

       /home/curators/work/unpack-dir/unpacked/ghc-lib-parser-9.8.2.20240223-2c6194cfab3c6d198ba355d48ffc919705e56dc74190a757398fed9
667763406/compiler/GHC/Unit/Types.hs:127:25: error: [GHC-87543]                                                                     
           Ambiguous occurrence ‘Unit’.                                                                                             
           It could refer to                                                                                                        
              either ‘Control.DeepSeq.Unit’,                                                                                        
                     imported from ‘Control.DeepSeq’ at compiler/GHC/Unit/Types.hs:104:1-22,                                        
                  or ‘GHC.Unit.Types.Unit’,                                                                                         
                     defined at compiler/GHC/Unit/Types.hs:289:1.                                                                   
           |                                                                                                                        
       127 | type Module = GenModule Unit                                                                                           
           |                         ^^^^                                                                                           

       /home/curators/work/unpack-dir/unpacked/ghc-lib-parser-9.8.2.20240223-2c6194cfab3c6d198ba355d48ffc919705e56dc74190a757398fed9
667763406/compiler/GHC/Unit/Types.hs:334:14: error: [GHC-87543]                                                                     
           Ambiguous occurrence ‘Unit’.                                                                                             
           It could refer to                                                                                                        
              either ‘Control.DeepSeq.Unit’,                                                                                        
                     imported from ‘Control.DeepSeq’ at compiler/GHC/Unit/Types.hs:104:1-22,                                        
                  or ‘GHC.Unit.Types.Unit’,                                                                                         
                     defined at compiler/GHC/Unit/Types.hs:289:1.
           |
       334 | instance Ord Unit where
           |              ^^^^

       /home/curators/work/unpack-dir/unpacked/ghc-lib-parser-9.8.2.20240223-2c6194cfab3c6d198ba355d48ffc919705e56dc74190a757398fed9
667763406/compiler/GHC/Unit/Types.hs:337:15: error: [GHC-87543]
           Ambiguous occurrence ‘Unit’.
           It could refer to
              either ‘Control.DeepSeq.Unit’,
                     imported from ‘Control.DeepSeq’ at compiler/GHC/Unit/Types.hs:104:1-22,
                  or ‘GHC.Unit.Types.Unit’,
                     defined at compiler/GHC/Unit/Types.hs:289:1.
           |
       337 | instance Data Unit where
           |               ^^^^
:
:
:

many more Unit errors.

shayne-fletcher commented 12 hours ago

ghc-lib-parser flavor 9.8.2 can't be built with a ghc 9.8.3 build compiler. when #561 lands it will correctly fail to produce a build plan for this case.

i've adjusted the bounds on 9.8.2.20240223

juhp commented 8 hours ago

Thanks and I see you already released for 9.8.3 :pray:

Only problem is it requires happy < 2 and we already have happy-2.0 in Stackage Nightly for quite a while now.

juhp commented 8 hours ago

Actually 9.8.2 seemed to build fine with happy-2.0 - is the bound now necessary?