mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH v2 1/3] host-gengetopt: add package
Date: Wed, 3 Jun 2020 17:19:53 +0200	[thread overview]
Message-ID: <20200603151953.GB1470@pengutronix.de> (raw)
In-Reply-To: <58ae46a9-3489-808e-6c40-349aac68b54c@pengutronix.de>

On Wed, Jun 03, 2020 at 10:30:51AM +0200, Marian Cichy wrote:
> 
> 
> On 6/2/20 3:18 PM, Michael Olbrich wrote:
> > On Mon, Jun 01, 2020 at 11:49:39AM +0200, Marian Cichy wrote:
> > > this package is a dependency for bayer2rgb3,
> > > which is a dependency for gst-bayer2rgb-neon
> > > ---
> > 
> > This is a v2 patch, so it should be marked as such.
> > 
> > 
> 
> I did use git format-patch -v 2 to mark it. Should I also mention v2 in the
> commit message?

Sorry, I am blind. You did the right thing here.

Michael

> > >   configs/ptxconfig         |  1 +
> > >   rules/host-gengetopt.in   |  9 +++++++++
> > >   rules/host-gengetopt.make | 26 ++++++++++++++++++++++++++
> > >   3 files changed, 36 insertions(+)
> > >   create mode 100644 rules/host-gengetopt.in
> > >   create mode 100644 rules/host-gengetopt.make
> > > 
> > > diff --git a/configs/ptxconfig b/configs/ptxconfig
> > > index 972e7615..67200af3 100644
> > > --- a/configs/ptxconfig
> > > +++ b/configs/ptxconfig
> > 
> > This file does not exist. I guess you created the patch from a BSP?
> > 
> > Michael
> > 
> 
> Yes. I thought that we need to update the ptxconfig entries with every new
> package.
> 
> > > @@ -105,6 +105,7 @@ PTXCONF__ptxconfig_MAGIC__=y
> > >   PTXCONF_HOST_BSDIFF=y
> > >   # PTXCONF_HOST_DFU_UTIL is not set
> > >   PTXCONF_HOST_FILE_STACK=y
> > > +# PTXCONF_HOST_GENGETOPT is not set
> > >   # PTXCONF_HOST_OPENOCD is not set
> > >   # PTXCONF_HOST_QEMU is not set
> > >   # PTXCONF_HOST_TESTFRAMEWORK is not set
> > > diff --git a/rules/host-gengetopt.in b/rules/host-gengetopt.in
> > > new file mode 100644
> > > index 00000000..f6855ab5
> > > --- /dev/null
> > > +++ b/rules/host-gengetopt.in
> > > @@ -0,0 +1,9 @@
> > > +## SECTION=hosttools
> > > +
> > > +config HOST_GENGETOPT
> > > +	tristate
> > > +	prompt "gengetopt"
> > > +	help
> > > +	  Gengetopt is a tool to write command line option
> > > +	  parsing code for C programs.
> > > +
> > > diff --git a/rules/host-gengetopt.make b/rules/host-gengetopt.make
> > > new file mode 100644
> > > index 00000000..2c1b3bb0
> > > --- /dev/null
> > > +++ b/rules/host-gengetopt.make
> > > @@ -0,0 +1,26 @@
> > > +# -*-makefile-*-
> > > +#
> > > +# Copyright (C) 2020 by Marian Cichy <m.cichy@pengutronix.de>
> > > +#
> > > +# For further information about the PTXdist project and license conditions
> > > +# see the README file.
> > > +#
> > > +
> > > +#
> > > +# We provide this package
> > > +#
> > > +HOST_PACKAGES-$(PTXCONF_HOST_GENGETOPT) += host-gengetopt
> > > +
> > > +#
> > > +# Paths and names
> > > +#
> > > +HOST_GENGETOPT_VERSION	:= 2.23
> > > +HOST_GENGETOPT_MD5	:= ea165d618640dbdecb42ae4491337965
> > > +HOST_GENGETOPT		:= gengetopt-$(HOST_GENGETOPT_VERSION)
> > > +HOST_GENGETOPT_SUFFIX	:= tar.xz
> > > +HOST_GENGETOPT_URL	:= https://ftp.gnu.org/gnu/gengetopt/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> > > +HOST_GENGETOPT_SOURCE	:= $(SRCDIR)/$(HOST_GENGETOPT).$(HOST_GENGETOPT_SUFFIX)
> > > +HOST_GENGETOPT_DIR	:= $(HOST_BUILDDIR)/$(HOST_GENGETOPT)
> > > +HOST_GENGETOPT_LICENSE	:= GPL-3.0-only
> > > +HOST_GENGETOPT_LICENSE_FILES := file://COPYING;md5=ff95bfe019feaf92f524b73dd79e76eb
> > > +# vim: syntax=make
> > > -- 
> > > 2.20.1
> > > 
> > > 
> > > _______________________________________________
> > > ptxdist mailing list
> > > ptxdist@pengutronix.de
> > > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> > > 
> > 
> 
> mci
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@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
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

      reply	other threads:[~2020-06-03 15:19 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-01  9:49 Marian Cichy
2020-06-01  9:49 ` [ptxdist] [PATCH v2 2/3] bayer2rgb3: " Marian Cichy
2020-06-02 13:22   ` Michael Olbrich
2020-06-01  9:49 ` [ptxdist] [PATCH v2 3/3] gst-bayer2rgb-neon: " Marian Cichy
2020-06-02 13:24   ` Michael Olbrich
2020-06-02 13:18 ` [ptxdist] [PATCH v2 1/3] host-gengetopt: " Michael Olbrich
2020-06-03  8:30   ` Marian Cichy
2020-06-03 15:19     ` Michael Olbrich [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200603151953.GB1470@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox