Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[build] update build to use gnu++17 #147

Merged
merged 4 commits into from
Aug 14, 2023
Merged

[build] update build to use gnu++17 #147

merged 4 commits into from
Aug 14, 2023

Conversation

step0035
Copy link
Collaborator

@step0035 step0035 commented Aug 14, 2023 •
edited
Loading

  • matter sdk update requires to use C++17
  • remove old register keywords
  • remove error-mapping from build

@step0035 step0035 requested a review from xshuqun August 14, 2023 02:39
@step0035 step0035 requested a review from AdamZhang0124 August 14, 2023 07:20
@step0035 step0035 mentioned this pull request Aug 14, 2023
Closed
@xshuqun xshuqun merged commit aa6fe51 into main Aug 14, 2023
@xshuqun xshuqun deleted the gnu17 branch August 14, 2023 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xshuqun xshuqun Awaiting requested review from xshuqun

@AdamZhang0124 AdamZhang0124 Awaiting requested review from AdamZhang0124

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@step0035 @xshuqun