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.

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

Build errro in bignum #28

Closed
ricl opened this issue Apr 29, 2014 · 13 comments
Closed

Build errro in bignum #28

ricl opened this issue Apr 29, 2014 · 13 comments

Comments

@ricl
Copy link

ricl commented Apr 29, 2014

When trying to build this I get an error in bignum as the module does not appear to be included.

..\bignum.cc(10): fatal error C1083: Cannot open include file: 'openssl/bn.h': No such file or directory [C:\U
ard\Documents\GitHub\torrent-stream\node_modules\bittorrent-tracker\node_modules\bignum\build\bignum.vcxproj]

@mafintosh
Copy link
Owner

Are you running this on Windows?

@ricl
Copy link
Author

ricl commented Apr 29, 2014

Yes - Win 8 - built on that as well.

@mafintosh
Copy link
Owner

I'm not sure if fuse works on windows, but bignum is actually an optional dependency should if everything else works it should be ok.

@ricl
Copy link
Author

ricl commented Apr 29, 2014

So is it possible just to ignore the errors given and use anyway - it breaks the latest peerflix because of the errors otherwise

@mafintosh
Copy link
Owner

Oh yeah. Sorry about the fuse thing (I thought I was commenting on another issue on torrent-mount). Yes you can just ignore them. Install still completes successfully right?

@ricl
Copy link
Author

ricl commented Apr 29, 2014

No - it fails though that may be for other reasons. Looks like you may need to move to ferros 0.4.1 to make the bignum optional.

@mafintosh
Copy link
Owner

I've published a workaround in 0.7.5 which uses an older version of the tracker which doesnt have the dependency. Could you try upgrading to that version?

@ricl
Copy link
Author

ricl commented Apr 29, 2014

Yeah. That appear to have fixed it. Peerflix now works as expected as well :-) Thanks.

@feross
Copy link
Collaborator

feross commented Apr 29, 2014

FYI, downgrading means that torrents larger than 4GB will no longer work. Will look into this tonight and see what we can do.

@ricl
Copy link
Author

ricl commented Apr 29, 2014

Thanks. I was actually trying to get a use of peerflix up to the latest versions and this was the first stumbling block I came across. Now I can use up to peerflix 0.5.5 (was 0.5.4 before :-) with this 'fix'.

Something in 6.0 and above breaks how it is being used there.

@mafintosh
Copy link
Owner

@feross oh. I thought it would report wrong stats

@feross
Copy link
Collaborator

feross commented Apr 29, 2014

@mafintosh Unfortunately that version throws an exception when trying to connect to a UDP tracker if the torrent size is > 4GB.

@mafintosh
Copy link
Owner

@feross oh. Let me know if can help. Maybe we could the js version of bignum?

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

No branches or pull requests

3 participants