Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

Update master with the latest develop (tagged at v0.5.1) #185

Merged
merged 21 commits into from
Feb 12, 2016
Merged

Conversation

tacticalchihuahua
Copy link
Contributor

PR includes bug fixes, including corrupted settings fix as well as OS specific improvements.

@tacticalchihuahua
Copy link
Contributor Author

@littleskunk when you get a moment, can you review/test/merge and publish the release for v0.5.1 (already tagged).

@littleskunk
Copy link
Contributor

autobin binaries (only available for team members)
last commit: 184ad6f
driveshare_0.5.1.amd64.deb
driveshare_0.5.1.osx64.dmg
driveshare_0.5.1.win32.exe

@littleskunk
Copy link
Contributor

Not working on ubuntu. Installation worked but if i try to open driveshare i get an error. I deleted ~/.config/Driveshare but still the same error message.

~$ driveshare 
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Cannot find module 'request'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:289:25)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (/opt/driveshare/resources/app.asar/lib/userdata.js:9:15)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)

The new Driveshare-gui config folder is empty. Config file is missing.

~/.config/DriveShare$ ls
DevTools Extensions

@tacticalchihuahua
Copy link
Contributor Author

@littleskunk yep, looks like you need to install the deps.

cd path/to/driveshare-gui && npm install

@littleskunk
Copy link
Contributor

My fault. My linux build server needs to run npm install --unsafe-perm. That got lost with the last storjnode update. This time fixed with storj-archived/autobin@e9a52e9.

Retest with new binary worked. Time to merge.

littleskunk added a commit that referenced this pull request Feb 12, 2016
Update master with the latest develop (tagged at v0.5.1)
@littleskunk littleskunk merged commit a43b9a3 into master Feb 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants