Github messages for voidlinux
 help / color / mirror / Atom feed
From: ahesford <ahesford@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Closed]: Samba 4.13.2
Date: Sat, 19 Dec 2020 23:54:52 +0100	[thread overview]
Message-ID: <20201219225452.XqzGkCIQYUGx2fKFNNN8yp-uRqMYXTUgaznZFtH8Qo0@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-27166@inbox.vuxu.org>

[-- Attachment #1: Type: text/plain, Size: 2813 bytes --]

There's a closed pull request on the void-packages repository

Samba 4.13.2
https://github.com/void-linux/void-packages/pull/27166

Description:
This is an attempt to resolve the outstanding issues in #17544 and bring Samba up to date. Regarding the issues described in that PR:
1. `sssd` is bumped
2. I removed `libwbclient` from `sssd` some time ago because we weren't doing it right anyway. No conflict remains.
3. I have removed most of the private libraries by telling Samba to build these into the public libraries. This seems preferrable than having a horde of private libraries that still have to be tracked for shlib dependency resolution, or else break resolution for all dependants. Unfotunately, not all private libraries can be built in without breaking the samba build, so we still have a few unversioned `*-samba4.so` shared objects to keep track of.
4. `common/shlibs` is properly arranged given the current package configuration.
5. I do not think there is great value in splitting samba *ad infinitum* like Alpine does. I have, however, taken some inspiration from Alpine in where subpackages contents are divided.
6. I have fixed the Python extension build process where necessary so that Python libraries have a `.so` extension rather than a `.cpython<arch-specific-garbage>.so`, so no problem here.
7. I have enabled the Python packages, but AD Domain Controller functionality is marked as experiment when used with system `krb5`, so I've left that disabled. However, the groundwork is laid for a future update where things are not experimental.

Also, everything properly uses `qemu-user-static` and the `--cross-execute` capability of `waf`, rather than relying on `cross.answer` files everywhere.

I have mostly followed tried to honor the existing subpackage structure, although there are some changes.
1. The `samba-cups` package has been replaced by `smbclient`, which assumes responsibility for `smbspool`. The `-cups` package was nothing but a symlink to `smbspool` anywhere, so there is no need to separate it out.
2. Samba 4 expands the collection of "client" programs, so I've added more programs to the `smbclient` subpackage, taking inspiration from Alpine. I almost want to rename this `samba-clients`, but I'm not sure it's worth the trouble given that `smbclient` is a leaf package so we'd have to do a transitioal package anyway.
3. I've renamed `libsmbclient` to `samba-libs` and made it assume all of the core samba libraries used by both the clients in `smbclient` and the servers in `samba`. We could try to split this further, but there are a lot of interdependencies and I'm not sure it's worth the effort.

The new package seems to work for me, especially when using `smbclient` to browse Win10 hosts; this was not working with the existing package.


      parent reply	other threads:[~2020-12-19 22:54 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-14 22:30 [PR PATCH] " ahesford
2020-12-14 23:11 ` [PR REVIEW] " ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:11 ` ericonr
2020-12-14 23:30 ` ahesford
2020-12-14 23:31 ` ahesford
2020-12-14 23:31 ` ahesford
2020-12-14 23:32 ` ahesford
2020-12-14 23:33 ` ahesford
2020-12-14 23:47 ` ericonr
2020-12-14 23:47 ` ericonr
2020-12-15  0:02 ` [PR PATCH] [Updated] " ahesford
2020-12-15  0:10 ` ahesford
2020-12-15  0:21 ` [PR REVIEW] " sgn
2020-12-15  0:22 ` [PR PATCH] [Updated] " ahesford
2020-12-15  0:23 ` [PR REVIEW] " sgn
2020-12-15  0:27 ` sgn
2020-12-15  2:19 ` ahesford
2020-12-15  2:20 ` ahesford
2020-12-15  2:33 ` ahesford
2020-12-15  2:39 ` ericonr
2020-12-15  2:42 ` [PR PATCH] [Updated] " ahesford
2020-12-15  2:49 ` ahesford
2020-12-15  3:01 ` ericonr
2020-12-15  3:07 ` [PR REVIEW] " sgn
2020-12-15  3:18 ` [PR PATCH] [Updated] " ahesford
2020-12-15  3:23 ` ahesford
2020-12-15  4:45 ` [PR REVIEW] " ericonr
2020-12-15  6:16 ` [PR PATCH] [Updated] " ahesford
2020-12-15  6:27 ` ahesford
2020-12-15  6:36 ` [PR PATCH] [Updated] " ahesford
2020-12-15  7:39 ` Anachron
2020-12-15 13:54 ` ericonr
2020-12-15 13:55 ` SirFerdek
2020-12-15 13:58 ` SirFerdek
2020-12-15 13:59 ` ahesford
2020-12-15 14:08 ` Anachron
2020-12-15 14:18 ` [PR PATCH] [Updated] " ahesford
2020-12-15 14:19 ` ahesford
2020-12-15 15:17 ` [PR PATCH] [Updated] " ahesford
2020-12-15 17:56 ` Anachron
2020-12-19 22:54 ` ahesford
2020-12-19 22:54 ` ahesford [this message]

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=20201219225452.XqzGkCIQYUGx2fKFNNN8yp-uRqMYXTUgaznZFtH8Qo0@z \
    --to=ahesford@users.noreply.github.com \
    --cc=ml@inbox.vuxu.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;
as well as URLs for NNTP newsgroup(s).