[ovs-dev] [PATCH 1/2] autoconf: Don't die on grep error.

Ben Pfaff blp at nicira.com
Wed Nov 3 00:05:20 UTC 2010


On Tue, Nov 02, 2010 at 04:53:09PM -0700, Jesse Gross wrote:
> The most likely cause of an error when grepping for a compatibility
> string is that the file doesn't exist.  Since this can be a valid
> result when files are added in later versions, we shouldn't kill
> the build.  Instead, note that the file doesn't exist but otherwise
> treat it as if the string was not found.

It's good to tolerate a missing file, but wouldn't it be better to do so
by checking for it beforehand with "test -r", instead of assuming that
any error from "grep" means a missing file?

Thanks,

Ben.




More information about the dev mailing list