Fossil SCM

Fix a logic error in processing of the FOSSIL_COLOR environment variable.

florian 2025-04-19 04:20 standard-cli-colors
Commit 6cb7a7e28dfd13afd904710111c29d105ff8f2168a2b76c27cf722543f95050d
1 file changed +2 -3
+2 -3
--- src/main.c
+++ src/main.c
@@ -689,13 +689,12 @@
689689
}
690690
}
691691
g.cliColor = "\033[91m";
692692
zEnvVar = fossil_getenv("FOSSIL_COLOR");
693693
if( zEnvVar ){
694
- if( fossil_strcmp(zEnvVar,"none")==0 &&
695
- g.colorOutput==COLOR_VT_UNSET ){
696
- g.colorOutput = COLOR_VT_NEVER;
694
+ if( fossil_strcmp(zEnvVar,"none")==0 ){
695
+ if( g.colorOutput==COLOR_VT_UNSET ) g.colorOutput = COLOR_VT_NEVER;
697696
}else{
698697
int i, fValid = 1;
699698
/* Rudimentary sanity check: only allow digits and semicolon. */
700699
for( i=0; zEnvVar[i]; i++ ){
701700
if( !strchr("0123456789;",zEnvVar[i]) ) fValid = 0;
702701
--- src/main.c
+++ src/main.c
@@ -689,13 +689,12 @@
689 }
690 }
691 g.cliColor = "\033[91m";
692 zEnvVar = fossil_getenv("FOSSIL_COLOR");
693 if( zEnvVar ){
694 if( fossil_strcmp(zEnvVar,"none")==0 &&
695 g.colorOutput==COLOR_VT_UNSET ){
696 g.colorOutput = COLOR_VT_NEVER;
697 }else{
698 int i, fValid = 1;
699 /* Rudimentary sanity check: only allow digits and semicolon. */
700 for( i=0; zEnvVar[i]; i++ ){
701 if( !strchr("0123456789;",zEnvVar[i]) ) fValid = 0;
702
--- src/main.c
+++ src/main.c
@@ -689,13 +689,12 @@
689 }
690 }
691 g.cliColor = "\033[91m";
692 zEnvVar = fossil_getenv("FOSSIL_COLOR");
693 if( zEnvVar ){
694 if( fossil_strcmp(zEnvVar,"none")==0 ){
695 if( g.colorOutput==COLOR_VT_UNSET ) g.colorOutput = COLOR_VT_NEVER;
 
696 }else{
697 int i, fValid = 1;
698 /* Rudimentary sanity check: only allow digits and semicolon. */
699 for( i=0; zEnvVar[i]; i++ ){
700 if( !strchr("0123456789;",zEnvVar[i]) ) fValid = 0;
701

Keyboard Shortcuts

Open search /
Next entry (timeline) j
Previous entry (timeline) k
Open focused entry Enter
Show this help ?
Toggle theme Top nav button