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

Warning C4005: 'snprintf' #110

Open
renancarrillo opened this issue Nov 18, 2015 · 1 comment
Open

Warning C4005: 'snprintf' #110

renancarrillo opened this issue Nov 18, 2015 · 1 comment

Comments

@renancarrillo
Copy link

I'm having error trying to install imagemagick.

When I try to install with npm install imagemagick-native throw me 'Warning C4005: 'snprintf'

I don't now what's going on, I all ready install:

*Visual Studio C++ 2010 Express
*Windows 7 64-bit SDK
*Imagemagick dll binary x64 Q16

I'm in windows 10.
@elad
Copy link
Owner

elad commented Dec 8, 2015

Hi,

I don't have a Windows machine to test with, so this will have to wait for someone else...

eirslett added a commit to eirslett/node-imagemagick-native that referenced this issue Mar 20, 2016
@eirslett
Fix build on Windows (Visual Studio 2015)
68b8ff3 
This solves elad#110.

It's similar to this issue on node-canvas: Automattic/node-canvas#605

Basically, Visual Studio didn't have snprintf, so it had to be polyfilled, like it has been done here. But now, in 2015, snprintf was finally implemented, which leads to it being defined twice.
This change lets me compile the project on Windows 7 x64, with Visual Studio 2015.
@eirslett eirslett mentioned this issue Mar 20, 2016
Merged
eirslett added a commit to eirslett/node-imagemagick-native that referenced this issue Mar 21, 2016
@eirslett
Fix build on Windows (Visual Studio 2015)
6fec2ec 
This solves elad#110.

It's similar to this issue on node-canvas: Automattic/node-canvas#605

Basically, Visual Studio didn't have snprintf, so it had to be polyfilled, like it has been done here. But now, in 2015, snprintf was finally implemented, which leads to it being defined twice.
This change lets me compile the project on Windows 7 x64, with Visual Studio 2015.
horiuchi pushed a commit to horiuchi/node-imagemagick-native that referenced this issue May 19, 2016
@eirslett @horiuchi
Fix build on Windows (Visual Studio 2015)
a99def5 
This solves elad#110.

It's similar to this issue on node-canvas: Automattic/node-canvas#605

Basically, Visual Studio didn't have snprintf, so it had to be polyfilled, like it has been done here. But now, in 2015, snprintf was finally implemented, which leads to it being defined twice.
This change lets me compile the project on Windows 7 x64, with Visual Studio 2015.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@elad @renancarrillo