Update Inkscape usage for new versions of inkscape

numixproject / numix-core

Builder for App Icon Themes

GNU General Public License v3.0

765 stars 146 forks source link

Update Inkscape usage for new versions of inkscape #5622

Closed Supernova4422 closed 3 years ago

Supernova4422 commented 3 years ago

If inkscape is installed with Brew, the current commands will fail. This will ensure that inkscape works. Worth noting that we should use subprocess.run so error checking can be done.

Foggalong commented 3 years ago

From what I can the old code still works with the latest version of Inkscape on Linux, but changing it doesn't seem to affect anything for Linux users so if it fixes things for macOS users I'm happy to merge :+1: