Skip to content

Commit e1e0f73

Browse files
committed
style: shorten FIPS musl compatibility comment
1 parent d1997d4 commit e1e0f73

1 file changed

Lines changed: 2 additions & 9 deletions

File tree

‎boringssl-static/pom.xml‎

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -313,15 +313,8 @@
313313
</then>
314314
</if>
315315

316-
<!-- Drop the DT_NEEDED on ld-linux-<arch>.so. It gets recorded because
317-
__tls_get_addr lives in glibc's loader, but musl defines that symbol itself,
318-
so only the filename lookup fails - on Alpine the library is unloadable
319-
purely because of this entry. Removing it is a no-op on glibc, where the
320-
loader is already mapped. patchelf exits 0 when the entry is absent, so this
321-
is safe on architectures that never had it. async-profiler does exactly the
322-
same thing when packaging its single glibc+musl binary:
323-
https://github.com/async-profiler/async-profiler/blob/bb8f0476a1804ac77b48b84a3905fb26e9ca7428/Makefile#L156
324-
See https://github.com/netty/netty-tcnative/issues/907 -->
316+
<!-- Same ld-linux DT_NEEDED removal as the release profiles.
317+
See docs/musl-compatibility.md -->
325318
<if>
326319
<equals arg1="${os.detected.name}" arg2="linux" />
327320
<then>

0 commit comments

Comments
 (0)