Code review comment for lp:~parthm/bzr/262450

Revision history for this message
Martin Pool (mbp) wrote :

So probably the permissions stuff just can't work like this on Windows, in which case we should probably:

1- make that test requireFeature something unixy; probably it would be good to declare a new Feature for unix permissions and just say it exists if os.platform=posix
2- check the actual runtime code passes on windows (even if it does nothing)
3- file a followon bug about inheriting acls on windows

« Back to merge proposal