Skip to contentSkip to conversation

kconfig: allow kconfig to handle longer path names

PATCH RESEND In review Markus Mayer 2 participants 2 messages Dec 9, 2015 – Dec 10, 2015

Subsystems (2)
Kbuild Kconfig
lore
Share & tools
lore

Markus Mayer

author
#0 lore raw

PATCH RESEND 1/1 kconfig: allow kconfig to handle longer path names

The current (arbitrary) limit of 128 characters for path names has proven too short for Android builds, as longer path names are used there.

Change conf.c, so it can handle path lengths up to PATH_MAX characters.

Signed-off-by Markus Mayer <[email protected]>
---
 scripts/kconfig/conf.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
scripts/kconfig/conf.c +4 -3
index 6c20431..866369f 100644
@@ -5,6 +5,7 @@
 
 #include <locale.h>
 #include <ctype.h>
+#include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

@@ -41,7 +42,7 @@ static int tty_stdio;
 static int valid_stdin = 1;
 static int sync_kconfig;
 static int conf_cnt;
-static char line[128];
+static char line[PATH_MAX];
 static struct menu *rootEntry;
 
 static void print_help(struct menu *menu)

@@ -109,7 +110,7 @@ static int conf_askvalue(struct symbol *sym, const char *def)
 		/* fall through */
 	case oldaskconfig:
 		fflush(stdout);
-		xfgets(line, 128, stdin);
+		xfgets(line, sizeof(line), stdin);
 		if (!tty_stdio)
 			printf("\n");
 		return 1;

@@ -311,7 +312,7 @@ static int conf_choice(struct menu *menu)
 			/* fall through */
 		case oldaskconfig:
 			fflush(stdout);
-			xfgets(line, 128, stdin);
+			xfgets(line, sizeof(line), stdin);
 			strip(line);
 			if (line[0] == '?') {
 				print_help(menu);
Signature · 1 line
1.9.1
#1 lore raw
Quoted 10 lines · Markus Mayer

The current (arbitrary) limit of 128 characters for path names has proven too short for Android builds, as longer path names are used there.

Change conf.c, so it can handle path lengths up to PATH_MAX characters.

Signed-off-by Markus Mayer <[email protected]>
---
 scripts/kconfig/conf.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

Thanks, applied to kbuild.git#kconfig.

Michal