Skip to content

Commit

Permalink
fix absolute path bug in darwin install - fix #45
Browse files Browse the repository at this point in the history
  • Loading branch information
stefanjudis committed Oct 15, 2015
1 parent a0ad55c commit 05466cb
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
5 changes: 4 additions & 1 deletion index.js
Expand Up @@ -23,11 +23,14 @@ var Builder = {
* @param {Function} callback callback
*/
build : function( options, callback ) {

options = options || {};
options.log = options.log || console.log;
options.out = options.out ? path.resolve( process.cwd(), options.out ) : process.cwd();


options.log(
'- Running electron-builder ' + require( './package' ).version
);

// make sure the output
// directory ends with a slash
Expand Down
6 changes: 3 additions & 3 deletions lib/osx.js
Expand Up @@ -28,12 +28,12 @@ var OSXBuilder = {
return callback( new Error( 'Invalid platform.' ) );
}

options.log( '- Starting build for ´' + options.platform + '´ - ' );
options.log( '- Starting creation of installer for ´' + options.platform );

options.log( 'Writing temporary ´appdmg.json´' );

options.config.osx.background = path.join( options.basePath, options.config.osx.background );
options.config.osx.icon = path.join( options.basePath, options.config.osx.icon );
options.config.osx.background = path.resolve( options.basePath, options.config.osx.background );
options.config.osx.icon = path.resolve( options.basePath, options.config.osx.icon );
options.config.osx.contents[ 1 ].path = options.appPath;

var configFilePath = path.join( os.tmpDir(), 'appdmg.json' );
Expand Down
2 changes: 1 addition & 1 deletion lib/win.js
Expand Up @@ -25,7 +25,7 @@ var WinBuilder = {
* @param {Function} callback callback
*/
build : function( options, callback ) {
options.log( '- Starting build for ´' + options.platform + '´ - ' );
options.log( '- Starting creation of installer for ´' + options.platform );

var configFilePath = writeConfigFile( 'installer.nsi', {
appPath : _windowsify( options.appPath ),
Expand Down

0 comments on commit 05466cb

Please sign in to comment.