All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v8 07/12] e2fsprogs: refactor to fix conflicts with busybox and util-linux
Date: Sun, 23 Apr 2017 00:34:31 +0200	[thread overview]
Message-ID: <d3b58050-8679-7eab-3b1a-ed5477f9f595@mind.be> (raw)
In-Reply-To: <20170422150317.576c0b54@free-electrons.com>



On 22-04-17 15:03, Thomas Petazzoni wrote:
> Hello,
> 
> On Sat, 22 Apr 2017 13:59:50 +0200, Arnout Vandecappelle
> (Essensium/Mind) wrote:
> 
>> +# If both e2fsprogs and busybox are selected, make certain e2fsprogs
>> +# wins the fight over who gets to have their utils actually installed
>> +ifeq ($(BR2_PACKAGE_BUSYBOX),y)
>> +E2FSPROGS_DEPENDENCIES += busybox
>> +endif
> 
> We recently had a ncurses/busybox discussion, in which to solve a
> circular dependency problem we removed such a dependency on Busybox. As
> part of this discussion, we started thinking about whether we should
> stop using the solution of depending on busybox to allow a package to
> override Busybox applets, and instead make sure the corresponding
> applets in Busybox are disabled when another package will install the
> full featured version of the program.
> 
> Should we generalize this idea, and apply it to e2fsprogs as well?

 Yes, but it should be a separate patch. This patch basically preserves the
existing behaviour of overriding the busybox versions. Note that the dependency
is redundant because we already depend on util-linux which depends on busybox
(for the same reason).


 Regards,
 Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

  reply	other threads:[~2017-04-22 22:34 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-22 11:59 [Buildroot] [PATCH v8 00/12] e2fsprogs: refactor to fix conflicts with busybox and util-linux Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 01/12] e2fsprogs: change upstream URL sf.net -> sourceforge.net Arnout Vandecappelle
2017-04-22 12:28   ` Thomas Petazzoni
2017-04-22 11:59 ` [Buildroot] [PATCH v8 02/12] e2fsprogs: add missing arch depends to comment Arnout Vandecappelle
2017-04-22 12:28   ` Thomas Petazzoni
2017-04-22 11:59 ` [Buildroot] [PATCH v8 03/12] e2fsprogs: change into menuconfig Arnout Vandecappelle
2017-04-22 12:28   ` Thomas Petazzoni
2017-04-22 11:59 ` [Buildroot] [PATCH v8 04/12] e2fsprogs: remove uuidgen support Arnout Vandecappelle
2017-04-22 12:29   ` Thomas Petazzoni
2017-04-22 21:57     ` Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 05/12] e2fsprogs: use libblkid and libuuid from util-linux for host Arnout Vandecappelle
2017-04-22 12:30   ` Thomas Petazzoni
2017-04-22 11:59 ` [Buildroot] [PATCH v8 06/12] e2fsprogs: let it create symlinks Arnout Vandecappelle
2017-04-22 12:49   ` Thomas Petazzoni
2017-04-22 22:32     ` Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 07/12] e2fsprogs: refactor to fix conflicts with busybox and util-linux Arnout Vandecappelle
2017-04-22 13:03   ` Thomas Petazzoni
2017-04-22 22:34     ` Arnout Vandecappelle [this message]
2017-04-22 11:59 ` [Buildroot] [PATCH v8 08/12] e2fsprogs: add help text for all config options Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 09/12] e2fsprogs: disable unneeded features for host Arnout Vandecappelle
2017-04-22 13:03   ` Thomas Petazzoni
2017-04-22 22:34     ` Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 10/12] e2fsprogs: add explicit --enable/--disable configure options Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 11/12] e2fsprogs: avoid conflict with util-linux fsck wrapper Arnout Vandecappelle
2017-04-22 11:59 ` [Buildroot] [PATCH v8 12/12] e2fsprogs: only add -lintl for static builds Arnout Vandecappelle
2017-04-22 13:04   ` Thomas Petazzoni

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=d3b58050-8679-7eab-3b1a-ed5477f9f595@mind.be \
    --to=arnout@mind.be \
    --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.