From d4d28f2ffe522d2509dccfa7be0eef709732d93f Mon Sep 17 00:00:00 2001
From: Jake Leahy <jake@leahy.dev>
Date: Fri, 3 Mar 2023 06:27:10 +1100
Subject: [PATCH] Allow `futureLogging` in release builds (#21448)

* Add test case

* Fixes 21447: Keeps stackTrace around when using futureLogging

* Remove extra whitespace
---
 lib/pure/asyncfutures.nim | 2 +-
 tests/async/t21447.nim    | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)
 create mode 100644 tests/async/t21447.nim

diff --git a/lib/pure/asyncfutures.nim b/lib/pure/asyncfutures.nim
index edf5f7277b..51aaaca367 100644
--- a/lib/pure/asyncfutures.nim
+++ b/lib/pure/asyncfutures.nim
@@ -29,7 +29,7 @@ type
     finished: bool
     error*: ref Exception              ## Stored exception
     errorStackTrace*: string
-    when not defined(release):
+    when not defined(release) or defined(futureLogging):
       stackTrace: seq[StackTraceEntry] ## For debugging purposes only.
       id: int
       fromProc: string
diff --git a/tests/async/t21447.nim b/tests/async/t21447.nim
new file mode 100644
index 0000000000..e4f7ae31f6
--- /dev/null
+++ b/tests/async/t21447.nim
@@ -0,0 +1,6 @@
+discard """
+  action: "compile"
+  cmd: "nim c -d:release -d:futureLogging $file"
+"""
+
+import std/asyncdispatch