Skip to content

Add getHostName - #621

Merged
kazu-yamamoto merged 1 commit into
haskell:masterfrom
pmiddend:add-hostname
Aug 18, 2026
Merged

Add getHostName#621
kazu-yamamoto merged 1 commit into
haskell:masterfrom
pmiddend:add-hostname

Conversation

@pmiddend

Copy link
Copy Markdown
Contributor

Just a first draft of the getHostName feature taken from the hostname library with permission of the author.

@kazu-yamamoto

Copy link
Copy Markdown
Collaborator

I ran CI. I will merge this if it passes.

@pmiddend

pmiddend commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

I ran CI. I will merge this if it passes.

I'm currently building this on Windows and am getting a warning pertaining the use of stdcall on Win64. I am fixing this.

@pmiddend

Copy link
Copy Markdown
Contributor Author

I ran CI. I will merge this if it passes.

I'm currently building this on Windows and am getting a warning pertaining the use of stdcall on Win64. I am fixing this.

Fixed! It builds fine on Windows and Linux it seems. Haven't tested OSX, we have to wait for CI to check that one. Thanks for the fast reply!

@pmiddend
pmiddend marked this pull request as ready for review August 18, 2026 07:37
@pmiddend

Copy link
Copy Markdown
Contributor Author

I guess we also need a "since" marker and a changelog entry?

@kazu-yamamoto

Copy link
Copy Markdown
Collaborator

I guess we also need a "since" marker and a changelog entry?

Right!

@pmiddend

Copy link
Copy Markdown
Contributor Author

I guess we also need a "since" marker and a changelog entry?

Right!

Added it!

@kazu-yamamoto kazu-yamamoto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All green. LGTM

@kazu-yamamoto
kazu-yamamoto merged commit 63914c9 into haskell:master Aug 18, 2026
19 checks passed
@kazu-yamamoto

Copy link
Copy Markdown
Collaborator

It has been merged.
Thank you.
I'm currently confirming the WinIO merge, so the release might be delayed.

@pmiddend

Copy link
Copy Markdown
Contributor Author

It has been merged. Thank you. I'm currently confirming the WinIO merge, so the release might be delayed.

No problem, there's no urgency. Thanks for the quick review!

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.

2 participants