Skip to content

Commit

Permalink
Merge #212
Browse files Browse the repository at this point in the history
212: tgetprotobyname: remove "ip" test query r=alaviss a=krux02

getProtoByName aparently fails on NetBSD. It also fails on my Arch
machine and my Manjaro machine. Removing this test is probably a good
idea.


Co-authored-by: Arne Döring <arne.doering@gmx.net>
  • Loading branch information
bors[bot] and krux02 authored Jan 28, 2022
2 parents 138a842 + 7d5d8f0 commit b7eb17f
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions tests/stdlib/tgetprotobyname.nim
Original file line number Diff line number Diff line change
@@ -1,9 +1,5 @@
import nativesockets

when not defined(netbsd):
# Ref: https://github.com/nim-lang/Nim/issues/15452 - NetBSD doesn't define an `ip` protocol
doAssert getProtoByName("ip") == 0

doAssert getProtoByName("ipv6") == 41
doAssert getProtoByName("tcp") == 6
doAssert getProtoByName("udp") == 17
Expand Down

0 comments on commit b7eb17f

Please sign in to comment.