search

turion / essence-of-live-coding-tutorial

Tutorial for essence-of-live-coding
BSD 3-Clause "New" or "Revised" License
16 stars 4 forks source link

Add note to README.md how to disable PulseAudio backend #7

Closed neosimsim closed 4 years ago

neosimsim commented 4 years ago

Sorry, I guess I was a bit slow with turion/essence-of-live-coding-tutorial/pull/5.

neosimsim commented 4 years ago

I'd guess most people have switched to new-style cabal by now, what do you think?

I know the Agda community still uses the old style a lot. I had a little discussion there. IIUC the v2- prefix is recommended to be used in scripts etc. to avoid breaking changes. For the same reason I like to use the prefix in instructions.

neosimsim commented 4 years ago

Sorry to be that noisy but.

For a future-proof way to use these commands in a script or tutorial that anticipates the possibility of another UI paradigm being devised in the future, there are also v2- prefixed versions that will reference the same functionality until such a point as it is completely removed from Cabal.

https://cabal.readthedocs.io/en/3.4/nix-local-build-overview.html

neosimsim commented 4 years ago

Please don’t merge!

I just realized that you are using pulse code in main by now. You should keep the flag, but only mention it once you implemented an alternative backend.

turion commented 4 years ago

Please don’t merge!

I just realized that you are using pulse code in main by now. You should keep the flag, but only mention it once you implemented an alternative backend.

That just means that if people switch that flag off then things won't compile anymore, and they'll need to comment out all Pulse related code.

neosimsim commented 4 years ago

That just means that if people switch that flag off then things won't compile anymore, and they'll need to comment out all Pulse related code.

For now, yes. In future an alternative should than be enabled. E.g that’s how integer-gmp and integer-simple are swapped.