New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TIMOB-24912] Android: Running a module after it was built fails on Windows. #9184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR'd and FR'd. APPROVED
android/cli/commands/_buildModule.js
Outdated
} | ||
function runTiCommand(cmd, args, logger, callback) { | ||
|
||
// when calling on Windows, we need to escape ampersands in the command |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the indention here. We use tabs, not spaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs some coding style fixes.
@cb1kenobi Updated with proper indentation. |
android/cli/commands/_buildModule.js
Outdated
if (process.platform == 'win32' && /\.bat$/.test(cmd)) { | ||
args.unshift('/S', '/C', cmd.replace(/\&/g, '^&')); | ||
cmd = 'cmd.exe'; | ||
function runTiCommand(cmd, args, logger, callback) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry to knit pick, but you have trailing whitespace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have totally missed that.
@cb1kenobi Updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go. APPROVED!
JIRA: https://jira.appcelerator.org/browse/TIMOB-24912
Change parameters to work both on macOS and Windows.
Description:
On Windows Node child processes need to be run in a new console. This changes the command to point to the node executable and adds the old command as a parameter. This way it should work on both operating systems.