Skip to content

Commit

Permalink
Pick the right architecture on linux atom#12972
Browse files Browse the repository at this point in the history
I don't know if the fixation of the architecture on non linux platforms makes any sense (line 25). Maybe simply 'arch': process.arch is the right way to go. Can't test it on my linux machine.
  • Loading branch information
flububb committed Oct 16, 2016
1 parent ce4b920 commit 35bc2cf
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion script/lib/package-application.js
Expand Up @@ -18,7 +18,12 @@ module.exports = function () {
'app-bundle-id': 'com.github.atom',
'app-copyright': `Copyright © 2014-${(new Date()).getFullYear()} GitHub, Inc. All rights reserved.`,
'app-version': CONFIG.appMetadata.version,
'arch': process.platform === 'win32' ? 'ia32' : 'x64',
'arch': (() => {
if (process.platform === 'linux') {
return process.arch
} else {
return process.platform === 'win32' ? 'ia32' : 'x64'
}})(),
'asar': {unpack: buildAsarUnpackGlobExpression()},
'build-version': CONFIG.appMetadata.version,
'download': {cache: CONFIG.electronDownloadPath},
Expand Down

0 comments on commit 35bc2cf

Please sign in to comment.