From: Ladislav Michl <ladis@linux-mips.org>
To: ptxdist@pengutronix.de
Cc: Roland Hieber <rhi@pengutronix.de>,
Felicitas Jung <f.jung@pengutronix.de>
Subject: Re: [ptxdist] [PATCH 2/2] fbgrab: version bump 1.0 -> 1.3.1
Date: Thu, 9 Apr 2020 10:23:26 +0200 [thread overview]
Message-ID: <20200409082326.GA1225140@lenoch> (raw)
In-Reply-To: <20200408105640.10427-2-rhi@pengutronix.de>
On Wed, Apr 08, 2020 at 12:56:40PM +0200, Roland Hieber wrote:
> The old URL is broken, but the new GitHub project doesn't have the old
> releases, so bump the version.
What about reordering patches, just in case someone does bisecting?
This way package is pulled from staging, but still broken...
> When the rule file is evaluated, the cross pkg-config wrapper may not
> exist yet, which will lead to confusing error messages during the build.
> But it turns out that calling pkg-config is not actually necessary
> because libpng is installed to the standard locations, and fbgrab builds
> just fine using '-lpng -lz' in its Makefile.
>
> Also set FBGRAB_CONF_TOOL for uniformity across rule files.
>
> fbgrab.c only says "This program is free Software, see the COPYING
> file". COPYING contains a GPL-2.0 license text, therefore the "-only"
> part is inferred.
>
> Reported-by: Felicitas Jung <f.jung@pengutronix.de>
> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
> rules/fbgrab.make | 21 ++++++++++++++-------
> 1 file changed, 14 insertions(+), 7 deletions(-)
>
> diff --git a/rules/fbgrab.make b/rules/fbgrab.make
> index eb05a024e4d4..ea7f07f6000a 100644
> --- a/rules/fbgrab.make
> +++ b/rules/fbgrab.make
> @@ -15,22 +15,29 @@ PACKAGES-$(PTXCONF_FBGRAB) += fbgrab
> #
> # Paths and names
> #
> -FBGRAB_VERSION := 1.0
> -FBGRAB_MD5 := 7af4d8774684182ed690d5da82d6d234
> +FBGRAB_VERSION := 1.3.1
> +FBGRAB_MD5 := d2f1f9a096954c252335317216dcd501
> FBGRAB := fbgrab-$(FBGRAB_VERSION)
> FBGRAB_SUFFIX := tar.gz
> -FBGRAB_URL := http://fbgrab.monells.se/$(FBGRAB).$(FBGRAB_SUFFIX)
> +FBGRAB_URL := https://github.com/GunnarMonell/fbgrab/archive/$(FBGRAB_VERSION).$(FBGRAB_SUFFIX)
> FBGRAB_SOURCE := $(SRCDIR)/$(FBGRAB).$(FBGRAB_SUFFIX)
> FBGRAB_DIR := $(BUILDDIR)/$(FBGRAB)
> +FBGRAB_LICENSE := GPL-2.0-only
> +FBGRAB_LICENSE_FILES := \
> + file://COPYING;md5=ea5bed2f60d357618ca161ad539f7c0a \
> + file://fbgrab.c;startline=6;endline=6;md5=03e379b2e488fbda4e42aa556e9cec93
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +FBGRAB_CONF_TOOL := NO
>
> # ----------------------------------------------------------------------------
> # Compile
> # ----------------------------------------------------------------------------
> -# overwrite some vars in the makefile
>
> -FBGRAB_MAKE_OPT = \
> - $(CROSS_ENV_CC) \
> - LDLIBS='`eval PATH=$(CROSS_PATH) $(CROSS_PKG_CONFIG) --libs libpng`'
> +FBGRAB_MAKE_ENV := $(CROSS_ENV_CC)
>
> # ----------------------------------------------------------------------------
> # Target-Install
> --
> 2.26.0.rc2
>
>
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
next prev parent reply other threads:[~2020-04-09 8:23 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-08 10:56 [ptxdist] [PATCH 1/2] fbgrab: remove from staging Roland Hieber
2020-04-08 10:56 ` [ptxdist] [PATCH 2/2] fbgrab: version bump 1.0 -> 1.3.1 Roland Hieber
2020-04-08 11:48 ` Michael Olbrich
2020-04-09 8:23 ` Ladislav Michl [this message]
2020-04-09 9:15 ` Roland Hieber
2020-04-09 12:41 ` Ladislav Michl
2020-04-09 12:54 ` Roland Hieber
2020-04-09 13:12 ` Michael Olbrich
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=20200409082326.GA1225140@lenoch \
--to=ladis@linux-mips.org \
--cc=f.jung@pengutronix.de \
--cc=ptxdist@pengutronix.de \
--cc=rhi@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