Attention is currently required from: Edward O'Callaghan, Anastasia Klimchuk, Peter Marheine.
View Change
3 comments:
File meson.build:
Patch Set #1, Line 431: lib
must stay 'flashrom', now it's liblibflashrom
Done
Patch Set #1, Line 477: # we can't just link_with libflashrom as we require all the internal symbols...
Obsolete comment?
Done
Patch Set #1, Line 489: link_with : libflashrom.get_static_lib(), # flashrom needs internal symbols of libflashrom
This will fail if default_library is set to anything but 'both'. […]
For now both_libraries will be fine. We might be reworking this when adding support for libpayload.
To view, visit change 64030. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic522610f59e00299ebfa1bd29482ff92120ec52b
Gerrit-Change-Number: 64030
Gerrit-PatchSet: 2
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Peter Marheine <pmarheine@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-Attention: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Attention: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Attention: Peter Marheine <pmarheine@chromium.org>
Gerrit-Comment-Date: Fri, 06 May 2022 21:21:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Thomas Heijligen <src@posteo.de>
Comment-In-Reply-To: Peter Marheine <pmarheine@chromium.org>
Gerrit-MessageType: comment