(discussion) Keeping an eye on RIO? about magicbane

[akhra]

Anecdotally, in testing the adjustment to our (admittedly still small) current codebase to each of Classy and RIO in isolation, the latter required less adjustment.

Would it be worthwhile to explore the option with a fork? (Should that fork be called Snickersnee, or Grayswandir? Only the most important questions, here.)

[valpackett]

Yeah, I saw RIO on various news link sites, it looks interesting I guess.

Magicbane was extracted from a Classy based application, but there's no actual reason to use one prelude over another here…

What about the option of not using custom preludes here? Honestly the only reason was that import Magicbane is the one and only import in the little demos, but that's silly. I'd totally accept a PR that replaces Classy with the regular base Prelude :)

[akhra]

That'd just be a matter of not re-exporting it, right? And fixing the examples.

[akhra]

I guess that would still force the dependency for people using another prelude. Still, it opens options and is a clean&easy first step.

[valpackett]

No, I mean completely removing it

[akhra]

Fair enough, shouldn't be too much more work.

[akhra]

99.9% done, examples/tiny.hs runs; examples/larger.hs is choking on:

No instance for (exceptions-0.8.3:Control.Monad.Catch.MonadMask
                         (MagicbaneApp LargeAppCtx))
        arising from a use of ‘timed’

Do you happen to have an insight on where that instance was coming from previously?

[akhra]

For reference, here's where I'm at. The only thing that seems to be broken is the timed call, both in larger.hs and the second readme example. It doesn't seem to be due to the absence of ClassyPrelude tho, as I went ahead and imported it for the examples (and moved it to "recommended" in readme) so basically everything it provides should be there. I may have broken something in Magicbane.App?

[akhra]

Strike that -- I just made a fresh clone of your repository and got the same errors. Not my fault! :P (Or, if my fault, something in tooling/environment, not code.) Gonna open the PR, further discussion there.