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: [PATCH cygport 1/3] Pass DEPEND through to .hint for source package as build-depends:


On 05/05/2017 21:42, Yaakov Selkowitz wrote:
On 2017-05-02 08:13, Jon Turney wrote:
Converting a dependency atom to a package name with full generality
requires
a database of all the pathnames contained in all packages, so we don't
even
try to do that here, leaving that task for downstream processing of the
.hint file...

Or should we just replace DEPEND, which allowed atoms which are
unsupported by setup and whose name is confusing wrt REQUIRES, with a
BUILDREQUIRES variable which is package-name only?

Changing the name to something less confusing seems like a good idea.

I kind of like the current scheme, as it can do some checks when cross-compiling as well.

I think that something (calm?) needs to keep a database of all package pathnames, to check for pathname collisions between packages (I suspect we have some unnoticed cases of that at the moment), so the extra cost of doing a conversion from cygport dependency atoms in a .hint to package names in setup.ini shouldn't be too much. It'll be a while before that happens, though... :)

People also like to typo this as DEPENDS, so some sort of warning that the .cygport sets a variable which has no effect might also be useful.


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