can't build app for windows if the app uses netconf driver

[idqdq]

Hi Carl
this is the follow-up issue I've described in the issue 54

the next issue I ran into - I can't build my app for windows platform
to be honest I've never tried it before so maybe I'm doing something wrong

this is what i was trying to do and the result

> GOOS=windows GOARCH=amd64 go build clear_ipsec.go
# github.com/scrapli/scrapligo/driver/base
../../driver/base/new_windows.go:43:22: d.TimeoutSocket undefined (type *Driver has no field or method TimeoutSocket)
../../driver/base/new_windows.go:44:22: d.TimeoutTransport undefined (type *Driver has no field or method TimeoutTransport)

my code

       d, _ := netconf.NewNetconfDriver(
		"192.168.99.94",
		base.WithAuthStrictKey(false),
		base.WithAuthUsername("clear_ipsec"),
		base.WithAuthPassword("strongpassword"),
		base.WithPort(22),
		//base.WithTimeoutTransport(20),
		//base.WithTimeoutSocket(20),
		base.WithTransportType(transport.StandardTransportName),
	)

	d.Open()

as you can see I've tried to play with Timeouts (commented out) but with no success

[carlmontanari]

👋

Hey Nikolay!

I'll respond to the other issue probably this weekend, but this one was easy enough 😁

You're lucky this was an easy one since I don't care at all about windows hahah. Anyway, just pushed a commit to main branch that should sort it out. Basically was just some cruft that was left over from overhauling some of the transport stuff a few months back. Since I don't care about/test/support windows (and never will!!!!! 🤣 ) it got missed.

Let us know if this commit sorts it out. Basically just setting the defaults instead of assigning from driver object -- then the options get applied toward the end of that function.

Carl

[idqdq]

Hi Carl
Yes it does work now
Thx

p.s I like your attitude to windows :)

[carlmontanari]

Awesome, thanks for confirming!