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

Working on Windows 7 #179

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Working on Windows 7 #179

wants to merge 2 commits into from

Conversation

jimjibone
Copy link
Collaborator

Made changes to binding.gyp to include the various files on Windows/Linux/etc and edited some C++ source to allow compilation on Windows.

Edited the README to cover a few build instructions for Windows.

Added a PCAP file reading example.

@mranney
Copy link
Collaborator

mranney commented Jan 11, 2016

Hi there. Thanks for the contribution. I've added you as a collaborator on this project. Feel free to merge as you see fit. Getting proper Windows support would be great.

@Segflow
Copy link

Segflow commented Feb 6, 2016

@jimjibone can you resolve the conflict please?

@k1epto
Copy link

k1epto commented Jun 10, 2019

CAN WE PLEASE GET THIS FIXED?!

@roccomuso
Copy link
Collaborator

@mranney can you add @mildsunrise as collaborator too?

@mildsunrise
Copy link
Contributor

@bipolarmorgan ummm, there's not an easy fix

To get this module working on Windows I'm afraid more work is needed, see #249 (comment)

@PatrickSantoZZ
Copy link

is this still not fixed?

@mildsunrise
Copy link
Contributor

@PatrickSantoZZ it's not a fix. It's a new feature. We'd need to write an important amount of code to adapt to Windows.

@PatrickSantoZZ
Copy link

@PatrickSantoZZ it's not a fix. It's a new feature. We'd need to write an important amount of code to adapt to Windows.

nice to know. thank you :3 👍

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.

7 participants