Re: [ptxdist] [PATCH v3] rules: Add fbterm

[Bernhard Walle <bernhard@bwalle.de>]

Hi,

* Michael Olbrich <m.olbrich@pengutronix.de> [2012-04-08 10:15]:
> 
> This is wrong. Now the whole package is only built on x86
> 
> PACKAGES-$(PTXCONF_FBTERM) += fbterm

Of course, that's right.

> > +# ----------------------------------------------------------------------------
> > +# Prepare
> > +# ----------------------------------------------------------------------------
> > +
> 
> ifndef PTXCONF_ARCH_X86
> undefine PTXCONF_FBTERM_VESA
> endif

That yields me to the question if it wouldn't be better to omit that
check at all. If we can't get Kconfig right, what gives us the check?
It disables VESA support even when enabled where it's not available, but
that's the same as configure does automatically.

But during testing, I discovered another problem: If the option is
selected, libx86 is built:

  PACKAGES-$(PTXCONF_ARCH_X86)-$(PTXCONF_LIBX86)

doesn't help here because the dependency is resolved by kconfig before
the makefiles get executed. Do you see a problem to solve that?

Otherwise I would either just omit that VESA option (I don't need it) or
leave it to the user to select libx86 when FBTERM_VESA is selected.



Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de