Skip to content

fix build on OpenBSD#68

Merged
bagder merged 1 commit intoc-ares:masterfrom
qbit:master
Oct 22, 2016
Merged

fix build on OpenBSD#68
bagder merged 1 commit intoc-ares:masterfrom
qbit:master

Conversation

@qbit
Copy link
Copy Markdown
Contributor

@qbit qbit commented Oct 22, 2016

From the OpenBSD getservent(3) man page:

     getservbyport_r(int port, const char *proto, struct servent *servent,
         struct servent_data *servent_data);

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.08%) to 95.5% when pulling 4aed24e on qbit:master into 2f467d2 on c-ares:master.

@bagder bagder merged commit b0af263 into c-ares:master Oct 22, 2016
@bagder
Copy link
Copy Markdown
Member

bagder commented Oct 22, 2016

thanks!

jasnell pushed a commit to nodejs/node that referenced this pull request Oct 28, 2016
Original PR: c-ares/c-ares#68

PR-URL: #9232
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
evanlucas pushed a commit to nodejs/node that referenced this pull request Nov 3, 2016
Original PR: c-ares/c-ares#68

PR-URL: #9232
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Nov 18, 2016
Original PR: c-ares/c-ares#68

PR-URL: #9232
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Nov 18, 2016
Original PR: c-ares/c-ares#68

PR-URL: #9232
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Nov 19, 2016
Original PR: c-ares/c-ares#68

PR-URL: #9232
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
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