Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3 1/3] New package: ImLib2
Date: Fri, 14 Oct 2011 00:18:05 +0200	[thread overview]
Message-ID: <201110140018.06102.arnout@mind.be> (raw)
In-Reply-To: <1318410838-24275-2-git-send-email-yegorslists@googlemail.com>

 [Will this discussion ever end? :-P ]

On Wednesday 12 October 2011 11:13:56, yegorslists at googlemail.com wrote:
[snip]
> +config BR2_PACKAGE_IMLIB2_X
> +	depends on BR2_PACKAGE_XORG7
[snip]
> +	IMLIB2_DEPENDENCIES += xlib_libX11 xlib_libXext

 If you ask me, dependencies in Config.in and in pkg.mk should be consistent.
So do select these two libraries in the Config.in (yes, all two of them, just in case that the dependency of Xext on X11 disappears some day; not that that's very likely).  The dependency on XORG7 should stay of course.

 I'm not sure if there should also be a comment that XORG7 needs to be
selected for the X backend.


 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111014/e942715e/attachment-0001.html>

  reply	other threads:[~2011-10-13 22:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-12  9:13 [Buildroot] Add utilities to set background image in Fluxbox yegorslists at googlemail.com
2011-10-12  9:13 ` [Buildroot] [PATCH v3 1/3] New package: ImLib2 yegorslists at googlemail.com
2011-10-13 22:18   ` Arnout Vandecappelle [this message]
2011-10-12  9:13 ` [Buildroot] [PATCH v3 2/3] New package: giblib yegorslists at googlemail.com
2011-10-12  9:13 ` [Buildroot] [PATCH v3 3/3] New package: feh yegorslists at googlemail.com

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=201110140018.06102.arnout@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox