Skip to content
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

osx 0.39.0 crash when the nwjs helper is changed #7075

Closed
chatcan opened this issue Jun 11, 2019 · 3 comments
Closed

osx 0.39.0 crash when the nwjs helper is changed #7075

chatcan opened this issue Jun 11, 2019 · 3 comments

Comments

@chatcan
Copy link

chatcan commented Jun 11, 2019

NWJS Version : 0.39.0
Operating System : osx

Expected behavior

we can change nwjs helper to any other name

image

Actual behavior

crashed

image

it seems to search for nwjs helper, the picture below shows that a nwjs helper folder will be created once we run the application

image

How to reproduce

@theDevelopper
Copy link

theDevelopper commented Jun 11, 2019

could be the same than this:
#7055
or
#7045

@Blatman
Copy link

Blatman commented Jun 11, 2019

Just checked this myself and renaming is broken with 0.39.0

MacOS High Sierra 10.13.6

Procedure works up to 0.38.5 but broken on 0.39.0

App will crash with trap 3 and leave a partial nwjs Helper package in the 75.0.3770.80 folder next to hippo Helper

In the Activity Monitor hippo process will appear with correct app icon but Helper processes fail to generate. With 0.38.5 the main hippo process will be shown with the correct app icon and a variable number of hippo Helper processes will appear also.

Changes to a typical app -

hippo.app/Contents/Info.plist - CFBundleDisplayName nwjs --> hippo

hippo.app/Contents/MacOS - remains as nwjs (never had to change this before, get can't be opened error if it is renamed even with Info.plist adjusted - docs ????)

Adjust the localisation files -

hippo.app/Contents/Resources/en.lproj/InfoPlist.strings - CFBundleDisplayName nwjs --> hippo

hippo.app/Contents/Resources/en_GB.lproj - CFBundleDisplayName nwjs --> hippo

Adjust the Helper -

hippo.app/Contents/Versions/75.0.3770.80/nwjs Helper/Info.plist - CFBundleDisplayName nwjs Helper --> hippo Helper

hippo.app/Contents/Versions/75.0.3770.80/nwjs Helper/Info.plist - CFBundleExecutable nwjs Helper --> hippo Helper

hippo.app/Contents/Versions/75.0.3770.80/nwjs Helper/Info.plist - CFBundleName nwjs Helper --> hippo Helper

hippo.app/Contents/Versions/75.0.3770.80/nwjs Helper/MacOS/ - nwjs Helper --> hippo Helper

hippo.app/Contents/Versions/75.0.3770.80/ - nwjs Helper --> hippo Helper

Note: for purposes of testing I just copied the app.icns, document.icns and app.nw folder into the Resources folder of the test package and did adjustments as above.

@rogerwang
Copy link
Member

This is fixed in git and will be available in the next nightly build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants