Sftp_test has errors

[spf13]

./sftp_test.go:203: cannot use debugStream (type io.Writer) as type string in argument to sftp.NewServer
./sftp_test.go:203: cannot use debugLevel (type int) as type sftp.ServerOption in argument to sftp.NewServer
./sftp_test.go:203: cannot use readOnly (type bool) as type sftp.ServerOption in argument to sftp.NewServer
./sftp_test.go:203: cannot use rootpath (type string) as type sftp.ServerOption in argument to sftp.NewServer

I'm not sure if it's just me, but this doesn't look like it should have ever built. If I'm doing something wrong, please let me know.

[xor-gate]

It did work, those guys started to break the master branch exactly 6 days ago:

pkg/sftp@8a7343c

https://github.com/pkg/sftp/commits/master

Maybe we should blame them 👍

Probably we should upgrade to the new sftp.ServerOption and then we are good to go.

In the (near) future the whole sftp package will be refactored: pkg/sftp#84

[spf13]

Doh. We need to vendor these deps.
On Sun, Jan 17, 2016 at 4:25 PM Jerry Jacobs notifications@github.com
wrote:

It did work, those guys started to break the master branch exactly 6 days
ago:

https://github.com/pkg/sftp/commits/master

[image: screen shot 2016-01-17 at 22 24 59]
https://cloud.githubusercontent.com/assets/1050166/12380059/417b56c0-bd69-11e5-9d9a-c6c3dc746536.png

—
Reply to this email directly or view it on GitHub
#59 (comment).

[0xmichalis]

Fixed by #157