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)
Mon, Jan 27, 5:01 PM
Unknown Object (File)
Mon, Jan 27, 4:45 PM
Unknown Object (File)
Fri, Jan 24, 5:55 PM
Unknown Object (File)
Fri, Jan 24, 4:49 AM
Unknown Object (File)
Thu, Jan 23, 6:36 PM
Unknown Object (File)
Sat, Jan 11, 6:05 AM
Unknown Object (File)
Wed, Jan 1, 3:05 PM
Unknown Object (File)
Nov 8 2024, 1:00 AM

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.