These two promises were identical except for the destination directory: "${g.rudder_tools}" (!windows|cygwin) vs "${g.rudder_tools}" (windows.!cygwin).

However, from site.cf:
```
    windows::
      "rudder_tools"               string => "${rudder_sbin}";
```

So this was redundant, and added complexity.