Uhh... context? I rather like my macro and would prefer to keep it unless there are real issues with it that cannot be solved otherwise. Can you explain how it "broke master quite spectacularly"?

see https://qa.coreboot.org/job/coreboot-gerrit/121768/testReport/junit/(root)/board/GOOGLE_SCARLET/ for the breakage it caused

the non-functional spdx license header changes caused some line numbers to shift which resulted in the same symbol being generated twice by the macro, which made the build fail. so yes, i don't like that macro too much. Maybe add a __COUNTER__ instead of the tag added in https://review.coreboot.org/c/coreboot/+/40140 as a hotfix, that made the tree build again?

View Change

To view, visit change 40141. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I06f825e045d3305dd8ab81311fd2d003d03ae43a
Gerrit-Change-Number: 40141
Gerrit-PatchSet: 5
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Mon, 06 Apr 2020 20:12:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment