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

"Problem with the binary" message should log the exception #1161

Closed
saper opened this issue Sep 18, 2015 · 0 comments
Closed

"Problem with the binary" message should log the exception #1161

saper opened this issue Sep 18, 2015 · 0 comments
Assignees
@xzyfer
Labels
npm install

Comments

@saper
Copy link
Member

saper commented Sep 18, 2015

In few cases (missing symbol) we won't even get that far to print that message, but if we do, use console logging to print the exception.
@saper saper added this to the next.patch milestone Sep 18, 2015
@saper saper changed the title "Problem with the binary" message shoud log the exception "Problem with the binary" message should log the exception Sep 24, 2015
xzyfer added a commit to xzyfer/node-sass that referenced this issue Apr 19, 2016
@xzyfer
Log the error when there is a problem with the binary
211f312 
This is a messy first cut. We can revisit this in upcomming patches.
The value this will bring to debugging installation errors makes
this worth considered we're about to release a LibSass bump.

Fixes sass#1161
@xzyfer xzyfer self-assigned this Apr 19, 2016
@xzyfer xzyfer mentioned this issue Apr 19, 2016
Merged
@xzyfer xzyfer modified the milestone: next.patch Sep 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xzyfer xzyfer

Labels
npm install
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@saper @xzyfer