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

fake netstat for WSL1 #4159

Closed
wants to merge 2 commits into from
Closed

fake netstat for WSL1 #4159

wants to merge 2 commits into from

Conversation

qyb
Copy link
Contributor

@qyb qyb commented Jul 1, 2022

Signed-off-by: qyb [email protected]

Fixes #4158

Signed-off-by: qyb <[email protected]>
@elliefm elliefm changed the title fix https://github.com/cyrusimap/cyrus-imapd/issues/4158 fake netstat for WSL1 Jul 1, 2022
@elliefm elliefm added the cassandane issues affecting specifically the cassandane test suite label Jul 1, 2022
@elliefm
Copy link
Contributor

elliefm commented Jul 1, 2022

[I've updated the original post to have a descriptive title, and to reference the related issue from the description rather than the title, so that GitHub can link the two together automatically]

Copy link
Contributor

@elliefm elliefm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if there's some way we can detect the path of netstat.exe, instead of it being hardcoded?

@qyb
Copy link
Contributor Author

qyb commented Jul 1, 2022

I wonder if there's some way we can detect the path of netstat.exe, instead of it being hardcoded?

Microsoft appends PATH variable in a magic place that I can't find. sudo to cyrus will lose the path. Maybe I could build the fakenetstat script when make in cassandane. Let me think twice. 🤔

Signed-off-by: qyb <[email protected]>

generate fakenetstat script in testrunner.pl

Signed-off-by: qyb <[email protected]>
@qyb qyb mentioned this pull request Jul 2, 2022
@qyb qyb closed this Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cassandane issues affecting specifically the cassandane test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

run testrunner.pl under WSL1
2 participants