Skip to content

Conversation

@rrrnld
Copy link
Contributor

@rrrnld rrrnld commented Sep 18, 2022

Heya! I just started playing around with tidalcycles and was wondering how I'd load up custom samples with this flake. I saw #2 and gave this a shot in this PR. It's backwards-compatible but allows passing in a custom script for startup as the first argument to superdirt-start. If no argument is given it behaves the same as before and loads the simple start script that just contains SuperDirt.start;.

It's a simple solution that I think could work? Thanks for providing this, it was a great help in getting started!

@rrrnld
Copy link
Contributor Author

rrrnld commented Sep 26, 2022

Any feedback on this? I noticed that the script name may be weird, because strictly speaking when using a custom script you can use it for other stuff than starting superdirt, but I think it still works because the environment is set up with all dependencies superdirt. Also keeping the name means it stays backwards-compatible.

Some additional context maybe: I use this flake system-wide. I don't know if the script I wrote makes sense in a dev shell as well, but I think it should.

Copy link
Owner

@mitchmindtree mitchmindtree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about the delayed response, only just spotted this!

I think this looks good. Eventually perhaps we can just forward all arguments onto sclang instead? That said, it's probably good to keep this simple, as it's really just a tiny wrapper around calling sclang directly which provides a lot more flexibility. E.g. within the devShell something like this should also work today:

sclang -l SUPERDIRT_SCLANG_CONF my-supercollider-script

@mitchmindtree mitchmindtree merged commit 3f3a820 into mitchmindtree:main Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants