pkg_libchk <package> is broken ?

[Kevin Oberman]

On Wed, Feb 26, 2014 at 10:32 PM, Sergey V. Dyatko <tiger at agava.com> wrote:

> On Thu, 27 Feb 2014 09:26:34 +0300
> "Sergey V. Dyatko" <sergey.dyatko at gmail.com> wrote:
>
> > Hi,
> >
> > I use sysutils/bsdadminscripts for years, thanks Dominic. After
> > switching to pkg[ng] I patch pkg_libchk for it ( pkg info with
> > corresponding keys instead pkg_info, etc). After r308906 (IIRC) I
> > revert local my patches. Today I spotted that pkg_libchk <package> is
> > broken, for example:
> >
> > tiger# pkg_libchk -q
> > compat9x-amd64-9.2.902000.201310
> > diablo-jdk-1.6.0.07.02_20
> > festival-2.1
> > opera-12.16
> >
> > tiger# pkg_libchk -n festival\*
> > tiger# pkg_libchk -n festival-2.1
> > tiger#
> >
> seems '\' missing on line 411
>
> --- pkg_libchk.orig     2014-02-27 09:31:31.204449447 +0300
> +++ pkg_libchk  2014-02-27 09:31:45.539441534 +0300
> @@ -407,7 +407,7 @@
>  # Get the packages to work on.
>  test -z "$packages" && packages="-a"
>  packages="$(pkg info -E $packages)"
> -test -z "$recursive" -a -z "$Recursive" || packages="$packages
> +test -z "$recursive" -a -z "$Recursive" || packages="$packages \
>  $(pkg info -q $recursive $Recursive "$packages" 2> /dev/null | \
>  sed -E 's|^@pkgdep[[:space:]]*||1')"
>
> --
> wbr, tiger
>

Isn't a '\' also needed on line 417?
-- 
R. Kevin Oberman, Network Engineer, Retired
E-mail: rkoberman at gmail.com