New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 825308 link

Starred by 1 user
Status: Fixed
Owner:
gwendal@chromium.org
Closed: Apr 2018
Cc:
reinauer@chromium.org
█ pgeorgi@chromium.org
adurbin@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

flashrom : compilation error: PATH_MAX not defined in linux_mtd.c

Project Member Reported by gwendal@chromium.org, Mar 23 2018 
I see the error locally and on the pre-cq builder.
https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/no_vmtest_pre_cq/246667

The https://chromium-review.googlesource.com/458921 needs to be extended to linux_mtd.c as well.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 27 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/flashrom/+/db133583f9f32ed62068513182ddb2549eefe4e9

commit db133583f9f32ed62068513182ddb2549eefe4e9
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Tue Mar 27 21:58:16 2018

linux-mtd: Include limits header for specific defines

PATH_MAX is defined in the limits header and used in the code.

[Follow up of CL:458921]

BUG= chromium:825308 
BRANCH=none
TEST=flashrom builds properly

Change-Id: I594d82aa5d5e07d21d90a16a066217d7351e03e0
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/978655
Reviewed-by: Stefan Reinauer <reinauer@google.com>

[modify] https://crrev.com/db133583f9f32ed62068513182ddb2549eefe4e9/linux_mtd.c

Comment 2 by gwendal@chromium.org, Apr 6 2018

Status: Fixed (was: Started)
►

Sign in to add a comment