libsamplerate: Define as PRIVATELIB
ClosedPublic
Actions

Authored by christos on Aug 31 2025, 4:30 PM.

Details

Reviewers

kevans
fuz

Commits

rG44114a581fba: libsamplerate: Define as PRIVATELIB
rGdafd960a3d65: libsamplerate: Define as PRIVATELIB

Summary

Fixes the following warnings:

ld: error: relocation R_X86_64_32S cannot be used against local symbol; recompile with -fPIC

defined in /usr/obj/usr/src/amd64.amd64/lib/libsamplerate/libsamplerate.a(samplerate.o)
referenced by samplerate.c:498 (/usr/src/contrib/libsamplerate/samplerate.c:498)
samplerate.o:(src_new) in archive /usr/obj/usr/src/amd64.amd64/lib/libsamplerate/libsamplerate.a

Related discussion: https://reviews.freebsd.org/D52306

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.Aug 31 2025, 4:30 PM

Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptAug 31 2025, 4:30 PM

christos requested review of this revision.Aug 31 2025, 4:30 PM

Harbormaster completed remote builds in B66715: Diff 161291.Aug 31 2025, 4:30 PM

christos mentioned this in D52306: libsamplerate: Compile with -fPIC.Aug 31 2025, 4:32 PM

Needs an entry in etc/mtree/BSD.usr.dist for include/private/samplerate (thanks, @kevans)

===> lib/libsamplerate (includes)
install -U  -C -o root -g wheel -m 444  /usr/home/fuz/src/repo/freebsd-src/contrib/libsamplerate/samplerate.h /usr/obj/usr/home/fuz/src/repo/freebsd-src/amd64.amd64/tmp/usr/include/private/samplerate/
install: /usr/obj/usr/home/fuz/src/repo/freebsd-src/amd64.amd64/tmp/usr/include/private/samplerate: No such file or directory
*** Error code 71

Stop.
make[4]: stopped making "includes" in /usr/home/fuz/src/repo/freebsd-src/lib/libsamplerate

This revision now requires changes to proceed.Aug 31 2025, 4:42 PM

Add to BSD.usr.dist

Harbormaster completed remote builds in B66716: Diff 161292.Aug 31 2025, 4:44 PM

christos added a child revision: D52308: virtual_oss: Port to base.Aug 31 2025, 4:53 PM

Seems to build fine.

This revision is now accepted and ready to land.Aug 31 2025, 6:06 PM

Closed by commit rGdafd960a3d65: libsamplerate: Define as PRIVATELIB (authored by christos). · Explain WhySep 15 2025, 1:15 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rGdafd960a3d65: libsamplerate: Define as PRIVATELIB.

christos added a commit: rG44114a581fba: libsamplerate: Define as PRIVATELIB.Wed, Oct 8, 12:55 PM

Herald added a subscriber: ivy. · View Herald TranscriptWed, Oct 8, 12:55 PM