Skip to content

Comments

yum: generate build_id in -debuginfo#371

Merged
ashie merged 1 commit intofluent:masterfrom
kenhys:disable-build-id
Apr 18, 2022
Merged

yum: generate build_id in -debuginfo#371
ashie merged 1 commit intofluent:masterfrom
kenhys:disable-build-id

Conversation

@kenhys
Copy link
Contributor

@kenhys kenhys commented Feb 14, 2022

build_id links may be useful for debug but there is a case that
build_id conflicts with other packages. (e.g. gitlab-ce)

Thus "_build_id_links compat" may not be suitable, so change it to
traditional alldebug.

Before:
td-agent contains /usr/lib/debug/.build-id/xx/yy
After:
td-agent-debuginfo contains /usr/lib/debug/.build-id/xx/yy

NOTE: rpm --excludepath may be a workaround to install, but it is not
so good.

Closes: #370

build_id links may be useful for debug but there is a case that
build_id conflicts with other packages. (e.g. gitlab-ce)

Thus "_build_id_links compat" may not be suitable, so change it to
traditional alldebug.

Before: td-agent contains /usr/lib/debug/.build-id/xx/yy
After: td-agent-debuginfo contains /usr/lib/debug/.build-id/xx/yy

NOTE: rpm --excludepath may be a workaround to install, but it is not
so good.

Closes: fluent#370

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
@kenhys kenhys requested a review from ashie February 14, 2022 06:41
@ashie ashie merged commit 8198a1e into fluent:master Apr 18, 2022
@Smasherr
Copy link

Thanks for resolving this!

@kenhys kenhys mentioned this pull request Apr 25, 2022
@kenhys kenhys deleted the disable-build-id branch April 25, 2022 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

td-agent RPM includes build-ids

3 participants