Page MenuHomeFreeBSD

lang/crystal: fix broken base SSL usage
ClosedPublic

Authored by dch on Aug 6 2021, 2:54 PM.
Tags
None
Referenced Files
F99650862: D31435.id93407.diff
Fri, Oct 11, 6:32 PM
F99638303: D31435.id93408.diff
Fri, Oct 11, 3:32 PM
F99618215: D31435.diff
Fri, Oct 11, 9:37 AM
Unknown Object (File)
Thu, Oct 10, 5:12 PM
Unknown Object (File)
Thu, Oct 10, 3:24 PM
Unknown Object (File)
Thu, Oct 10, 8:00 AM
Unknown Object (File)
Thu, Oct 10, 4:54 AM
Unknown Object (File)
Thu, Oct 10, 12:45 AM

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dch requested review of this revision.Aug 6 2021, 2:54 PM

If this is made into an extra-patch conditional on ${SSL_DEFAULT} == base, this could be applied until there's an upstream solution, I guess

  • move hardcoded patches into EXTRA_PATCHES
  • apply only when base ssl is required
  • do substitution as required for various FreeBSD releases
  • 14/CURRENT could conceivably change, but not clear what value to pick
  • hard code LIBRESSL version as its not present in base

If we're applying an extra-patch and then doing a REINPLACE_CMD, maybe we could just do a REINPLACE_CMD directly just against the 0.0.0 and the ldflags?

Also, you MUST NOT support ${OSVERSION} < 1200000! There's a clue as to why in the BOOTSTRAP_PATH variable ;)

Use OSREL:R for clarity, we don't require a specific kernel version, as
we don't require a specific kernel, just track FreeBSD version.

dch retitled this revision from lang/crystal: thought experiment on handling base SSL to lang/crystal: fix broken base SSL usage.Aug 9 2021, 11:24 AM
dch edited the test plan for this revision. (Show Details)
dch added a reviewer: val_packett.cool.
In D31435#709288, @greg_unrelenting.technology wrote:

If we're applying an extra-patch and then doing a REINPLACE_CMD, maybe we could just do a REINPLACE_CMD directly just against the 0.0.0 and the ldflags?

Yes I tried this earlier and in the end, a multi-line sed match just becomes unwieldy and messy.
These files are unlikely to be altered much, until we or upstream find a suitable permanent solution.

Also, you MUST NOT support ${OSVERSION} < 1200000! There's a clue as to why in the BOOTSTRAP_PATH variable ;)

fixed, thanks!

Patches work fine with the current version, just fix the distversion/portrevision conflict when rebasing

This revision is now accepted and ready to land.Feb 25 2022, 12:24 PM
This revision was automatically updated to reflect the committed changes.