Skip to content

Conversation

jpcima
Copy link
Collaborator

@jpcima jpcima commented May 6, 2020

This adds a build helper for DPF plugins with sfizz.
There is an explanation how to use it at the top of the file.

@rghvdberg

@jpcima jpcima requested a review from paulfd May 6, 2020 08:02
@jpcima
Copy link
Collaborator Author

jpcima commented May 6, 2020

Abseil seems to pull more deps under DEBUG=true, hence I updated to take it in consideration.

@pdesaulniers hi, possibly it interests you to check this work as well.

@paulfd
Copy link
Member

paulfd commented May 6, 2020

Btw, how come we pull in so many unrelated things from abseils? Like the time zone and synchronization primitives. I think we mostly use optional, span, the atoi-type helpers and a couple string functions here and there no? And the string_views..

@jpcima
Copy link
Collaborator Author

jpcima commented May 6, 2020

I've included there the strict necessary set of libraries which is needed to please the linker, not one more.
Their respective source listings are from abseil's subdirectory CMakeLists, listed under SRCS.

@paulfd
Copy link
Member

paulfd commented May 6, 2020

The question was more aimed at absl than your script 🙂

@jpcima jpcima merged commit 1d53ad9 into sfztools:develop May 6, 2020
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