This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: New Setup for Cygwin 1.7 on cygwin.com
On Aug 21 07:08, Charles Wilson wrote:
> Corinna Vinschen wrote:
>
> > bash <-> _update_info_dir
> > texinfo <-> _update_info_dir
> >
> > All other dependencies are looking sound from what I can tell.
> >
> > So, I would like to propose to remove the "requires: bash texinfo" line
> > from _update_info_dir's setup.hint file. It appears that these
> > requirements break the whole ordering. Maybe we should simply
> > replace it with "requires: cygwin" instead.
>
> Seems ok to me, but we'd have to put texinfo into Base, in that case.
> Otherwise it won't get installed automatically, and _update_info_dir
> would fail on a Base-only install.
Good hint.
> Aside: since setup.hint allows comments (which do not clutter
> setup.ini), we should probably annotate setup.hints whose requires: have
> been...manipulated...in odd ways.
>
> e.g.
> ===== _update_info_dir/setup.hint =====
> sdesc: "Generate info/dir file automatically"
> category: _PostInstallLast
> requires: cygwin
> # also requires texinfo bash, but
> # a) both are in Base, so we rely on that
> # b) we don't explicitly require them because that introduces
> # a bad dependency loop with regards to postinstall scripts
> autodep: usr/(?:share/)?info/.*
> incver_ifdep: yes
> # verpat: (_update_info)(dir-\d+-\d+)(.*)
Yes, that probably makes sense.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat