public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Ian Stakenvicius" <axs@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: net-fs/samba/, net-fs/samba/files/
Date: Thu,  8 Nov 2018 17:51:43 +0000 (UTC)	[thread overview]
Message-ID: <1541699496.c3c0a86e506899288b9677d732d5394eb006b34e.axs@gentoo> (raw)

commit:     c3c0a86e506899288b9677d732d5394eb006b34e
Author:     Ian Stakenvicius <axs <AT> gentoo <DOT> org>
AuthorDate: Thu Nov  8 17:50:06 2018 +0000
Commit:     Ian Stakenvicius <axs <AT> gentoo <DOT> org>
CommitDate: Thu Nov  8 17:51:36 2018 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c3c0a86e

net-fs/samba: do not install python DSDB bits when USE=-addc

If ADDC is disabled, samba-4.8.6 was still installing python module
smbd_dns.so , which links to libdnsserver-common-samba4.so but the
latter is NOT installed.  This causes infinite revdep-rebuild errors.

This commit patches the build system to prevent said file from being
installed.  Bumped directly to stable by maintainers.

Signed-off-by: Ian Stakenvicius <axs <AT> gentoo.org>
Package-Manager: Portage-2.3.49, Repoman-2.3.11

 .../files/samba-4.8.6-no-pydsdb-when-no-addc.patch | 36 ++++++++++++++++++++++
 .../{samba-4.8.6.ebuild => samba-4.8.6-r1.ebuild}  |  0
 2 files changed, 36 insertions(+)

diff --git a/net-fs/samba/files/samba-4.8.6-no-pydsdb-when-no-addc.patch b/net-fs/samba/files/samba-4.8.6-no-pydsdb-when-no-addc.patch
new file mode 100644
index 00000000000..1093b1de71d
--- /dev/null
+++ b/net-fs/samba/files/samba-4.8.6-no-pydsdb-when-no-addc.patch
@@ -0,0 +1,36 @@
+Do not install the python DSDB modules when ADDC is disabled.
+This is a change integrated upstream in 4.9.1 (or earlier).
+
+Without this, the python dsdb_dns.so module is installed, which
+is linked to libdnsserver-common.so that is NOT installed, and so
+potential breakage ensues.
+
+--- a/source4/dsdb/wscript_build	2018-01-17 03:08:39.000000000 -0500
++++ b/source4/dsdb/wscript_build	2018-11-08 10:43:58.064486832 -0500
+@@ -63,10 +63,11 @@
+ 	)
+ 
+ bld.SAMBA_PYTHON('python_dsdb',
+ 	source='pydsdb.c',
+ 	# the dependency on dcerpc here is because gensec
+ 	# depends on dcerpc but the waf circular dependency finder
+ 	# removes it so we end up with unresolved symbols.
+ 	deps='samdb pyldb-util dcerpc com_err pyrpc_util pyparam_util dsdb_garbage_collect_tombstones',
+-	realname='samba/dsdb.so'
++	realname='samba/dsdb.so',
++	enabled=bld.AD_DC_BUILD_IS_ENABLED()
+ 	)
+--- a/source4/dns_server/wscript_build	2018-11-08 11:49:03.450296286 -0500
++++ b/source4/dns_server/wscript_build	2018-11-08 11:49:23.327864408 -0500
+@@ -64,9 +64,10 @@
+                   private_library=True,
+                   deps='samba-hostconfig samdb-common gensec popt dnsserver_common',
+                   enabled=bld.AD_DC_BUILD_IS_ENABLED())
+ 
+ 
+ bld.SAMBA_PYTHON('python_dsdb_dns',
+ 	         source='pydns.c',
+ 	         deps='samdb pyldb-util pyrpc_util dnsserver_common pytalloc-util',
+-	         realname='samba/dsdb_dns.so')
++	         realname='samba/dsdb_dns.so',
++	         enabled=bld.AD_DC_BUILD_IS_ENABLED())

diff --git a/net-fs/samba/samba-4.8.6.ebuild b/net-fs/samba/samba-4.8.6-r1.ebuild
similarity index 100%
rename from net-fs/samba/samba-4.8.6.ebuild
rename to net-fs/samba/samba-4.8.6-r1.ebuild


             reply	other threads:[~2018-11-08 17:51 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-08 17:51 Ian Stakenvicius [this message]
  -- strict thread matches above, loose matches on Subject: below --
2022-10-27  4:48 [gentoo-commits] repo/gentoo:master commit in: net-fs/samba/, net-fs/samba/files/ Sam James
2022-09-30  2:46 Sam James
2022-07-29 18:12 Ben Kohler
2022-06-24  5:26 Sam James
2021-05-22 20:12 Lars Wendler
2020-02-09 14:46 Ben Kohler
2019-04-09  6:05 Lars Wendler
2018-05-24  8:12 Lars Wendler
2018-01-23 21:36 Mike Frysinger
2017-12-25 20:52 Lars Wendler
2017-11-13 15:49 Lars Wendler
2017-08-11 21:23 Ian Stakenvicius
2017-01-19  2:44 Lars Wendler
2016-04-13  5:42 Sergey Popov
2016-01-13 22:40 Ian Stakenvicius
2016-01-05 22:58 Victor Ostorga

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=1541699496.c3c0a86e506899288b9677d732d5394eb006b34e.axs@gentoo \
    --to=axs@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /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