Re: [ptxdist] [PATCH 6/8] host-canfestival: add host package

[Roland Hieber <rhi@pengutronix.de>]

On Wed, May 17, 2023 at 02:36:39PM +0200, Michael Olbrich wrote:
> On Mon, May 15, 2023 at 09:20:52AM +0200, Roland Hieber wrote:
> > Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> > ---
> >  rules/host-canfestival.in   |  6 ++++++
> >  rules/host-canfestival.make | 29 +++++++++++++++++++++++++++++
> >  2 files changed, 35 insertions(+)
> >  create mode 100644 rules/host-canfestival.in
> >  create mode 100644 rules/host-canfestival.make
> > 
> > diff --git a/rules/host-canfestival.in b/rules/host-canfestival.in
> > new file mode 100644
> > index 000000000000..46ad4eef5cba
> > --- /dev/null
> > +++ b/rules/host-canfestival.in
> > @@ -0,0 +1,6 @@
> > +## SECTION=hosttools_noprompt
> > +
> > +config HOST_CANFESTIVAL
> > +	tristate
> > +	default y if ALLYES
> > +	select HOST_SYSTEM_PYTHON
> 
> Aaaahh, we want to get rid of Python 2 not add more dependencies :-/.
> Someone really needs to port that stuff to Python 3.

Hmm. Those Python scripts are apparently only used on the host and are
not installed on the target at all. And I don't understand why the
canfestival target package links them into the host sysroot…?!?

I guess I'll just leave out this patch for v2.

 - Roland

> 
> Michael
> 
> > diff --git a/rules/host-canfestival.make b/rules/host-canfestival.make
> > new file mode 100644
> > index 000000000000..186381581db1
> > --- /dev/null
> > +++ b/rules/host-canfestival.make
> > @@ -0,0 +1,29 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2023 by Roland Hieber <rhi@pengutronix.de>
> > +#
> > +# For further information about the PTXdist project and license conditions
> > +# see the README file.
> > +#
> > +
> > +#
> > +# We provide this package
> > +#
> > +HOST_PACKAGES-$(PTXCONF_HOST_CANFESTIVAL) += host-canfestival
> > +
> > +# ----------------------------------------------------------------------------
> > +# Prepare
> > +# ----------------------------------------------------------------------------
> > +
> > +#
> > +# not really autoconf
> > +#
> > +HOST_CANFESTIVAL_CONF_TOOL	:= autoconf
> > +HOST_CANFESTIVAL_CONF_OPT	:=  \
> > +	--prefix=/usr \
> > +	--can=socket \
> > +	--timers=unix \
> > +	--wx=0 \
> > +	$(call ptx/ifdef,PTXCONF_ENDIAN_BIG,--CANOPEN_BIG_ENDIAN=1) \
> > +
> > +# vim: syntax=make
> > -- 
> > 2.39.2
> > 
> > 
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |