Replaced `/home/nc` with environment variable

kernicPanel / bitwig-randomizer

Bitwig randomizer

41 stars 5 forks source link

Replaced `/home/nc` with environment variable #1

Closed erroreyes closed 2 years ago

erroreyes commented 2 years ago

Hello, I am a fellow Bitwig Discord user, and I mentioned that I was interested in learning from your extension.

For this PR, I updated the hardcoded /home/nc path in the pom file with a general user.home environment variable so that others interested in compiling this project will have the extension file copied to their correct extensions path.

kernicPanel commented 2 years ago

Hi, thanks for your contribution! Do you think it would work on every platforms ? Anyway this is definitely an improvement.

erroreyes commented 2 years ago

I read the manual provided by DrivenByMoss, and it looks like Windows, OSX, and Linux have different paths:

Windows: %USERPROFILE%\Documents\Bitwig Studio\Extensions\ Mac: ~/Documents/Bitwig Studio/Extensions/ Linux: ~/Bitwig Studio/Extensions/

So it'd be good to update pom.xml to cater to each OS.

kernicPanel commented 2 years ago

Sorry for my late reply… It seems like moss uses a custom variable for this part in the POM. But I don't know how to it manage yet…