Always set executability bit for files

John Arbash Meinel john at arbash-meinel.com
Fri Mar 9 00:29:46 GMT 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hey Aaron-

I'm doing some debugging on win32 for handling the executable bit, and
we still have some problems. I think the problem is that whatever
workaround we had stopped working with dirstate.

The attached patch fixes it for me on win32, but I'm wondering if the
fix seems reasonable.

Basically, the problem is that TreeTransform is generating a new
inventory, and when doing so, it is setting InventoryFile.executable
based on what it has in self._new_executability.

We could certainly change it to "if file_status == 'modified' or
sys.platform == 'win32':"

So it would only always set the executability on win32. Certainly it
seems like it is causing us to update every file on disk, which we don't
want to do.

I'm going to keep digging, since I feel like there should be a different
fix. But I wanted to get some feedback on this one.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFF8Kp6JdeBCYSNAAMRAgjxAJ9IK8gk1sEd89ejRfHTmcYKui67TwCeJsP5
Cg1S3LVgrnMJUuwmU1Bfttk=
=fADj
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: executability.diff
Type: text/x-patch
Size: 1143 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070308/243f7636/attachment.bin 


More information about the bazaar mailing list