Skip to content

Commit

Permalink
Fix linux binary (#14)
Browse files Browse the repository at this point in the history
linux jqExecutableName should be one of `jq-linux64` or `jq-linux32` (not `jq-linux-x64` or `jq-linux-x32`)

Reviewed-by: Jamie Young <jmeyoung@gmail.com>
Reviewed-by:@adamjmcgrath
Fixes: #15 
PR-URL: #14
  • Loading branch information
adamjmcgrath authored and jamsyoung committed May 31, 2019
1 parent 5bf9b9c commit 00d417b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions link-binary.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ const debug = require('debug')('jqcw');
const BinWrapper = require('bin-wrapper');
const base = 'https://github.com/stedolan/jq/releases/download/jq-1.5';
const platform = process.platform === 'darwin' ? 'osx' : 'linux';
const arch = process.platform === 'darwin' ? 'amd64' : process.arch;
const jqExecutableName = `jq-${platform}-${arch}`;
const arch = process.platform === 'darwin' ? '-amd64' : process.arch.replace(/^x/, '');
const jqExecutableName = `jq-${platform}${arch}`;
const bin = new BinWrapper()
.src(`${base}/jq-osx-amd64`, 'darwin')
.src(`${base}/jq-linux64`, 'linux', 'x64')
Expand Down

0 comments on commit 00d417b

Please sign in to comment.