pamplejuce icon indicating copy to clipboard operation
pamplejuce copied to clipboard

Should the line listed below in the cmake reference the project name rather than PAMPLEJUCE_IPP ?

Open 7sharp9 opened this issue 2 years ago • 4 comments

https://github.com/sudara/pamplejuce/blob/8824f49341223043fbccf2dc3953865abacf9659/CMakeLists.txt#L136

7sharp9 avatar May 11 '23 11:05 7sharp9

e.g PROJECT_NAME + _IPP

7sharp9 avatar May 11 '23 11:05 7sharp9

Ah, I see it referenced here: https://github.com/sudara/pamplejuce/blob/8824f49341223043fbccf2dc3953865abacf9659/Tests/PluginBasics.cpp#L19

Perhaps a generic name rather than the PAMPLEJUCE prefix?

7sharp9 avatar May 11 '23 11:05 7sharp9

Hi there!

Thanks for the the suggestion and taking the time to share it.

I like the benefit of the name staying the same across multiple projects. JUCE has JUCE_IPP_AVAILABLE which is what lead me to name it something differentiated that spoke to its origins.

I guess it's pretty easy to rename for anyone kicking off a new project...

Did it cause you a headache of some kind? (like you thought you should also change it to your project name along with everything else "Pamplejuce"y)

sudara avatar May 15 '23 18:05 sudara

It was really just for some consistancy rather than being tied to the template really. I dont think I got snagged my it as it would only really affect the test projects. Im sort of inbetween migrating to your template but thanks for doing this its a big help!

7sharp9 avatar May 26 '23 01:05 7sharp9