Pike-devel November 2014

pike-devel@lists.lysator.liu.se

7 participants
12 discussions

Re: Proposal: Enhance Process.run() to accept functions for stdout/stderr
by Chris Angelico 03 Nov '14

03 Nov '14

On Mon, Nov 3, 2014 at 10:55 PM, Per Hedbor () @ Pike (-) developers forum <10353(a)lyskom.lysator.liu.se> wrote: >>Any views on this feature? (I just rebased the branch on top of >>current 8.1, no other changes.) Fears of breakage? Interest in using >>it? Anything? > > It looks good to me. And, yes, it is worth adding since it is > something you often want. Cool. Merged into 8.1. Should I put it into 8.0 as well? ChrisA

2 1

Proposal: Enhance Process.run() to accept functions for stdout/stderr
by Chris Angelico 03 Nov '14

03 Nov '14

I periodically find myself needing something similar to Process.run(), but processing intermediate output. Example: https://github.com/Rosuav/FrozenOST/blob/master/build.pike#L114 It needs to take whatever's sent to stderr, turn all \n into \r, and send it on to the real stdout. (Yes, it sends stderr to stdout.) I usually end up lifting the code of Process.run() and making a cut-down version (as I don't need all its features); but the obvious alternative would be to have Process.run itself provide this facility. The code is now at branch rosuav/process_run_with_func; the equivalent to my above code would be simply: Process.run(({"sox","-S","-m","-v",".5"})+tracklist/1*({"-v",".5"})+({soundtrack}), (["stderr":lambda(string data) {write(replace(data,"\n","\r"));}])); Is this a feature worth adding? ChrisA

2 2

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

Pike-devel November 2014