Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 28 Jun 2021 14:39:57 +0000
From:      bugzilla-noreply@freebsd.org
To:        desktop@FreeBSD.org
Subject:   [Bug 256076] devel/libsoup: circular dependencies with samba413 and tracker
Message-ID:  <bug-256076-39348-pZLisheyOf@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-256076-39348@https.bugs.freebsd.org/bugzilla/>
References:  <bug-256076-39348@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D256076

--- Comment #5 from commit-hook@FreeBSD.org ---
A commit in branch main references this bug:

URL:
https://cgit.FreeBSD.org/ports/commit/?id=3Ded9df2d08f7837c866f525515b8fc8c=
83a919694

commit ed9df2d08f7837c866f525515b8fc8c83a919694
Author:     Adriaan de Groot <adridg@FreeBSD.org>
AuthorDate: 2021-06-12 14:19:46 +0000
Commit:     Adriaan de Groot <adridg@FreeBSD.org>
CommitDate: 2021-06-28 14:39:46 +0000

    devel/libsoup: make samba a (not registered) runtime dep

    This picks up an idea from PR 235207 again, where NTLM authentication
    is optional. Previous commits made samba a build-dependency so
    that the --enable-ntlm option would work. The meson build -- even
    though the dep is entirely runtime -- would check for the presence
    of ntlm_auth at build-time, to be able to pass in the path to
    ntlm_auth to the build.

    Re-jig this to not do the check, accept the option as-is, and
    use a likely path to ntlm_auth instead (which is what the original
    patch in 235207 did).

    A circular dependency from samba41[23] -> libsoup and back
    can be triggered by setting option SMB in libsoup and
    option SPOTLIGHT in samba. Originally the runtime dependency
    was registered as
            USES=3D   samba:run
    A _RUN dependency still wants to build, though, and package and
    index builds failed. Cut the circular dependency by **not**
    registering samba as a runtime dependency and using a pkg-message
    to tell users to possibly install samba.

    Anyone who now has "but my samba is installed in a different prefix"
    can file a PR with a patch fixing meson better.

    PR:             256076
    Reported by:    Dimitry Andric

 devel/libsoup/Makefile                      | 31 +++++++++++++++++++++----=
----
 devel/libsoup/files/patch-meson.build (new) | 17 ++++++++++++++++
 devel/libsoup/pkg-message (new)             | 10 ++++++++++
 3 files changed, 50 insertions(+), 8 deletions(-)

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-256076-39348-pZLisheyOf>