Re: [ptxdist] [RFC PATCH 3/3] host-mfgtools: new package

[Michael Olbrich <m.olbrich@pengutronix.de>]

On Thu, Mar 26, 2020 at 08:28:26AM +0000, Denis Osterland-Heim wrote:
> Hi Roland,
> 
> sorry for the confusion yesterday, seems it was to late for me...
> 
> Am Mittwoch, den 25.03.2020, 19:33 +0100 schrieb Roland Hieber:
> > On Tue, Mar 24, 2020 at 02:53:23PM +0000, Denis Osterland-Heim wrote:
> > > Tool to USB boot i.MX systems from NXP via Android fastboot.
> > > 
> > > Signed-off-by: Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > ---
> > >  rules/host-mfgtools.in   | 13 ++++++++++++
> > >  rules/host-mfgtools.make | 43 ++++++++++++++++++++++++++++++++++++++++
> > >  2 files changed, 56 insertions(+)
> > >  create mode 100644 rules/host-mfgtools.in
> > >  create mode 100644 rules/host-mfgtools.make
> > > 
> > > diff --git a/rules/host-mfgtools.in b/rules/host-mfgtools.in
> > > new file mode 100644
> > > index 000000000..c9046f10d
> > > --- /dev/null
> > > +++ b/rules/host-mfgtools.in
> > > @@ -0,0 +1,13 @@
> > > +## SECTION=hosttools
> > > +
> > > +config HOST_MFGTOOLS
> > > +	tristate
> > > +	default y if ALLYES
> > > +	prompt "uuu (Universal Update Utility), mfgtools 3.0"
> > > +	select HOST_CMAKE
> > > +	select HOST_LIBUSB
> > > +	select HOST_ZLIB
> > > +	select HOST_BZIP2
> > > +	select HOST_LIBZIP
> > > +	help
> > > +	  Freescale/NXP I.MX Chip image deploy tools.
> > > diff --git a/rules/host-mfgtools.make b/rules/host-mfgtools.make
> > > new file mode 100644
> > > index 000000000..fb4b2cf05
> > > --- /dev/null
> > > +++ b/rules/host-mfgtools.make
> > > @@ -0,0 +1,43 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2020 by Denis Osterland-Heim <Denis.Osterland@diehl.com>
> > > +#
> > > +# For further information about the PTXdist project and license conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +HOST_PACKAGES-$(PTXCONF_HOST_MFGTOOLS) += host-mfgtools
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +HOST_MFGTOOLS_VERSION	:= 1.3.136-9-g47ddef5
> > 
> > Apparently this commit was released a few days ago as version 1.3.154:
> > <https://github.com/NXPmicro/mfgtools/releases>
> I have already changed it locally, v2 will come ;-)
> 
> > 
> > > +HOST_MFGTOOLS_MD5	:= 1555253943e1f39746694365c7f89eb1
> > > +HOST_MFGTOOLS		:= mfgtools-$(HOST_MFGTOOLS_VERSION)
> > > +HOST_MFGTOOLS_SUFFIX	:= tar.gz
> > > +HOST_MFGTOOLS_URL	:= https://github.com/NXPmicro/mfgtools.git;tag=47ddef51d6c24b1f917f348d0f62b4af38852994
> > 
> > So you can use the default release tarball here, and use the default
> > extract stage below.
> Not yet, but hopefully with next version.
 
> > 
> > > +HOST_MFGTOOLS_SOURCE	:= $(SRCDIR)/$(HOST_MFGTOOLS).$(HOST_MFGTOOLS_SUFFIX)
> > > +HOST_MFGTOOLS_DIR	:= $(HOST_BUILDDIR)/$(HOST_MFGTOOLS)
> > > +HOST_MFGTOOLS_LICENSE	:= BSD-3-Clause
> > > +HOST_MFGTOOLS_LICENSE_FILES	:= \
> > > +	file://LICENSE;md5=38ec0c18112e9a92cffc4951661e85a5
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Extract
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +$(STATEDIR)/host-mfgtools.extract:
> > > +	@$(call targetinfo)
> > > +	@$(call clean, $(HOST_MFGTOOLS_DIR))
> > > +	@$(call extract, HOST_MFGTOOLS)
> > > +	@echo "uuu_$(HOST_MFGTOOLS_VERSION)" > $(HOST_MFGTOOLS_DIR)/.tarball-version
> > 
> > NB: The released tarballs also don't have this file, is it really needed?
> Unfortunately it is still need, until upstream generates a tarball with the file.
> https://github.com/NXPmicro/mfgtools/issues/173

Do this in extract.post not extract.

Michael

> Regards Denis
> 
> > 
> >  - Roland
> > 
> > > +	@$(call patchin, HOST_MFGTOOLS)
> > > +	@$(call touch)
> > > +
> > > +# ----------------------------------------------------------------------------
> > > +# Prepare
> > > +# ----------------------------------------------------------------------------
> > > +
> > > +HOST_MFGTOOLS_CONF_TOOL	:= cmake
> > > +
> > > +# vim: syntax=make
> > > -- 
> > > 2.25.2
> > > 
> > > 
> > > 
> > > Diehl Connectivity Solutions GmbH
> > > Geschäftsführung: Horst Leonberger
> > > Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> > > Nürnberg: HRB 32315
> > > ___________________________________________________________________________________________________
> > > 
> > > 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.
> > > - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> > > 
> > > 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. 
> > > - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > > 
> > 
> > 
> 
> 
> Diehl Connectivity Solutions GmbH
> Geschäftsführung: Horst Leonberger
> Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht
> Nürnberg: HRB 32315
> ___________________________________________________________________________________________________
> 
> 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.
> - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/
> 
> 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. 
> - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
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 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de