Project

General

Profile

Actions

Bug #17801

closed

Cannot use $(runtime.outdir) for outputBinding glob

Added by Peter Amstutz almost 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
CWL
Target version:
Story points:
-
Release relationship:
Auto

Description

User reported regression, was using $(runtime.outdir) to capture the output directory, but this no longer works in 2.2.


Subtasks 1 (0 open1 closed)

Task #17824: Review 17801-runtime-outdir ResolvedPeter Amstutz06/18/2021Actions
Actions #1

Updated by Peter Amstutz almost 3 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz almost 3 years ago

  • Assigned To set to Peter Amstutz
Actions #3

Updated by Peter Amstutz almost 3 years ago

17801-runtime-outdir @ 8655bd8d309537ee7bb9c696c909ebffe3501483

Fix bug & add new integration test.

developer-run-tests: #2540

Actions #4

Updated by Peter Amstutz almost 3 years ago

  • Status changed from New to In Progress
Actions #5

Updated by Michael Crusoe almost 3 years ago

Peter Amstutz wrote:

User reported regression, was using $(runtime.outdir) to capture the output directory, but this no longer works in 2.2.

Please file tickets requesting new conformance tests for all discovered regressions related to CWL.

Example: https://github.com/common-workflow-language/cwl-v1.2/issues/109

Actions #6

Updated by Nico César almost 3 years ago

review @ 8655bd8d309537ee7bb9c696c909ebffe3501483 17801-runtime-outdir

The 1-line change and test look good to me.

I tried to understand a little more about the case that rest will be ".", it seems that having "/." in the path will cause this to happen and being innocuous

Ready to merge

Actions #7

Updated by Peter Amstutz almost 3 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Peter Amstutz over 2 years ago

  • Release set to 39
Actions

Also available in: Atom PDF