This is the mail archive of the cygwin-apps 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: New Setup for Cygwin 1.7 on

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
> 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 Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

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