coreutils: install.exe.manifest file needs execute permissions

coreutils: install.exe.manifest file needs execute permissions

[Corinna Vinschen]

Hi,

I'm just hacking setup.exe so that the permission info in the tar
archives is used to set real POSIX permissions on the extracted files.
The permission settings already look good, however, while testing, I
started bash from a fresh install and got these messages:

  Copying skeleton files.
  These files are for the user to personalise
  their cygwin experience.

  These will never be overwritten.

  bash: /usr/bin/install: Permission denied
  bash: /usr/bin/install: Permission denied
  bash: /usr/bin/install: Permission denied

The problem is that the /usr/bin/install.exe.manifest file has no
execute permissions:

  $ ls -l /usr/bin/install.exe*
  -rwxr-xr-x 1 corinna Users 146432 Jun  2 17:10 /usr/bin/install.exe
  -rw-r--r-- 1 corinna Users    570 Jun  2 17:08 /usr/bin/install.exe.manifest

  $ /usr/bin/install
  bash: /usr/bin/install: Permission denied

  $ chmod +x install.exe.manifest
  $ /usr/bin/install
  /usr/bin/install: missing file operand
  [...]
    
For the executables with manifest files to work correctly, the manifest
files *must* be executable.  This is usually not a problem with the old
setup for 1.5, but if we want real POSIX perms with setup for 1.7, the
manifest files must be stored in the tar files with execute permissions
set.

This affects

- coreutils (install.exe.manifest)
- patch     (patch.exe.manifest)
- texinfo   (install-info.exe.manifest)

Patch and texinfo both come with manifest files which have execute
permissions set, just coreutils needs to be tweaked.  Eric, would you
mind to upload a new coreutils for 1.7?


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

Re: coreutils: install.exe.manifest file needs execute permissions

[Eric Blake]

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

According to Corinna Vinschen on 8/19/2008 1:47 PM:
> 
> The problem is that the /usr/bin/install.exe.manifest file has no
> execute permissions:

I'm on it - thanks for the hint.  Look for a new coreutils on release-2 by
tomorrow (I'm working on git 1.6.0 first, though).

- --
Don't work too hard, make some time for fun as well!

Eric Blake             ebb9@byu.net
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkirdXMACgkQ84KuGfSFAYA/MACgycGN32ez3KYX2Z4Vh0BICBlz
Qp4Anj4j0CXIfvQNRX/VbqUxluqvR7bN
=MQVi
-----END PGP SIGNATURE-----