Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Create a troubleshooting guide #921

Merged
merged 3 commits into from
May 3, 2015
Merged

Conversation

xzyfer
Copy link
Contributor

@xzyfer xzyfer commented May 3, 2015

This PR introduces a basic troubleshooting guide for common issues.

This will give us a common places to link people to from issues, and single point of reference for debugging information.

I don't by any means expect this to be perfect right off the bat, but we need start somewhere.

Fixes #916.

/cc @am11 @saper

@xzyfer xzyfer self-assigned this May 3, 2015
@xzyfer xzyfer added this to the 3.0 milestone May 3, 2015
@xzyfer xzyfer force-pushed the feat/docs/troubleshooting branch 3 times, most recently from 6679eb3 to 597dbc8 Compare May 3, 2015 06:38
@xzyfer xzyfer force-pushed the feat/docs/troubleshooting branch from 597dbc8 to 813b99f Compare May 3, 2015 06:39
@xzyfer xzyfer mentioned this pull request May 3, 2015
19 tasks
xzyfer added a commit that referenced this pull request May 3, 2015
@xzyfer xzyfer merged commit f2df3d9 into sass:master May 3, 2015
@am11
Copy link
Contributor

am11 commented May 3, 2015

👍

Note that @piscisaureus particularly tested the delay-load hook solution with node-sass nodejs/node#1251 (comment) and I can confirm with that io.js process works with node prefix and there is no module did not self-register error with node-sass anymore; given we build node-sass binary for Windows with the distilled node-gyp dependency under io.js repo.

With that said: to turn on delay-load-hook, use:

{
  'targets': [
    {
      'target_name': 'binding',
      'win_delay_load_hook': 'true',
      ...

in binding.gyp.

Also, the same solution is merged in original node-gyp repo: nodejs/node-gyp#599, but the real caveat is that node-gyp is yet to bring io.js support: nodejs/node-gyp#564.

@xzyfer
Copy link
Contributor Author

xzyfer commented May 3, 2015

I admit to only having a passing familiarity with the issue. Would
appreciate a PR to clarify the doc.
On 3 May 2015 18:22, "Adeel Mujahid" notifications@github.com wrote:

[image: 👍]

Note that @piscisaureus https://github.com/piscisaureus particularly
tested the delay-load hook solution with node-sass nodejs/node#1251
(comment) nodejs/node#1251 (comment)
and I can confirm with that io.js process works with node prefix and
there is no module did not self-register error with node-sass anymore;
given we build node-sass binary for Windows with the distilled node-gyp
dependency
https://github.com/iojs/io.js/tree/master/deps/npm/node_modules/node-gyp
under io.js repo.

With that said: to turn on delay-load-hook, use:

{
'targets': [
{
'target_name': 'binding',
'win_delay_load_hook': 'true',
...

in binding.gyp.

Also, the same solution is merged in original node-gyp repo:
nodejs/node-gyp#599
nodejs/node-gyp#599, but the real caveat
is that node-gyp is yet to bring io.js support: nodejs/node-gyp#564
nodejs/node-gyp#564.


Reply to this email directly or view it on GitHub
#921 (comment).

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

Successfully merging this pull request may close these issues.

Create a troubleshooting guide
2 participants