Re: [PATCH] dtdiff: change to POSIX shell

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



On 10 Jan 2018 16:57, David Gibson wrote:
> On Wed, Jan 10, 2018 at 12:47:56AM -0500, Mike Frysinger wrote:
> > This changes from the bash-specific process substitution feature to
> > reading with pipes.  It relies on /dev/fd or /proc/self/fd existing.
> > 
> > URL: https://crbug.com/756559
> > Signed-off-by: Mike Frysinger <vapier@xxxxxxxxxxxx>
> 
> Yeah, sorry, replacing a dependency on a complex but widely available
> shell with a much more cryptic dependency on a Linux specific feature
> does not seem like a win to me.

bash's <(...) only works when the host supports either /dev/fd or /proc/self/fd,
so i don't think this is less portable.  /dev/fd also isn't Linux-specific.
-mike

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux