papertrail / remote_syslog2

To install, see "Releases" tab. Self-contained daemon for reading local files and emitting remote syslog (without using local syslogd).
http://help.papertrailapp.com/
MIT License
637 stars 157 forks source link

syslog does not pass tests under go 1.11.5 so the build shows failed #223

Closed ckelloug2 closed 5 years ago

ckelloug2 commented 5 years ago

There are two errors related to formatting output errors that keep the syslog subpackage from building and passing tests. The first is in syslog_test.go, the second is in packet_test.go. I'm including the patch I used to get it to build and pass tests.

diff --git a/syslog/packet_test.go b/syslog/packet_test.go index ef3e419..6209846 100644 --- a/syslog/packet_test.go +++ b/syslog/packettest.go @@ -17,7 +17,7 @@ func TestPacketPriority(t *testing.T) { for , test := range tests { p := Packet{Severity: test.severity, Facility: test.facility} if result := p.Priority(); result != test.priority {

-func panicf(s string, i ...interface{}) { panic(fmt.Sprintf(s, i)) } +func panicf(s string, i ...interface{}) { panic(fmt.Sprintf(s, i...)) }

type testServer struct { Addr string

cce commented 5 years ago

Thanks! Fixed in #225.