Page MenuHomeFreeBSD

dtc: Sync with upstream commit 26a0fe5
ClosedPublic

Authored by jlduran_gmail.com on Aug 16 2023, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 10, 7:37 AM
Unknown Object (File)
Wed, May 8, 12:46 AM
Unknown Object (File)
Wed, May 8, 12:46 AM
Unknown Object (File)
Wed, May 8, 12:45 AM
Unknown Object (File)
Wed, May 8, 12:45 AM
Unknown Object (File)
Wed, May 8, 12:45 AM
Unknown Object (File)
Tue, May 7, 8:19 PM
Unknown Object (File)
Mon, May 6, 2:10 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_gmail.com 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?

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.