Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use fluidlite instead of fluidsynth, for easier compilation #2149

Merged
merged 8 commits into from
Aug 26, 2024

Conversation

timothyschoen
Copy link
Contributor

I leave it up to you if you want this, it has a few drawbacks.

  • Some sf3 files that use obscure formats internally may not work. I've never encountered this in the real world, because why would anyone do that!
  • Printing preset info doesn't work yet

The advantage is that it has a super simple build system with no external dependencies. I've been using this for years in plugdata without any problems, and without breaking compilation ever. It also gets rid of all the pre-built libs you have in the sfont folder.

It may be possible to bring preset info dump back, though I'm not sure how.

@timothyschoen
Copy link
Contributor Author

OK, this now actually compiles sfont~ in the actions, for each platform

@porres
Copy link
Owner

porres commented Aug 16, 2024

It may be possible to bring preset info dump back, though I'm not sure how.

I'm really fond of being able to do this :/

@porres porres force-pushed the master branch 3 times, most recently from 0a1fae8 to 5a4489f Compare August 20, 2024 00:44
@timothyschoen
Copy link
Contributor Author

OK, I've accidentally put the compilation fix for play.file~ into this branch

@timothyschoen
Copy link
Contributor Author

Also implemented preset printing for [sfont~], so this should be ready for merge!

@porres
Copy link
Owner

porres commented Aug 26, 2024

you're the best, let me try this!

@porres porres merged commit 14c6b23 into porres:master Aug 26, 2024
4 checks passed
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