Fix dynamic publish path (#5809)

Signed-off-by: Ben Sherman <bentshermann@gmail.com>
nextflow-io · Feb 24, 2025 · 6a81c01 · 6a81c01
1 parent 18b6934
commit 6a81c01
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/modules/nextflow/src/main/groovy/nextflow/extension/PublishOp.groovy b/modules/nextflow/src/main/groovy/nextflow/extension/PublishOp.groovy
@@ -138,8 +138,8 @@ class PublishOp {
 
         // if the resolved publish path is a string, resolve it
         // against the base output directory
-        if( resolvedPath instanceof String )
-            return outputDir.resolve(resolvedPath)
+        if( resolvedPath instanceof CharSequence )
+            return outputDir.resolve(resolvedPath.toString())
 
         // if the resolved publish path is a closure, use the closure
         // to transform each published file and resolve it against

diff --git a/modules/nextflow/src/test/groovy/nextflow/script/OutputDslTest.groovy b/modules/nextflow/src/test/groovy/nextflow/script/OutputDslTest.groovy
@@ -56,7 +56,7 @@ class OutputDslTest extends Specification {
 
         when:
         dsl.target('bar') {
-            path('barbar')
+            path { v -> "${'barbar'}" }
             index {
                 path 'index.csv'
             }