Bug 15916 - missing dependency on samba-debug for mscat library
Summary: missing dependency on samba-debug for mscat library
Status: NEW
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: Build (show other bugs)
Version: 4.23.0
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Samba QA Contact
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-09-14 14:30 UTC by Michael Tokarev
Modified: 2025-09-18 05:56 UTC (History)
0 users

See Also:


Attachments
libmscat-deps.patch (295 bytes, patch)
2025-09-14 14:30 UTC, Michael Tokarev
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Tokarev 2025-09-14 14:30:23 UTC
Created attachment 18725 [details]
libmscat-deps.patch

mscat library (lib/mscat) uses symbols from samba-debug library (dbgtext, debuglevel_get_class, dbghdrclass etc), but it lacks dependency on samba-debug.
The result it that for libmscat-private-samba.so, private library directoty is not specified in LD_RUNPATH, and libsamba-debug-private-samba.so can't be found.

The attached simple patch fixes this issue.

Th dependencies subsystem in wafsamba is really fragile, it looks like.

Additionally, there's no way to build libmscat-private-samba.so directly into the only its user, dumpmscat.  When trying to use --builtin-libraries=mscat, the following error is produced:

ERROR: target mscat.builtin.objlist: dependency target samba-util is missing samba_builtin_subsystem

So apparently either samba-util definition is also wrong, or --builtin-libraries isn't working.

BTW, what this tool - dumpmscat - is used for, to begin with?

Thanks,

/mjt
Comment 1 Douglas Bagnall 2025-09-17 21:38:26 UTC
> BTW, what this tool - dumpmscat - is used for, to begin with?

Dumping MS Catalog files I guess. The implied question -- should this really be installed rather than just a dev file? -- is a good one. I don't know.
Comment 2 Michael Tokarev 2025-09-18 05:56:50 UTC
I already removed that binary from the debian samba package yesterday (together with the library in question).  *And* applied the attached patch :)