Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 0/9] Better handling of freescale self-extractible binaries
Date: Thu, 27 Nov 2014 22:29:59 +0100	[thread overview]
Message-ID: <20141127222959.726ca490@free-electrons.com> (raw)
In-Reply-To: <1416432642-26005-1-git-send-email-jezz@sysmic.org>

Dear J?r?me Pouiller,

On Wed, 19 Nov 2014 22:30:33 +0100, J?r?me Pouiller wrote:

> J?r?me Pouiller (9):
>   freescale-imx: Add helper to extract self-extractible binaries

Applied, after doing several fixes. One of the fix was very important,
since without it, your patch was simply not working. It would be good
to at least minimally test build patches before sending them. See my
changelog:

    [Thomas:
     - reword the comment of the new macro, and add details about the
       expected arguments, as suggested by Yann E. Morin.
     - rework the logic to move the files in $(@D), as it was simply not
       working in the original patch. $(basename $(1)) was used to get the
       filename without the extension, but it also contains the full
       path. $(basename $(notdir $(1))) is needed.]

>   firmware-imx: Use self-extractible helper
>   imx-vpu: Use self-extractible helper
>   libfslcodec: Use self-extractible helper
>   libfslparser: Use self-extractible helper
>   libfslvpuwrap: Use self-extractible helper

All applied, after removing the useless "See freescale-imx.mk" comment.

>   gpu-viv-bin-mx6q: Use self-extractible helper

This patch does not apply to the next branch. Could you respin?

>   libz160: Use self-extractible helper

Applied, but merged into the patch adding libz160.

>   gpu-amd-bin-mx51: Use self-extractible helper

I haven't handled this new package yet.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

      parent reply	other threads:[~2014-11-27 21:29 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-19 21:30 [Buildroot] [PATCH v2 0/9] Better handling of freescale self-extractible binaries Jérôme Pouiller
2014-11-19 21:30 ` [Buildroot] [PATCH v2 1/9] freescale-imx: Add helper to extract " Jérôme Pouiller
2014-11-27 18:13   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 2/9] firmware-imx: Use self-extractible helper Jérôme Pouiller
2014-11-27 18:15   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 3/9] imx-vpu: " Jérôme Pouiller
2014-11-27 18:16   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 4/9] libfslcodec: " Jérôme Pouiller
2014-11-27 18:18   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 5/9] libfslparser: " Jérôme Pouiller
2014-11-27 18:19   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 6/9] libfslvpuwrap: " Jérôme Pouiller
2014-11-27 18:20   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 7/9] gpu-viv-bin-mx6q: " Jérôme Pouiller
2014-11-27 19:22   ` Yann E. MORIN
2014-11-19 21:30 ` [Buildroot] [PATCH v2 8/9] libz160: " Jérôme Pouiller
2014-11-19 21:30 ` [Buildroot] [PATCH v2 9/9] gpu-amd-bin-mx51: " Jérôme Pouiller
2014-11-27 21:29 ` Thomas Petazzoni [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=20141127222959.726ca490@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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