This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1


Hi Chuck,

On Aug 16 22:26, Charles Wilson wrote:
> Corinna --
> 
> Attached see
> 
> (1) bugfix for ssh-user-config -- self-explanatory.
> 
> (2) behavior enhancement for ssh-host-config
> 
> Takes advantage of new functions and cleans up behavior under 1.7
>   -- esp. 'mount' doesn't have a -t option anymore, so we need
>      another mechanism to deal with the dos-format
>      WINDOWS/system32/drivers/etc/services file
> 
> Should still work fine under 1.5
> Requires csih-0.1.17 (uses new function csih_cygver_is_oneseven)

I'm about to create a new openssh package for 1.7 with fixes along the
lines of what you sent in your OP.

While I'm at it, I see that there's a bit of dual work in the
csih-0.1.7 script and the ssh-host-config script:

/usr/share/csih/cygwin-service-installation-helper.sh, func _csih_setup:

  csih_make_dir "${LOCALSTATEDIR}/run"
  csih_make_dir "${LOCALSTATEDIR}/log"
  csih_make_dir "${LOCALSTATEDIR}/empty"
  [... checking permissions ...]
  csih_make_dir "${SYSCONFDIR}"

/bin/ssh-host-config:

  csih_make_dir "${SYSCONFDIR}" "Cannot create global configuration files."
  chmod 775 "${SYSCONFDIR}"
  setfacl -m u:system:rwx "${SYSCONFDIR}"

  csih_make_dir "${LOCALSTATEDIR}/log" "Cannot create log directory."
  chmod 1777 "${LOCALSTATEDIR}/log"
  setfacl -m u:system:rwx "${LOCALSTATEDIR}/log"

I would like to propose to do all the work in the csih script, like
this:

  csih_make_dir "${LOCALSTATEDIR}/run"
  chmod 1777 "${LOCALSTATEDIR}/run"
  csih_make_dir "${LOCALSTATEDIR}/log"
  chmod 1777 "${LOCALSTATEDIR}/log"
  csih_make_dir "${LOCALSTATEDIR}/empty"
  chmod 755 "${LOCALSTATEDIR}/empty"
  [... checking permissions ...]
  csih_make_dir "${SYSCONFDIR}"
  chmod 755 "${LOCALSTATEDIR}/etc"

The /var dirs are now (setup 2.609) created with 01777 permissions
anyway, but it can't hurt to keep this code in.  The 01777 permissions
make sure that all daemons can create their files in /var/run, /var/log.

The code in /bin/ssh-host-config would go away entirely.  The script
would also test for the *next* csih version 0.1.8.

Sounds good?


Corinna

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

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]