Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Nov 2020 13:30:03 +0000 (UTC)
From:      Mathieu Arnold <mat@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r554894 - head/Mk/Scripts
Message-ID:  <202011111330.0ABDU3RK035945@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mat
Date: Wed Nov 11 13:30:03 2020
New Revision: 554894
URL: https://svnweb.freebsd.org/changeset/ports/554894

Log:
  Ignore grep return value.
  
  Most of the time, we use grep in a test, where its return value is
  important.  In this instance, it is only used for filtering, and we do
  not care about its return value, so ignore it.
  
  PR:             250723
  Exp-run by:     antoine
  Differential Revision:  https://reviews.freebsd.org/D27007

Modified:
  head/Mk/Scripts/create-manifest.sh   (contents, props changed)

Modified: head/Mk/Scripts/create-manifest.sh
==============================================================================
--- head/Mk/Scripts/create-manifest.sh	Wed Nov 11 13:29:52 2020	(r554893)
+++ head/Mk/Scripts/create-manifest.sh	Wed Nov 11 13:30:03 2020	(r554894)
@@ -63,7 +63,8 @@ EOT
 
 # Then the key/values sections
 echo "deps: { "
-eval ${dp_ACTUAL_PACKAGE_DEPENDS} | grep -v -E ${dp_PKG_IGNORE_DEPENDS} | sort -u
+# Ignore grep's return value.
+eval ${dp_ACTUAL_PACKAGE_DEPENDS} | { grep -v -E ${dp_PKG_IGNORE_DEPENDS} || :; } | sort -u
 echo "}"
 
 echo "options: {"



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202011111330.0ABDU3RK035945>