From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] Patchwork cleanup, week #22
Date: Thu, 2 Jun 2016 09:17:58 +0200 [thread overview]
Message-ID: <20160602091758.714c6990@free-electrons.com> (raw)
In-Reply-To: <20160601235354.1248a84e@gmx.net>
Hello,
On Wed, 1 Jun 2016 23:53:54 +0200, Peter Seiderer wrote:
> > 6/ Makefile: Fix overlay overwriting everything
> > http://patchwork.ozlabs.org/patch/581463/
> >
> > A problem with the merged /usr option, and when the rootfs overlay
> > contains specific directories. Discussion has happened, no decision.
> >
> [...]
>
> Was not aware of this patch (and the discussion) but the same change is
> already committed with a slightly different reasoning ([1])...
Thanks for the feedback. When I looked at Maxime's patch, I also had
the impression that the problem had already been fixed, but couldn't
spot the fix (I thought we fixed it in a different way, in the skeleton
package).
However, in the review of Maxime's patch, a number of issues have been
raised, so I'm wondering we have in the end merged your patch which
does the same, without answering those concerns.
Yann?
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-06-02 7:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-01 21:26 [Buildroot] Patchwork cleanup, week #22 Thomas Petazzoni
2016-06-01 21:53 ` Peter Seiderer
2016-06-02 7:17 ` Thomas Petazzoni [this message]
2016-06-02 22:46 ` Yann E. MORIN
2016-06-02 7:41 ` Jeroen Roovers
2016-06-02 7:49 ` Thomas Petazzoni
2016-06-02 8:24 ` Jeroen Roovers
2016-06-02 22:38 ` Yann E. MORIN
2016-06-03 9:11 ` Yegor Yefremov
2016-06-03 19:55 ` Carlos Santos
2016-06-08 20:13 ` Thomas Petazzoni
2016-06-09 8:02 ` Romain Izard
2016-06-09 8:09 ` Thomas Petazzoni
2016-06-15 19:52 ` Thomas Petazzoni
2016-06-16 15:11 ` Carlos Santos
2016-06-26 11:20 ` Jörg Krause
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=20160602091758.714c6990@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.