Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Make the tor binary executable on all platforms.
Browse files Browse the repository at this point in the history
Also, do it to the binary, not to the zip file, on Windows.  Not sure
how this worked before!  Maybe the file got truncated rather than
replaced?

Auditors:
@diracdeltas
@jumde

Test Plan:

Launch Brave.  New private tab.  Turn on Tor.  Visit
https://check.torproject.org and confirm Tor is working.
  • Loading branch information
riastradh-brave authored and diracdeltas committed May 7, 2018
1 parent 1c5ec3c commit 297afb2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tools/package_tor.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,9 @@ execute([cmd], '', (err) => {
console.log('tor binary checksum matches')
// unzip on windows
if (isWindows) {
fs.chmodSync(torBinary, 0o755)
fs.createReadStream(torBinary).pipe(unzip.Extract({ path: torPath }))
}
// make it executable
fs.chmodSync(torBinary, 0o755)
console.log('done')
})

0 comments on commit 297afb2

Please sign in to comment.