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

build: Adds win_delay_hook so iojs runs with alias #926

Merged
merged 1 commit into from
May 6, 2015

Conversation

am11
Copy link
Contributor

@am11 am11 commented May 6, 2015

Before building, first apply the following patch
to pangyp: rvagg/archived-pangyp#5 (until it is merged).

Issue URL: #870.

@xzyfer
Copy link
Contributor

xzyfer commented May 6, 2015

Hey @am11, do we need to ship this for 3.0.0?

Before building, first apply the following patch
to pangyp: rvagg/archived-pangyp#5 (until it is merged).

Issue URL: sass#870.
@am11
Copy link
Contributor Author

am11 commented May 6, 2015

I think yes as this change has no side effect.
I have changed the pangyp version to my master, until they merge that PR.

@am11
Copy link
Contributor Author

am11 commented May 6, 2015

This actually fixes that module registration bug on Windows (so it has +ve effect). :)

@xzyfer
Copy link
Contributor

xzyfer commented May 6, 2015

Argh alright. I guess I can revoke the tag since we haven't created the release :) /cc @saper if you figure out what's up with the FreeBSD watcher we can ship it in 3.0.0.

@saper
Copy link
Member

saper commented May 6, 2015

This change has no effect for us until we inherit objects from v8 or something like that, right?

@xzyfer
Copy link
Contributor

xzyfer commented May 6, 2015

I believe it's related to node-sass install script failing with windows on iojs

@xzyfer
Copy link
Contributor

xzyfer commented May 6, 2015

This doesn't affect the 3.0.0 node-sass-binaries so I'll ship this and re-tag v3.0.0.

xzyfer added a commit that referenced this pull request May 6, 2015
build: Adds win_delay_hook so iojs runs with alias
@xzyfer xzyfer merged commit b3ce14d into sass:master May 6, 2015
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this pull request Apr 7, 2024
Fix parsing and output of unknown at-rules (Fixes sass#855)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants