Bug 241941 - [PATCH] databases/tdb does not respect DEBUG option
Summary: [PATCH] databases/tdb does not respect DEBUG option
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Timur I. Bakeyev
URL:
Keywords: buildisok, patch
Depends on:
Blocks:
 
Reported: 2019-11-13 09:27 UTC by ml
Modified: 2023-01-09 22:06 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (timur)


Attachments
SVN patch that corrects Makefile (542 bytes, patch)
2019-11-13 09:27 UTC, ml
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ml 2019-11-13 09:27:41 UTC
Created attachment 209126 [details]
SVN patch that corrects Makefile

Binaries are stripped anyway.
Comment 1 Automation User 2019-11-28 00:09:18 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/99097176
Comment 2 Timur I. Bakeyev freebsd_committer freebsd_triage 2020-01-14 01:33:51 UTC
I'm wondering, if:

DEBUG_VARS=                     WITH_DEBUG=yes

would also fix this issue? As STRIP should be redefined to `true` in this case.
Comment 3 ml 2020-01-27 10:53:14 UTC
I tried compiling "WITH_DEBUG=yes" and deployed it to a few servers (AD DCs and AD members).

On all of them this started producing tons of secrets.ldb.trace.[0-9]*, sam.ldb.trace.[0-9]*, schannel_store.tdb.trace.[0-9]*, etc... in /var/db/samba4/private.

Monitoring advised that that partition was getting full and saved me.

Rebuilding without "WITH_DEBUG=yes" reverted everything to normal.



Now we might discuss what the DEBUG option should do: I thought it just meant compiling with debug info and not stripping. If it's so, then this change is wrong.
OTOH if this *should* produce that result, then it's ok:

# file /usr/local/bin/tdbbackup 
/usr/local/bin/tdbbackup: ELF 64-bit LSB executable, x86-64, version 1 (FreeBSD), dynamically linked, interpreter /libexec/ld-elf.so.1, for FreeBSD 11.3, FreeBSD-style, with debug_info, not stripped
Comment 4 Daniel Engberg freebsd_committer freebsd_triage 2022-08-07 22:23:59 UTC
Still broken using the current version in tree?
Comment 5 ml 2022-08-08 09:26:48 UTC
(In reply to Daniel Engberg from comment #4)

Look like it's OK to me.
Putting "WITH_DEBUG_PORTS+=databases/tdb" in /usr/local/etc/poudriere.d/make.conf, makes the binaries not stripped (with or without the DEBUG option).

So I think we can close the bug.

Some more info on DEBUG option, though, would be useful IMHO.
Comment 6 Graham Perrin freebsd_committer freebsd_triage 2022-10-17 12:40:32 UTC
Keyword: 

    patch
or  patch-ready

– in lieu of summary line prefix: 

    [patch]

* bulk change for the keyword
* summary lines may be edited manually (not in bulk). 

Keyword descriptions and search interface: 

    <https://bugs.freebsd.org/bugzilla/describekeywords.cgi>