Fix message suffix due to warnings with dump-logs: warning

mpilgrem · mpilgrem · commit 11406e50abda · 2023-09-17T19:06:37.000+01:00
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -14,6 +14,9 @@ Other enhancements:
 
 Bug fixes:
 
+* Restore message suffix `due to warnings` with `dump-logs: warning` (broken
+  with Stack 2.11.1).
+
 ## v2.13.0.1 (release candidate) - 2023-09-16
 
 Release notes:
diff --git a/src/Stack/Build/Execute.hs b/src/Stack/Build/Execute.hs
@@ -578,7 +578,7 @@ withExecuteEnv bopts boptsCli baseConfigOpts locals globalPackages snapshotPacka
          fillSep
            ( ( fillSep
                  ( flow "Dumping log file"
-                 : [ flow msgSuffix | L.null msgSuffix ]
+                 : [ flow msgSuffix | not (L.null msgSuffix) ]
                  )
              <> ":"
              )

Original file line number	Diff line number	Diff line change
`@@ -578,7 +578,7 @@ withExecuteEnv bopts boptsCli baseConfigOpts locals globalPackages snapshotPacka`
`578`	`578`	`fillSep`
`579`	`579`	`( ( fillSep`
`580`	`580`	`( flow "Dumping log file"`
`581`		`- : [ flow msgSuffix \| L.null msgSuffix ]`
	`581`	`+ : [ flow msgSuffix \| not (L.null msgSuffix) ]`
`582`	`582`	`)`
`583`	`583`	`<> ":"`
`584`	`584`	`)`