Message ID | 20181203001936.8284-1-stewart@linux.ibm.com |
---|---|
State | Accepted |
Headers | show |
Series | ffspart: Increase MAX_LINE to above PATH_MAX | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | master/apply_patch Successfully applied |
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot | success | Test snowpatch/job/snowpatch-skiboot on branch master |
On Mon, 2018-12-03 at 11:19 +1100, Stewart Smith wrote: > Otherwise we saw failures in CI and the ~221 character paths Jankins > likes to have. > > Signed-off-by: Stewart Smith <stewart@linux.ibm.com> > --- > external/ffspart/ffspart.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/external/ffspart/ffspart.c b/external/ffspart/ffspart.c > index 179ed582170b..eeee0d4d657b 100644 > --- a/external/ffspart/ffspart.c > +++ b/external/ffspart/ffspart.c > @@ -50,7 +50,7 @@ > * Plus \n 40 > * Lets do 50. > */ > -#define MAX_LINE 255 > +#define MAX_LINE (PATH_MAX+255) > #define MAX_TOCS 10 > #define SEPARATOR ',' > Reviewed-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>
Stewart Smith <stewart@linux.ibm.com> writes: > Otherwise we saw failures in CI and the ~221 character paths Jankins > likes to have. > > Signed-off-by: Stewart Smith <stewart@linux.ibm.com> > --- > external/ffspart/ffspart.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Merged to master as of 7ac3a779319015a3a1c6b6f6ef9bbb106dcc9223
diff --git a/external/ffspart/ffspart.c b/external/ffspart/ffspart.c index 179ed582170b..eeee0d4d657b 100644 --- a/external/ffspart/ffspart.c +++ b/external/ffspart/ffspart.c @@ -50,7 +50,7 @@ * Plus \n 40 * Lets do 50. */ -#define MAX_LINE 255 +#define MAX_LINE (PATH_MAX+255) #define MAX_TOCS 10 #define SEPARATOR ','
Otherwise we saw failures in CI and the ~221 character paths Jankins likes to have. Signed-off-by: Stewart Smith <stewart@linux.ibm.com> --- external/ffspart/ffspart.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)