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

gnet v2 supports Windows #339

Closed
amos-lsl opened this issue Mar 9, 2022 · 15 comments · Fixed by #461
Closed

gnet v2 supports Windows #339

amos-lsl opened this issue Mar 9, 2022 · 15 comments · Fixed by #461
Assignees
Labels
pending development Requested PR owner to improve code and waiting for the result proposal accepted Proposal has been accepted proposal Proposal for this repo working on it

Comments

@amos-lsl
Copy link

amos-lsl commented Mar 9, 2022

image

@amos-lsl amos-lsl added help wanted Extra attention is needed question Further information is requested labels Mar 9, 2022
@panjf2000
Copy link
Owner

看起来像是环境配置的问题,go test 能跑吗?

@panjf2000 panjf2000 added the waiting for response waiting for the response from commenter label Mar 9, 2022
@amos-lsl
Copy link
Author

amos-lsl commented Mar 9, 2022

不行,我用的ide是vscode
image

@amos-lsl
Copy link
Author

amos-lsl commented Mar 9, 2022

gnet包中,定义了两个eventloop struct ??
image
image

@panjf2000
Copy link
Owner

你是什么操作系统?

@bwb0101
Copy link

bwb0101 commented Mar 16, 2022

应该是windows系统下的开发

@bagualing
Copy link

一样一样的问题,
Windows 7、vscode下,编译github.com/gnet-io/gnet-examples下的示例程序,提示:
图片

@bwb0101
Copy link

bwb0101 commented Apr 6, 2022

2.0现在不支持win,我是win下用v1,linux下用v2 这样分开的,搞得挺麻烦

@gcggcg
Copy link

gcggcg commented Apr 20, 2022

v2好像不支持win,运行报错一样的问题。大佬有解决方法吗。
image

@panjf2000 panjf2000 changed the title gnet v2.0 这个是什么原因导致的? gnet v2.0 支持 Windows Apr 21, 2022
@panjf2000 panjf2000 added proposal Proposal for this repo pending development Requested PR owner to improve code and waiting for the result proposal accepted Proposal has been accepted and removed help wanted Extra attention is needed question Further information is requested waiting for response waiting for the response from commenter labels Apr 25, 2022
@josejuanmontiel
Copy link

github.com\panjf2000\gnet\v2@v2.0.3\engine.go

//go:build linux || freebsd || dragonfly || darwin
// +build linux freebsd dragonfly darwin

To be notice "this is the problem" :) i'm not expert... by i'll read about it https://www.sobyte.net/post/2021-09/golang-netpoll/ and understand why...

@panjf2000 panjf2000 changed the title gnet v2.0 支持 Windows gnet v2 supports Windows Apr 28, 2022
@ivanjaros
Copy link

ivanjaros commented Aug 26, 2022

oh 💩 i just noticed this is not working on windows. just wasted hour of my time setting up a new project. @panjf2000 you should at least implement workable windows version for basic development. it does not have to provide the same performance, obviously, but it should at least work.

go has built-in epoll implementation for windows so writing a simple wrapper for github.com/panjf2000/gnet/v2/internal/netpoll should not be a problem.

just fyi, windows still dominates not just the internet but developers as well(SO 2021 survey)

obrázok

@ivanjaros
Copy link

I noticed entire code base is windows-free so it's not just the netpoll library. This really sucks.

@maxbad
Copy link

maxbad commented Sep 21, 2022

围观一下, 期待早日支持windows

@maxbad
Copy link

maxbad commented Feb 1, 2023

2023新期待支持一下windows

@gedw99
Copy link

gedw99 commented Mar 14, 2023

+1 windows support for V2

@trungdlp
Copy link

I'm waiting

panjf2000 added a commit that referenced this issue May 9, 2023
panjf2000 added a commit that referenced this issue May 9, 2023
panjf2000 added a commit that referenced this issue May 10, 2023
panjf2000 added a commit that referenced this issue May 10, 2023
panjf2000 added a commit that referenced this issue May 10, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 11, 2023
panjf2000 added a commit that referenced this issue May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending development Requested PR owner to improve code and waiting for the result proposal accepted Proposal has been accepted proposal Proposal for this repo working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

11 participants