search

turion / sonnendemo

Demonstration app for rhine-gloss containing a little simulation of a solar panel, a wind turbine and a battery
BSD 3-Clause "New" or "Revised" License
5 stars 0 forks source link

support rhine-gloss-0.7 and lts-17? #13

Open juhp opened 3 years ago

juhp commented 3 years ago

Currently it doesn't build with lts-17 but lts-16 is fine.

/var/home/petersen/haskell/rhine/sonnendemo/sonnendemo/MainGloss.hs:189:35: error:
    Ambiguous occurrence ‘backgroundColor’
    It could refer to
       either ‘FRP.Rhine.Gloss.backgroundColor’,
              imported from ‘FRP.Rhine.Gloss’ at sonnendemo/MainGloss.hs:23:1-22
              (and originally defined in ‘FRP.Rhine.Gloss.Common’)
           or ‘Main.backgroundColor’,
              defined at sonnendemo/MainGloss.hs:34:1
    |
189 |   , translate (-20) (-10) $ color backgroundColor $ circleSolid 40
    |                                   ^^^^^^^^^^^^^^^

/var/home/petersen/haskell/rhine/sonnendemo/sonnendemo/MainGloss.hs:351:59: error:
    Ambiguous occurrence ‘backgroundColor’
    It could refer to
       either ‘FRP.Rhine.Gloss.backgroundColor’,
              imported from ‘FRP.Rhine.Gloss’ at sonnendemo/MainGloss.hs:23:1-22
              (and originally defined in ‘FRP.Rhine.Gloss.Common’)
           or ‘Main.backgroundColor’,
              defined at sonnendemo/MainGloss.hs:34:1
    |
351 | main = flowGloss (InWindow "sonnen" (1024, 768) (10, 10)) backgroundColor 30 glossRhine
    |                                                           ^^^^^^^^^^^^^^^
turion commented 3 years ago

Yes, there were breaking changes from 0.6 to rhine-gloss-0.7. Probably the minimum change should be an upper version bound for the meantime. But it is not too hard to update this package. I'll let the issue free for a while in case anyone wants to give it a try :)