Differential D40662

bsd.sys.mk: Stop muting -Wunused-but-set-variable for GCC.
ClosedPublic
Actions

Authored by jhb on Jun 20 2023, 4:34 PM.

Details

Reviewers

emaste

Commits

rG39240ddf854f: bsd.sys.mk: Stop muting -Wunused-but-set-variable for GCC.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 52197
Build 49088: arc lint + arc unit

Event Timeline

jhb created this revision.Jun 20 2023, 4:34 PM

Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptJun 20 2023, 4:34 PM

jhb requested review of this revision.Jun 20 2023, 4:34 PM

Harbormaster completed remote builds in B52197: Diff 123537.Jun 20 2023, 4:34 PM

jhb added a parent revision: D40661: tests/fusefs: Suppress false warning from GCC..Jun 20 2023, 4:34 PM

jhb added a child revision: D40663: bsd.sys.mk: Add NO_WUNUSED_BUT_SET_VARIABLE helper variable..

jhb added a reviewer: emaste.Jun 20 2023, 4:39 PM

Do we have a cirrus-ci run with this stack, out of curiosity?

This revision is now accepted and ready to land.Jun 20 2023, 4:54 PM

In D40662#925563, @emaste wrote:

Do we have a cirrus-ci run with this stack, out of curiosity?

No, I have just done local builds on my desktop (GCC 12 for amd64 and clang for tinderbox)

Closed by commit rG39240ddf854f: bsd.sys.mk: Stop muting -Wunused-but-set-variable for GCC. (authored by jhb). · Explain WhyJun 28 2023, 6:53 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG39240ddf854f: bsd.sys.mk: Stop muting -Wunused-but-set-variable for GCC..

Revision Contents
Changeset List

Path

Size

share/

mk/

bsd.sys.mk

1 line

Diff 123537

View Options

bsd.sys.mk: Stop muting -Wunused-but-set-variable for GCC.ClosedPublicActions