-
Notifications
You must be signed in to change notification settings - Fork 43
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
修正 windows build fail 問題 #68
修正 windows build fail 問題 #68
Conversation
cache/systemvshm_windows.go
Outdated
@@ -22,3 +26,7 @@ func shmdt(shmaddr uintptr) (int, error) { | |||
func shmctl(shmid int, cmd int, buf *ShmidDs) (int, error) { | |||
return 0, fmt.Errorf("windows do not implement shmctl") | |||
} | |||
|
|||
func ShmSegsz(shmid int, cmd int, buf *ShmidDs) (int, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
這個應該可以不用加?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
對,我改一下
cache/mmap_windows.go
Outdated
) | ||
|
||
func openMmap(fd int, size int) ([]byte, error) { | ||
return nil, fmt.Errorf("windows do not implement shmget") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return nil, fmt.Errorf("windows do not implement shmget") | |
return nil, fmt.Errorf("TODO: implement open mmap") |
cache/mmap_windows.go
Outdated
} | ||
|
||
func closeMmap(buf []byte) error { | ||
return fmt.Errorf("windows do not implement shmget") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return fmt.Errorf("windows do not implement shmget") | |
return fmt.Errorf("TODO: implement close mmap") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
👏 解決掉的 issue / Resolved Issues
📝 相關的 issue / Related Issues
無
⛏ 變更內容 / Details of Changes
增加了 windows build 的支援。