Bug 274190 - mail/claws-mail: links with net/openldap26-client if installed
Summary: mail/claws-mail: links with net/openldap26-client if installed
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Chris Hutchinson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-01 03:12 UTC by Ivan Rozhuk
Modified: 2023-10-01 20:06 UTC (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan Rozhuk 2023-10-01 03:12:56 UTC 
claws-mail is missing a required shared library: libldap.so.2
claws-mail is missing a required shared library: liblber.so.2
after install claws-mail on system without libreoffice.
Build host have libreoffice and net/openldap26-client as dep.
Comment 1 Xin LI freebsd_committer 2023-10-01 05:14:38 UTC 
The problem is that Makefile.claws is incorrectly overriding CONFIGURE_ARGS.  This makes all option-derived CONFIGURE_ARGS to get lost.

Something like this would fix it.


===
diff --git a/mail/claws-mail/Makefile.claws b/mail/claws-mail/Makefile.claws
index 53ee30e66310..9ea0a1b576fe 100644
--- a/mail/claws-mail/Makefile.claws
+++ b/mail/claws-mail/Makefile.claws
@@ -8,7 +8,7 @@ LICENSE=        GPLv3
 USES+=         autoreconf gettext-tools gmake iconv libtool pkgconfig tar:xz
 GNU_CONFIGURE= yes
 
-CONFIGURE_ARGS=        ${ICONV_CONFIGURE_ARG} --disable-dbus --disable-static
+CONFIGURE_ARGS+=       ${ICONV_CONFIGURE_ARG} --disable-dbus --disable-static
 
 .include "${.CURDIR}/../claws-mail/Makefile.ver"
 .include "${.CURDIR}/../claws-mail/Makefile.claws.plugins"
Comment 2 bsd 2023-10-01 05:45:22 UTC 
(In reply to Xin LI from comment #1)
Testing proposed patch...
Comment 3 bsd 2023-10-01 20:06:56 UTC 
Building Claws Mail with OPTION LDAP both on and off for both GTK2 and GTK3 version succeeds with proposed patch applied. All those builds are basically clean ones, in a dedicated jail. So patch proposed by Xin Li is not damaging anything, from my point of view. OP should test whether Xin's patch solves his issue.