Re: [ptxdist] Handling build time only dependencies

[Alexander Dahl]

Hello,

On Fri, Sep 22, 2017 at 04:06:19PM +0200, Michael Olbrich wrote:
> On Fri, Sep 22, 2017 at 03:24:08PM +0200, Alexander Dahl wrote:
> > Can I express this somehow in ptxdist and how?
> 
> "select LIBFOO if BUILDTIME"

This is it, thank you. :-)

Greets
Alex

-- 
»With the first link, the chain is forged. The first speech censured, 
the first thought forbidden, the first freedom denied, chains us all 
irrevocably.« (Jean-Luc Picard, quoting Judge Aaron Satie)
*** GnuPG-FP: C28E E6B9 0263 95CF 8FAF  08FA 34AD CD00 7221 5CC6 ***


signature.asc
Description: PGP signature
___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH 6/7] host-cups: new package: CUPS PPD compiler

[Roland Hieber]

On 22.09.2017 16:04, Michael Olbrich wrote:

On Fri, Sep 22, 2017 at 02:06:58PM +0200, Juergen Borleis wrote:

When you install to "$(PTXDIST_SYSROOT_HOST)/bin" and
"$(PTXDIST_SYSROOT_HOST)/lib" the cross-ppdc variant can be reduced to:

( \
echo '#!/bin/sh'; \
echo $(PTXDIST_SYSROOT_HOST)/bin/ppdc -I 
$(PTXDIST_SYSROOT_HOST)/share/cups/ppdc "$$@"'; \
) > $(CROSS_PPDC)


This realy depends on what '-I' means and the difference between the
content of $(SYSROOT)/usr/share/cups/ppdc and
$(PTXDIST_SYSROOT_HOST)/share/cups/ppdc.

ppdc is used at build-time by packages that create extra cups filters,
right? If there are differences, then the original command-line is correct.
In that case host-cups must depend on cup and the package using ppdc must
depend on host-cups.
Or, if such packages depend on cups anyways (maybe link to something?) you
could create an options CUPS_PPDC that trigger a dependency on host-cups
and create the cross ppdc wrapper in cups.make.


That was also my first thought. But a diff -r on 
$(PTXDIST_SYSROOT_HOST)/share/cups/ppdc and 
$(PTXDIST_SYSROOT_TARGET)/usr/share/cups/ppdc shows that they don't 
differ. So I think we can spare a dependency on target-CUPS if only the 
host ppdc is needed.


 - Roland


___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] screen: Fix ptxdist error message.

[Michael Olbrich]

On Fri, Sep 22, 2017 at 01:58:42PM +, Denis OSTERLAND wrote:
> Am Freitag, den 22.09.2017, 15:30 +0200 schrieb Michael Olbrich:
> > On Fri, Sep 15, 2017 at 01:10:39PM +, Denis OSTERLAND wrote:
> > > 
> > > Copy from causes this error message:
> > > ptxdist: error: file 
> > > 'BSP/platform-Name/packages/screen-4.5.0/usr/bin/screen' is a link
> > I would like to keep /usr/bin/screen as the actuall binary. So change the
> > files in pkgdir instead. Either with a patch for Makefile.in or a custom
> > install target that moves the files after the normal world/install.
> > 
> > Michael
> OK.
> What about this:
> - @$(call install_copy, screen, 0, 0, 0755, -, /usr/bin/screen)
> + @$(call install_copy, screen, 0, 0, 0755, \
$(SCREEN_PKGDIR)/usr/bin/screen-$(SCREEN_VERSION), \
/usr/bin/screen)

Ok, I suppose, but break lines like this.

Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] Handling build time only dependencies

[Michael Olbrich]

On Fri, Sep 22, 2017 at 03:24:08PM +0200, Alexander Dahl wrote:
> Hello,
> 
> I'm a little stuck with a problem I consider a maybe common one. I
> have an ordinary C library. For some projects I compile and link
> against it the usual way, so in my applications .in rule I just do
> 'SELECT LIBFOO' and the library package will be copied to the target
> including .so file, loading at runtime etc.
> 
> Now for another BSP I have another application where I only need to
> include some header files of this library, but do not call any API
> functions and do not link against it. So I only need the library at
> build time, no library package has to be copied to the target later
> and the library doesn't need to be loaded at runtime.
> 
> Can I express this somehow in ptxdist and how?
> 
> (This is just to save space on the target, I know I could let copy the
> lib to the target, where it won't be loaded then, which is the way
> it's done currently.)

"select LIBFOO if BUILDTIME"

Michael


-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH 6/7] host-cups: new package: CUPS PPD compiler

[Michael Olbrich]

On Fri, Sep 22, 2017 at 02:06:58PM +0200, Juergen Borleis wrote:
> Hi Roland,
> 
> On Thursday 24 August 2017 00:39:10 Roland Hieber wrote:
> > [...]
> > +#
> > +# autoconf
> > +#
> > +# The --with-* options are only used to specify strings, --without-* does
> > +# nothing. So we're omitting them here. The only exception is 
> > +# --with-components=core, which we are setting to only builds libcups* 
> > (which is
> > +# needed by ppdc). 
> > +#
> > +# We have to set --exec-prefix and --libdir, otherwise the libs end up
> > in +# ${prefix}/lib64, which is not what we want.
> > +#
> > +HOST_CUPS_CONF_TOOL:= autoconf
> > +HOST_CUPS_CONF_OPT := \
> > +   $(HOST_AUTOCONF) \
> > +   --exec-prefix=/usr/ \
> > +   --libdir=/usr/lib/ \
> 
> It seems "$(HOST_AUTOCONF)" and "--libdir=/lib" is enough. And installing to
> "$(PTXDIST_SYSROOT_HOST)/bin" and "$(PTXDIST_SYSROOT_HOST)/lib" is handled
> by PTXdist automatically (library search path handling and so on).
> 
> > [...]
> > +
> > +CROSS_PPDC = $(PTXDIST_SYSROOT_CROSS)/bin/ppdc
> > +$(STATEDIR)/host-cups.install.post:
> > +   @$(call targetinfo)
> > +   @$(call world/install.post, HOST_CUPS)
> > +
> > +   ( \
> > +   echo '#!/bin/sh'; \
> > +   echo 'LD_LIBRARY_PATH=$(PTXDIST_SYSROOT_HOST)/usr/lib 
> > $(PTXDIST_SYSROOT_HOST)/usr/bin/ppdc -I $(SYSROOT)/usr/share/cups/ppdc 
> > "$$@"'; \
> > +   ) > $(CROSS_PPDC)
> 
> You should not mix "$(PTXDIST_SYSROOT_HOST)" and "$(SYSROOT)" here, because
> then the "HOST-CUPS" package has a dependency to "CUPS"!
> 
> When you install to "$(PTXDIST_SYSROOT_HOST)/bin" and
> "$(PTXDIST_SYSROOT_HOST)/lib" the cross-ppdc variant can be reduced to:
> 
>   ( \
>   echo '#!/bin/sh'; \
>   echo $(PTXDIST_SYSROOT_HOST)/bin/ppdc -I 
> $(PTXDIST_SYSROOT_HOST)/share/cups/ppdc "$$@"'; \
>   ) > $(CROSS_PPDC)

This realy depends on what '-I' means and the difference between the
content of $(SYSROOT)/usr/share/cups/ppdc and
$(PTXDIST_SYSROOT_HOST)/share/cups/ppdc.

ppdc is used at build-time by packages that create extra cups filters,
right? If there are differences, then the original command-line is correct.
In that case host-cups must depend on cup and the package using ppdc must
depend on host-cups.
Or, if such packages depend on cups anyways (maybe link to something?) you
could create an options CUPS_PPDC that trigger a dependency on host-cups
and create the cross ppdc wrapper in cups.make.

Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] screen: Fix ptxdist error message.

[Denis OSTERLAND]

Am Freitag, den 22.09.2017, 15:30 +0200 schrieb Michael Olbrich:
> On Fri, Sep 15, 2017 at 01:10:39PM +, Denis OSTERLAND wrote:
> > 
> > Copy from causes this error message:
> > ptxdist: error: file 
> > 'BSP/platform-Name/packages/screen-4.5.0/usr/bin/screen' is a link
> I would like to keep /usr/bin/screen as the actuall binary. So change the
> files in pkgdir instead. Either with a patch for Makefile.in or a custom
> install target that moves the files after the normal world/install.
> 
> Michael
OK.
What about this:
-   @$(call install_copy, screen, 0, 0, 0755, -, /usr/bin/screen)
+   @$(call install_copy, screen, 0, 0, 0755, 
$(SCREEN_PKGDIR)/usr/bin/screen-$(SCREEN_VERSION), /usr/bin/screen)

Denis
> 
> > 
> > Install it as source does, to avoid error message.
> > 
> > Signed-off-by: Denis Osterland 
> > ---
> >  rules/screen.make | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/rules/screen.make b/rules/screen.make
> > index 82a00cd..3afc965 100644
> > --- a/rules/screen.make
> > +++ b/rules/screen.make
> > @@ -53,7 +53,8 @@ $(STATEDIR)/screen.targetinstall:
> >     @$(call install_fixup, screen,AUTHOR,"Robert Schwebel 
> > ")
> >     @$(call install_fixup, screen,DESCRIPTION,missing)
> >  
> > -   @$(call install_copy, screen, 0, 0, 0755, -, /usr/bin/screen)
> > +   @$(call install_copy, screen, 0, 0, 0755, -, 
> > /usr/bin/screen-$(SCREEN_VERSION))
> > +   @$(call install_link, screen, screen-$(SCREEN_VERSION), /usr/bin/screen)
> >  
> >  ifdef PTXCONF_SCREEN_ETC_SCREENRC
> >     @$(call install_alternative, screen, 0, 0, 0644, /etc/screenrc, n)
> > -- 
> > 2.7.4
> > 
> > 
> > Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
> > Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner 
> > (Mitglied)
> > Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 
> > 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-
> > Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 
> > –
> > Vorstand: Dr.-Ing. E.h. Thomas Diehl (†) (Vorsitzender), Herr 
> > Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), 
> > Dipl.-Kfm.
> > Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. 
> > Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH)
> > Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
> > ___
> > Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese 
> > E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> > Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise 
> > erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede
> > unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung 
> > und/oder Publikation dieser E-Mail ist strengstens untersagt.
> > The contents of the above mentioned e-mail is not legally binding. This 
> > e-mail contains confidential and/or legally protected information. Please
> > inform us if you have received this e-mail by mistake and delete it in such 
> > a case. Each unauthorized reproduction, disclosure, alteration,
> > distribution and/or publication of this e-mail is strictly prohibited.
> > 
> > ___
> > ptxdist mailing list
> > ptxdist@pengutronix.de
Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner 
(Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 
620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – 
Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. E.h. Thomas Diehl (†) (Vorsitzender), Herr 
Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), 
Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, 
Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) 
Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail 
enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten 
haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form 
der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation 
dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail 
contains confidential and/or legally protected information. Please inform us if 
you have received this e-mail by mistake and 

Re: [ptxdist] [PATCH] protobuf: version bump 3.3.1 -> 3.3.2

[Michael Olbrich]

On Wed, Sep 20, 2017 at 04:19:50PM +0200, David Jander wrote:
> On Thu, 27 Jul 2017 16:31:15 -0400
> j...@ringle.org wrote:
> 
> > From: Jon Ringle 
> > 
> > This fixes unused parameter 'deterministic' issue:
> > src/backend-common/backend_ipc.pb.cc:32994:10: error: unused parameter 
> > 'deterministic' [-Werror=unused-parameter]
> >  bool deterministic, ::google::protobuf::uint8* target) const {
> >   ^
> > 
> > Signed-off-by: Jon Ringle 
> > ---
> >  patches/{protobuf-3.3.1 => protobuf-3.3.2}/autogen.sh | 0
> >  rules/protobuf.make   | 4 ++--
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >  rename patches/{protobuf-3.3.1 => protobuf-3.3.2}/autogen.sh (100%)
> > 
> > diff --git a/patches/protobuf-3.3.1/autogen.sh 
> > b/patches/protobuf-3.3.2/autogen.sh
> > similarity index 100%
> > rename from patches/protobuf-3.3.1/autogen.sh
> > rename to patches/protobuf-3.3.2/autogen.sh
> > diff --git a/rules/protobuf.make b/rules/protobuf.make
> > index 35d065b..97761d7 100644
> > --- a/rules/protobuf.make
> > +++ b/rules/protobuf.make
> > @@ -16,8 +16,8 @@ PACKAGES-$(PTXCONF_PROTOBUF) += protobuf
> >  #
> >  # Paths and names
> >  #
> > -PROTOBUF_VERSION   := 3.3.1
> > -PROTOBUF_MD5   := 20c685147753b515ce380421442044b5
> > +PROTOBUF_VERSION   := 3.3.2
> > +PROTOBUF_MD5   := 19ed45d0cbbb88de2c4922978235d314
> >  PROTOBUF   := protobuf-$(PROTOBUF_VERSION)
> >  PROTOBUF_SUFFIX:= tar.gz
> >  PROTOBUF_URL   := 
> > https://github.com/google/protobuf/archive/v$(PROTOBUF_VERSION).$(PROTOBUF_SUFFIX)
> 
> If I download protobuf-3.3.2 today from this URL, I get a different MD5sum:
> 
> $ md5sum src/protobuf-3.3.2.tar.gz 
> ef2a6a6bb3b92d8fa8d71e3cef741f2f  src/protobuf-3.3.2.tar.gz
> 
> The contents of the file look very legit though (no accidental HTML error or
> such). It definitely contains something very similar to your protobuf-3.3.2
> AFAICS. Are github download archives MD5-stable? Am I opening some ugly can of
> worms here, or is there some serious security issue going on?

I have the file with the old md5 and got the new one with your md5. The
content is identical. So far github URLs were quite stable. I hope this
doesn't start a new trend. I'll add the new md5 to the rule.

Regards,
Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] screen: Fix ptxdist error message.

[Michael Olbrich]

On Fri, Sep 15, 2017 at 01:10:39PM +, Denis OSTERLAND wrote:
> Copy from causes this error message:
> ptxdist: error: file 'BSP/platform-Name/packages/screen-4.5.0/usr/bin/screen' 
> is a link

I would like to keep /usr/bin/screen as the actuall binary. So change the
files in pkgdir instead. Either with a patch for Makefile.in or a custom
install target that moves the files after the normal world/install.

Michael

> Install it as source does, to avoid error message.
> 
> Signed-off-by: Denis Osterland 
> ---
>  rules/screen.make | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/rules/screen.make b/rules/screen.make
> index 82a00cd..3afc965 100644
> --- a/rules/screen.make
> +++ b/rules/screen.make
> @@ -53,7 +53,8 @@ $(STATEDIR)/screen.targetinstall:
>   @$(call install_fixup, screen,AUTHOR,"Robert Schwebel 
> ")
>   @$(call install_fixup, screen,DESCRIPTION,missing)
>  
> - @$(call install_copy, screen, 0, 0, 0755, -, /usr/bin/screen)
> + @$(call install_copy, screen, 0, 0, 0755, -, 
> /usr/bin/screen-$(SCREEN_VERSION))
> + @$(call install_link, screen, screen-$(SCREEN_VERSION), /usr/bin/screen)
>  
>  ifdef PTXCONF_SCREEN_ETC_SCREENRC
>   @$(call install_alternative, screen, 0, 0, 0644, /etc/screenrc, n)
> -- 
> 2.7.4
> 
> 
> Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
> Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner 
> (Mitglied)
> Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 
> 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – 
> Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
> Vorstand: Dr.-Ing. E.h. Thomas Diehl (†) (Vorsitzender), Herr 
> Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), 
> Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, 
> Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. 
> (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
> ___
> Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail 
> enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten 
> haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form 
> der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation 
> dieser E-Mail ist strengstens untersagt.
> The contents of the above mentioned e-mail is not legally binding. This 
> e-mail contains confidential and/or legally protected information. Please 
> inform us if you have received this e-mail by mistake and delete it in such a 
> case. Each unauthorized reproduction, disclosure, alteration, distribution 
> and/or publication of this e-mail is strictly prohibited.
> 
> ___
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

[ptxdist] Handling build time only dependencies

[Alexander Dahl]

Hello,

I'm a little stuck with a problem I consider a maybe common one. I
have an ordinary C library. For some projects I compile and link
against it the usual way, so in my applications .in rule I just do
'SELECT LIBFOO' and the library package will be copied to the target
including .so file, loading at runtime etc.

Now for another BSP I have another application where I only need to
include some header files of this library, but do not call any API
functions and do not link against it. So I only need the library at
build time, no library package has to be copied to the target later
and the library doesn't need to be loaded at runtime.

Can I express this somehow in ptxdist and how?

(This is just to save space on the target, I know I could let copy the
lib to the target, where it won't be loaded then, which is the way
it's done currently.)

Greets
Alex

-- 
»With the first link, the chain is forged. The first speech censured, 
the first thought forbidden, the first freedom denied, chains us all 
irrevocably.« (Jean-Luc Picard, quoting Judge Aaron Satie)
*** GnuPG-FP: C28E E6B9 0263 95CF 8FAF  08FA 34AD CD00 7221 5CC6 ***


signature.asc
Description: PGP signature
___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH 6/7] host-cups: new package: CUPS PPD compiler

[Juergen Borleis]

Hi Roland,

On Thursday 24 August 2017 00:39:10 Roland Hieber wrote:
> [...]
> +#
> +# autoconf
> +#
> +# The --with-* options are only used to specify strings, --without-* does
> +# nothing. So we're omitting them here. The only exception is 
> +# --with-components=core, which we are setting to only builds libcups* 
> (which is
> +# needed by ppdc). 
> +#
> +# We have to set --exec-prefix and --libdir, otherwise the libs end up
> in +# ${prefix}/lib64, which is not what we want.
> +#
> +HOST_CUPS_CONF_TOOL  := autoconf
> +HOST_CUPS_CONF_OPT   := \
> + $(HOST_AUTOCONF) \
> + --exec-prefix=/usr/ \
> + --libdir=/usr/lib/ \

It seems "$(HOST_AUTOCONF)" and "--libdir=/lib" is enough. And installing to
"$(PTXDIST_SYSROOT_HOST)/bin" and "$(PTXDIST_SYSROOT_HOST)/lib" is handled
by PTXdist automatically (library search path handling and so on).

> [...]
> +
> +CROSS_PPDC = $(PTXDIST_SYSROOT_CROSS)/bin/ppdc
> +$(STATEDIR)/host-cups.install.post:
> + @$(call targetinfo)
> + @$(call world/install.post, HOST_CUPS)
> +
> + ( \
> + echo '#!/bin/sh'; \
> + echo 'LD_LIBRARY_PATH=$(PTXDIST_SYSROOT_HOST)/usr/lib 
> $(PTXDIST_SYSROOT_HOST)/usr/bin/ppdc -I $(SYSROOT)/usr/share/cups/ppdc 
> "$$@"'; \
> + ) > $(CROSS_PPDC)

You should not mix "$(PTXDIST_SYSROOT_HOST)" and "$(SYSROOT)" here, because
then the "HOST-CUPS" package has a dependency to "CUPS"!

When you install to "$(PTXDIST_SYSROOT_HOST)/bin" and
"$(PTXDIST_SYSROOT_HOST)/lib" the cross-ppdc variant can be reduced to:

( \
echo '#!/bin/sh'; \
echo $(PTXDIST_SYSROOT_HOST)/bin/ppdc -I 
$(PTXDIST_SYSROOT_HOST)/share/cups/ppdc "$$@"'; \
) > $(CROSS_PPDC)

Cheers
Juergen

-- 
Pengutronix e.K.                             | Juergen Borleis             |
Industrial Linux Solutions                   | http://www.pengutronix.de/  |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH 2/7] qpdf: new package: PDF inspection library and tools

[Roland Hieber]

On 11.09.2017 12:11, Juergen Borleis wrote:

Hi Roland,

On Thursday 24 August 2017 00:39:06 Roland Hieber wrote:

[...]
+#
+# Paths and names
+#
+QPDF_VERSION   := 6.0.0
+QPDF_MD5   := a0601b0bc56d3f412fd3afecfce2721c
+QPDF   := qpdf-$(QPDF_VERSION)
+QPDF_SUFFIX:= tar.gz
+QPDF_URL   := $(call ptx/mirror, SF,
/qpdf/qpdf/$(QPDF_VERSION)/$(QPDF).$(QPDF_SUFFIX)) +QPDF_SOURCE :=
$(SRCDIR)/$(QPDF).$(QPDF_SUFFIX)
+QPDF_DIR   := $(BUILDDIR)/$(QPDF)
+QPDF_LICENSE   := Artistic-2.0


Downloading the qpdf archive leads to a different content and thus the MD5
sum differs. The a0601b0bc56d3f412fd3afecfce2721c package contains an
unreleased version, which needs to create the 'configure' script first.

Where did you download your archive?


Huh, indeed. When I downloaded it from SourceForge (through ptxdist 
get), I got the version with the a0601b0 MD5, which didn't have a 
configure script. The maintainers seem to have released a new tarball 
under the same name. Will update the rules in v2.


 - Roland

___
ptxdist mailing list
ptxdist@pengutronix.de