pkgsrc-Users archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: x11/motif fails to compile due to library not found for -lintl

On 8/22/14, 8:48 PM, J. Lewis Muir wrote:
> The patch adds a build-dependency include in devel/flex/
> for devel/gettext-lib/ which causes the wrapper phase of
> x11/motif to create the needed symlinks, and then x11/motif builds
> successfully.

The patch works, but I'm not exactly confident it is correct.  It adds
the following two lines to devel/flex/

BUILDLINK_DEPMETHOD.gettext?=   build
.include "../../devel/gettext-lib/"

But what if BUILDLINK_DEPMETHOD.flex is set to "full"
by whatever includes devel/flex/  That
would mean BUILDLINK_DEPMETHOD.flex would be "full", but
BUILDLINK_DEPMETHOD.gettext would be "build" (assuming it has not been
set yet).  So maybe BUILDLINK_DEPMETHOD.gettext should always match
BUILDLINK_DEPMETHOD.flex?  In that case, the two lines added by the
patch should be changed to something like this instead:

.include "../../devel/gettext-lib/"

But this all makes me question the "BUILDLINK_DEPMETHOD.flex?= build"
in devel/flex/  Why is that there?  I don't understand
that.  It seems like that should never be done inside a
It seems like the default should always be "full", and it should be
up to the caller that includes it to set "BUILDLINK_DEPMETHOD.flex?=
build" before including it in order to change that default.  Otherwise,
every place a is included, the developer has to inspect
the to see whether it defaults to a "full" or "build"
BUILDLINK_DEPMETHOD.<pkg>.  That seems confusing.



Home | Main Index | Thread Index | Old Index