Paths

Table of Contentst

Differential D44896

libcrypto: compile all PIC objects with -DOPENSSL_PIC
ClosedPublic
Actions

Authored by ngie on Apr 21 2024, 5:45 PM.

Tags

None

Referenced Files

	F157169960: D44896.id173966.diff
	Tue, May 19, 12:13 AM

	F157154358: D44896.id137465.diff
	Mon, May 18, 8:20 PM

	F157123787: D44896.id173963.diff
	Mon, May 18, 2:54 PM

	Unknown Object (File)
	Sun, May 17, 7:21 PM

	Unknown Object (File)
	Sun, May 17, 5:08 PM

	Unknown Object (File)
	Fri, May 15, 3:18 AM

	Unknown Object (File)
	Tue, Apr 28, 6:29 PM

	Unknown Object (File)
	Tue, Apr 28, 6:21 PM

Subscribers

Details

Reviewers

Commits

rG689cc820446a: libcrypto: compile all PIC objects with -DOPENSSL_PIC -fPIC
rG3797fe720a37: libcrypto: compile all PIC objects with -DOPENSSL_PIC -fPIC
rG14b9955e57cc: libcrypto: compile all PIC objects with -DOPENSSL_PIC

Summary

This change modifies the libcrypto PIC objects to always link with
-DOPENSSL_PIC to restore parity with the upstream build process. This
ensures that -DOPENSSL_PIC is used whenever building the auxiliary
shared objects, e.g., the fips provider, the legacy provider, etc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71560
Build 68443: arc lint + arc unit

Event Timeline

ngie created this revision.Apr 21 2024, 5:45 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 21 2024, 5:45 PM

ngie requested review of this revision.Apr 21 2024, 5:45 PM

Harbormaster completed remote builds in B57260: Diff 137465.Apr 21 2024, 5:45 PM

Fix the change to properly append the CFLAG to PICFLAG and do it after including bsd.lib.mk to properly compile/link libcrypto.so.

Harbormaster completed remote builds in B57262: Diff 137467.Apr 21 2024, 6:37 PM

e.g., the fips provider, the legacy provider, etc.

Actually it is precisely just those two

Move the definition to secure/lib/libcrypto/modules/Makefile.inc

This only affects modules - nothing else.

Harbormaster completed remote builds in B71560: Diff 173963.Mar 19 2026, 11:59 PM

emaste accepted this revision.Mar 20 2026, 12:44 AM

This revision is now accepted and ready to land.Mar 20 2026, 12:44 AM

Closed by commit rG14b9955e57cc: libcrypto: compile all PIC objects with -DOPENSSL_PIC (authored by ngie). · Explain WhyMar 20 2026, 12:52 AM

This revision was automatically updated to reflect the committed changes.

ngie added a commit: rG14b9955e57cc: libcrypto: compile all PIC objects with -DOPENSSL_PIC.

ngie added a reverting change: rG67b918e03475: Revert "libcrypto: compile all PIC objects with -DOPENSSL_PIC".Mar 20 2026, 8:12 PM

ngie added a commit: rG3797fe720a37: libcrypto: compile all PIC objects with -DOPENSSL_PIC -fPIC.Mar 22 2026, 1:39 AM

ngie added a commit: rG689cc820446a: libcrypto: compile all PIC objects with -DOPENSSL_PIC -fPIC.Sat, Apr 25, 4:08 PM

Revision Contents
Changeset List

Path

Size

secure/

lib/

libcrypto/

modules/

2 lines

Diff 173963

secure/lib/libcrypto/modules/Makefile.inc

Loading...