Opened 5 years ago

Closed 5 years ago

#2928 closed defect (fixed)

SetPropertyFromCommand shouldn't use stderr

Reported by: vlovich Owned by:
Priority: major Milestone: 0.9.+
Version: 0.8.9 Keywords: simple, steps
Cc:

Description (last modified by dustin)

Having SetPropertyFromCommand include stderr in the property that is set feels like an inflexible default. I think a better approach is to add two new keywords:

includeStdout=True & includeStderr=False.

That way those that need it have a way to trivially include it while the default is likely the default behaviour expected.

At the very least, the current behaviour needs to be documented.

Saw this on 0.8.8 but I'm assuming this is unchanged in 0.8.9 nor master since I haven't seen anything in the changelog.

Change History (2)

comment:1 Changed 5 years ago by dustin

  • Description modified (diff)
  • Keywords simple steps added
  • Milestone changed from undecided to 0.9.+

I don't think this has changed in more recent versions, no.

comment:2 Changed 5 years ago by dustin

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.