Page MenuHomeFreeBSD

dtc: Sync with upstream commit 26a0fe5
ClosedPublic

Authored by jlduran on Aug 16 2023, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 1, 1:39 PM
Unknown Object (File)
Thu, Jun 25, 3:02 AM
Unknown Object (File)
Wed, Jun 24, 11:24 PM
Unknown Object (File)
Wed, Jun 24, 11:49 AM
Unknown Object (File)
Tue, Jun 23, 4:22 AM
Unknown Object (File)
Sun, Jun 21, 6:56 PM
Unknown Object (File)
Fri, Jun 19, 1:27 AM
Unknown Object (File)
Thu, Jun 18, 10:56 PM

Details

Summary
  • 0206c0f ("Handle top-level /delete-node/ directives.")
  • d612a9e ("Remove C++11 standard constrain")
  • Remove extra white lines after the $FreeBSD$ tag removal

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

  • Rebase (remove FreeBSD RCSIDs).
This revision is now accepted and ready to land.Aug 17 2023, 2:19 AM
jlduran edited the summary of this revision. (Show Details)
  • Remove extra white lines after the $FreeBSD$ tag removal
This revision now requires review to proceed.Aug 17 2023, 2:29 AM
This revision is now accepted and ready to land.Aug 17 2023, 8:21 AM
This revision was automatically updated to reflect the committed changes.

I noticed this change broke macOS cross-builds on GitHub. I guess due to the fact that man clang on macOS says: The default C++ language standard is gnu++98.

Specifying a C standard in the Makefile (CXXSTD= c++14) could do the trick, but I would prefer to set it globally or on the GitHub runner itself. Suggestions?

In D41482#957685, @jlduran_gmail.com wrote:

I would prefer to set it globally or on the GitHub runner itself. Suggestions?

If the default C++ standard in base system Clang / Clang 15/16 doesn't match what's in macOS Clang then I suspect we should set it in the cross-build script or actions config.