I think instead of assigning to windir, we should be appending to it. ie:

```ruby
windir << contents.gsub("\r\n", '')
```

I think the idea with the `execute` blocks is that the data can arrive in chunks.  Although the result of this call is really short and will probably always work, we might as well follow the general pattern.

Check out how it's used in the [vagrant core code](https://github.com/mitchellh/vagrant/blob/5e48391b05ce3ff864547497502bcff73e1227af/plugins/guests/linux/cap/read_ip_address.rb).