From: Peter Korsgaard <peter@korsgaard.com>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Bernd Kuhls <bernd@kuhls.net>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2 2/2] package/transmission: fix gtk dependencies
Date: Tue, 29 Aug 2023 22:18:54 +0200 [thread overview]
Message-ID: <87pm3538lt.fsf@48ers.dk> (raw)
In-Reply-To: <20230723141918.GE2503916@scaer> (Yann E. MORIN's message of "Sun, 23 Jul 2023 16:19:18 +0200")
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> Bernd, All,
> On 2023-07-23 10:31 +0200, Bernd Kuhls spake thusly:
>> Buildroot commit 80793107a65d95f8dfd0317a726d3a737c78ef67 forgot to add
>> packages needed for gtk support:
>> https://github.com/transmission/transmission/blob/main/CMakeLists.txt#L40
>>
>> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
>> ---
>> v2: removed unneeded glibmm, added gtkmm3 dependencies (Thomas)
>>
>> package/transmission/Config.in | 8 +++++++-
>> package/transmission/transmission.mk | 2 +-
>> 2 files changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/transmission/Config.in b/package/transmission/Config.in
>> index fd4c897497..7ac18ae6d9 100644
>> --- a/package/transmission/Config.in
>> +++ b/package/transmission/Config.in
>> @@ -37,11 +37,17 @@ config BR2_PACKAGE_TRANSMISSION_DAEMON
>>
>> config BR2_PACKAGE_TRANSMISSION_GTK
>> bool "transmission-gtk"
>> + depends on !BR2_nios2 # gtkmm3
>> + depends on BR2_TOOLCHAIN_HAS_SYNC_4 # pangomm, libgtk3 -> pango -> harfbuzz -> gtkmm3
>> + depends on BR2_INSTALL_LIBSTDCPP # glibmm, libpangomm -> glibmm/pango -> gtkmm3
>> + depends on BR2_USE_WCHAR # *mm/libgtk3 -> libglib2 -> gtkmm3
> C++ and xide-char are alrady dependencies of transmission, so no need to
> replicate them here (yeah, the rule is fuzzy and unwritten, and there is
> historical baggage where they are replicated).
>> depends on BR2_PACKAGE_LIBGTK3 && BR2_SYSTEM_ENABLE_NLS
>> + select BR2_PACKAGE_GTKMM3
>> help
>> Install transmission GTK-based GUI interface.
>>
>> -comment "transmission-gtk needs NLS enabled"
>> +comment "transmission-gtk needs libgtk3 and a toolchain with NLS enabled"
>> + depends on !BR2_nios2
>> depends on BR2_PACKAGE_LIBGTK3 && !BR2_SYSTEM_ENABLE_NLS
> Still, the sync-4 dependency is missing here, no?
> Applied to master with the above changes, thanks.
Committed to 2023.05.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-08-29 20:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-23 8:31 [Buildroot] [PATCH 1/2] package/transmission: fix comment Bernd Kuhls
2023-07-23 8:31 ` [Buildroot] [PATCH v2 2/2] package/transmission: fix gtk dependencies Bernd Kuhls
2023-07-23 14:19 ` Yann E. MORIN
2023-08-29 20:18 ` Peter Korsgaard [this message]
2023-07-23 14:00 ` [Buildroot] [PATCH 1/2] package/transmission: fix comment Yann E. MORIN
2023-08-29 20:18 ` Peter Korsgaard
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=87pm3538lt.fsf@48ers.dk \
--to=peter@korsgaard.com \
--cc=bernd@kuhls.net \
--cc=buildroot@buildroot.org \
--cc=yann.morin.1998@free.fr \
/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.