Always set executability bit for files

[John Arbash Meinel]

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

John Arbash Meinel wrote:
> 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
> =:->

Ignore this, I found the problem. TT deals in inventories, and
WorkingTree4._generate_inventory() was ignoring the executable bit when
generating an inventory for the working tree.

Sorry for the noise.

John
=:->

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

iD8DBQFF8K3BJdeBCYSNAAMRAlsJAJ9Y9XnTThfHdxif0tN7SJNAKkPflwCgm8EW
62OYs1WRQp/Qsuwykxktm6U=
=ucrp
-----END PGP SIGNATURE-----