CLI: iOS: generated xcode project cannot find "node"

Details

Description

When i generate a titanium mobile project and try to run it it wont build.
It tells me that it cant find "node"
If i change the prebuild command from 

node "/usr/local/bin/titanium" build --platform iphone --sdk 3.1.0 --no-prompt --no-banner --xcode
exit $?

to

/usr/local/bin/node "/usr/local/bin/titanium" build --platform iphone --sdk 3.1.0 --no-prompt --no-banner --xcode
exit $?

Then it works.

Looking at log. In the xcode build process, PATH does not contain /usr/local/bin

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Chris Barber added a comment -

Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3391
3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3392

Show
Chris Barber added a comment - Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3391 3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3392
Hide
Permalink
Martin Guillon added a comment -

i found out today that this fix is not enough.
i found myself in sitations where:
this.cli.argv.$0 = "node"
this.cli.argv.$0 - "/usr/local/bin/node"

So i had to fix it again
https://github.com/Akylas/titanium_mobile/commit/6eb5588b18287db5a52322efc4cb9df5301c9dfe

Show
Martin Guillon added a comment - i found out today that this fix is not enough. i found myself in sitations where: this.cli.argv.$0 = "node" this.cli.argv.$0 - "/usr/local/bin/node" So i had to fix it again https://github.com/Akylas/titanium_mobile/commit/6eb5588b18287db5a52322efc4cb9df5301c9dfe
Hide
Permalink
Chris Barber added a comment -

@Martin, I see what you're saying. No prob, I'll get this fixed in a jiffy.

Show
Chris Barber added a comment - @Martin, I see what you're saying. No prob, I'll get this fixed in a jiffy.
Hide
Permalink
Chris Barber added a comment -

Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3767
3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3768

Comparison of code change:

Before only works when this.cli.argv.$0 is "node /usr/local/bin/titanium".

> 'node /usr/local/bin/titanium'.replace(/^node /, '')
'/usr/local/bin/titanium'

If this.cli.argv.$0 has a full path to node, "node" is not stripped:

> '/usr/local/bin/node /usr/local/bin/titanium'.replace(/^node /, '')
'/usr/local/bin/node /usr/local/bin/titanium'

After the fix, both work perfectly:

> 'node /usr/local/bin/titanium'.replace(/^(.+\/)*node /, '')
'/usr/local/bin/titanium'
> '/usr/local/bin/node /usr/local/bin/titanium'.replace(/^(.+\/)*node /, '')
'/usr/local/bin/titanium'
Show
Chris Barber added a comment - Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3767 3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3768 Comparison of code change: Before only works when this.cli.argv.$0 is "node /usr/local/bin/titanium". > 'node /usr/local/bin/titanium'.replace(/^node /, '') '/usr/local/bin/titanium' If this.cli.argv.$0 has a full path to node, "node" is not stripped: > '/usr/local/bin/node /usr/local/bin/titanium'.replace(/^node /, '') '/usr/local/bin/node /usr/local/bin/titanium' After the fix, both work perfectly: > 'node /usr/local/bin/titanium'.replace(/^(.+\/)*node /, '') '/usr/local/bin/titanium' > '/usr/local/bin/node /usr/local/bin/titanium'.replace(/^(.+\/)*node /, '') '/usr/local/bin/titanium'
Hide
Permalink
Martin Guillon added a comment -

@chris: that's it. thanks

Show
Martin Guillon added a comment - @chris: that's it. thanks

People

  • Assignee:
    Chris Barber
    Reporter:
    Martin Guillon
    Reviewer:
    Bryan Hughes (Inactive)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: