Fix installer paths on Linux (fixes #96)

ByzantineFailure / BPM-for-Discord

BPM for Discord's Desktop App. Includes one-click installers, update notifications, and custom script support.

GNU Affero General Public License v3.0

17 stars 8 forks source link

Fix installer paths on Linux (fixes #96) #97

Closed Jibodeah closed 6 years ago

Jibodeah commented 6 years ago

Also adds separate PTB and Canary installer scripts for Linux, much like those that already exist for Mac and Windows.

ByzantineFailure commented 6 years ago

You fixed the paths and added ptb and canary installers? ... ❤️

Will merge once I'm home, add the new files to the Makefile, and re-release working for Linux :)

Hithroc commented 6 years ago

This doesn't work on my system

% sudo ./install_linux.sh
Could not find node of version v4.2.x, found node version v9.3.0. What would you like to do?
1) Download correct node and install  3) Exit
2) Install using local version
#? 2
Please enter the install directory
/opt/discord
Installing to /opt/discord
{ _: [ '/home/hithroc/Downloads/discord' ], p: '/opt/discord' }
Cannot find discord modules.  Please run discord once before installing!

Am I doing something wrong?

Hithroc commented 6 years ago

Okay, figured this one pretty quickly. Now that paths.js uses environment variables like HOME and XDG_CONFIG_HOME, I needed to preserve them with sudo.

sudo -E ./install_linux.sh