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.

Sign up for GitHub

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

Jump to bottom

[windows] exe bails out? #10

Closed
imlucas opened this issue May 14, 2014 · 3 comments
Closed

[windows] exe bails out? #10

imlucas opened this issue May 14, 2014 · 3 comments
Labels
bug

Comments

@imlucas
Copy link
Owner

imlucas commented May 14, 2014

can build the exe. app runs just fine on windows 8. exe starts, get the first few lines of express init debugging and then just bails :/
@imlucas imlucas added the bug label May 14, 2014
imlucas added a commit that referenced this issue May 14, 2014
@imlucas
[#10] adding debugging
b2b6a32
@imlucas
Copy link
Owner Author

imlucas commented May 14, 2014

smells like a straight up segfault... current theory: need to re-install native add-ons with the freshly compiled node, as the node.lib mismatch causes all hell to break loose.

imlucas added a commit that referenced this issue May 14, 2014
@imlucas
[#10} hot on the trail
00abf57
@imlucas
Copy link
Owner Author

imlucas commented May 14, 2014

confirmed this is being caused by calling dlopen https://github.com/joyent/node/blob/master/src/node.cc#L2045 confused as to why it doesnt actually throw though :/

imlucas added a commit that referenced this issue May 16, 2014
@imlucas
[#10] punt. apply patches to get around win32 dlopen segfault
446e649
imlucas added a commit that referenced this issue May 16, 2014
@imlucas
[#10] clean out logging
4657d6c
imlucas added a commit that referenced this issue May 16, 2014
@imlucas
[#10] i hate diff make patch.
760b3ee
imlucas added a commit that referenced this issue May 16, 2014
@imlucas
[#10] rewrite windows line endings for patches
37a413f 
see line 2061 of diff-make-patch/index.js
imlucas added a commit that referenced this issue May 16, 2014
@imlucas
[#10] dont try and patch a file that doesnt exist
a77ac9b
@imlucas
Copy link
Owner Author

imlucas commented May 16, 2014

punting on this one for now in hopes of a magical unicorn in node 0.12.0 or @mapbox/node-pre-gyp. feel free to reopen if/when you need native add-ons for windows.

@imlucas imlucas closed this as completed May 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@imlucas