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 cygwin.com


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.

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+)(.*)


--
Chuck


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