Skip to content

On Windows, copy required DLLs next to ".node" file#442

Merged
TooTallNate merged 1 commit intoAutomattic:masterfrom
pandell:copy-dlls
Jun 26, 2014
Merged

On Windows, copy required DLLs next to ".node" file#442
TooTallNate merged 1 commit intoAutomattic:masterfrom
pandell:copy-dlls

Conversation

@milang
Copy link
Contributor

@milang milang commented Jun 26, 2014

This is a pull request on top of #441, but could be rebased and used independently.

It adds a postbuild target that will copy required DLLs from (GTK_Root)/bin to target directory so that canvas.node file can be loaded and used without requiring (GTK_Root)/bin to be in PATH.

@TooTallNate
Copy link
Contributor

Great idea!

@milang
Copy link
Contributor Author

milang commented Jun 26, 2014

Rebased on top of merged #441.

@101100
Copy link

101100 commented Jun 26, 2014

I was able to run several of the examples on my Windows 8.0 machine without adding the GTK DLLs to my path. Without the copied DLLs, the same examples fail. 🍪

TooTallNate added a commit that referenced this pull request Jun 26, 2014
On Windows, copy required DLLs next to ".node" file
@TooTallNate TooTallNate merged commit 2d1d3e3 into Automattic:master Jun 26, 2014
@TooTallNate
Copy link
Contributor

Thanks!

@milang
Copy link
Contributor Author

milang commented Jun 26, 2014

@TooTallNate thank you for merging. Any chance you might publish the updates to npm?

@TooTallNate
Copy link
Contributor

@milang Doesn't look like I have npm publish rights.

@visionmedia @kangax Can one of you cd node-canvas && npm owner add tootallnate please?

@milang milang deleted the copy-dlls branch June 26, 2014 22:39
@kangax
Copy link
Collaborator

kangax commented Jun 26, 2014

@TooTallNate sure, try now

@TooTallNate
Copy link
Contributor

Ok, v1.1.5 published to npm. Cheers!

On Thu, Jun 26, 2014 at 3:57 PM, Juriy Zaytsev notifications@github.com
wrote:

@TooTallNate https://github.com/TooTallNate sure, try now


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants