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

Added documentation for future build of the bindings in Windows #164

Merged
merged 1 commit into from
Oct 3, 2013
Merged

Added documentation for future build of the bindings in Windows #164

merged 1 commit into from
Oct 3, 2013

Conversation

LaurentGoderre
Copy link
Contributor

This is for anyone who needs to recompile the bindings for Windows in the future. Wouldn't want to waste their time trying to figure it out too.

@andrew
Copy link
Contributor

andrew commented Oct 3, 2013

Brilliant, thanks

andrew added a commit that referenced this pull request Oct 3, 2013
Added documentation for future build of the bindings in Windows
@andrew andrew merged commit 805ef4f into sass:master Oct 3, 2013
@nschonni
Copy link
Contributor

nschonni commented Oct 3, 2013

👍 How should this be dealt with for #148?
On Windows should it run node-gyp configure first and then strip the line with a regex?

@nschonni
Copy link
Contributor

nschonni commented Oct 3, 2013

Might be good to link to the upstream node-gyp issue so people can check if it's resolved

@LaurentGoderre
Copy link
Contributor Author

@nschonni already did:

nodejs/node-gyp#335

@nschonni
Copy link
Contributor

nschonni commented Oct 3, 2013

Yup I saw that, I was talking about in the README files though.

@LaurentGoderre
Copy link
Contributor Author

Oooh, we could, you're right.

@LaurentGoderre
Copy link
Contributor Author

See #165

@andrew
Copy link
Contributor

andrew commented Oct 23, 2013

Just published this as part of v0.7.0: https://github.com/andrew/node-sass/releases/tag/v0.7.0

jamesw-minted pushed a commit to jamesw-minted/node-sass that referenced this pull request May 11, 2017
changes spelling of Sass in README
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this pull request Apr 7, 2024
Updating autogen.sh script.
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.

3 participants