Skip to content

Conversation

@VlkrS
Copy link
Contributor

@VlkrS VlkrS commented May 8, 2024

As described in #753 the change merged with #659 is ill-advised and leads to breakage in applications using c-ares on OpenBSD.

See also nodejs/node#52439

As described in c-ares#753 the change merged with c-ares#659 is ill-advised and leads to breakage in applications using c-ares on OpenBSD. 

See also nodejs/node#52439
@bradh352
Copy link
Member

bradh352 commented May 8, 2024

Very interesting, I didn't evaluate SOCK_DNS when I accepted the original PR other than to check the relevant manpage that was linked in the PR which made no mention about its real use case or restrictions on its use.

It sounds like its an extension that is only allowed to be used by libc and not external 3rd parties? Odd.

Its unfortunate Theo replied in such a manner, however. As a maintainer looking in from the outside, I'm not sure how I could have caught this not being an OpenBSD user myself when the documentation it woefully incomplete. Also, the fact that no CI system seems to natively support OpenBSD means testing of this can't be automated.

@bradh352 bradh352 merged commit daf00b9 into c-ares:main May 8, 2024
@bradh352 bradh352 mentioned this pull request May 8, 2024
bradh352 pushed a commit that referenced this pull request May 8, 2024
As described in #753 the change
merged with #659 is ill-advised and
leads to breakage in applications using c-ares on OpenBSD.

See also nodejs/node#52439
bradh352 pushed a commit that referenced this pull request May 8, 2024
As described in #753 the change
merged with #659 is ill-advised and
leads to breakage in applications using c-ares on OpenBSD.

See also nodejs/node#52439
bradh352 pushed a commit that referenced this pull request May 8, 2024
As described in #753 the change
merged with #659 is ill-advised and
leads to breakage in applications using c-ares on OpenBSD.

See also nodejs/node#52439
bradh352 pushed a commit that referenced this pull request May 8, 2024
As described in #753 the change
merged with #659 is ill-advised and
leads to breakage in applications using c-ares on OpenBSD.

See also nodejs/node#52439
bradh352 added a commit that referenced this pull request May 8, 2024
Github actions appears to support building on a few "unsupported"
platforms these days:
https://github.com/cross-platform-actions/action

Lets go ahead and add OpenBSD to try to prevent issues like #754 

Fix By: Brad House (@bradh352)
@VlkrS
Copy link
Contributor Author

VlkrS commented May 8, 2024

Its unfortunate Theo replied in such a manner, however. As a maintainer looking in from the outside, I'm not sure how I could have caught this not being an OpenBSD user myself

Thanks for merging this so quickly.
I can't speak for anyone else, but as far as I am concerned, I see both your point and how this appeared to be a good idea to the person making the PR.

@VlkrS VlkrS deleted the fix-openbsd branch May 8, 2024 11:39
@catap
Copy link

catap commented May 8, 2024

Also, the fact that no CI system seems to natively support OpenBSD means testing of this can't be automated.

You may use https://github.com/vmactions/openbsd-vm to make CI for OpenBSD

@bradh352
Copy link
Member

bradh352 commented May 8, 2024

i actually found an alternative with cross-platform-actions and added minimal CI for OpenBSD and NetBSD

@catap
Copy link

catap commented May 8, 2024

i actually found an alternative with cross-platform-actions and added minimal CI for OpenBSD and NetBSD

I'm quite interesting in alternative solution. May you share the link? :)

@bradh352
Copy link
Member

bradh352 commented May 8, 2024 via email

@catap
Copy link

catap commented May 9, 2024 via email

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.

3 participants