Code review comment for lp://qastaging/~apparmor-dev/apparmor/apparmor-ubuntu-citrain.lp1295816

Hello,

Am Freitag, 21. März 2014 schrieb Jamie Strandboge:
> Jamie Strandboge has proposed merging
> lp:~apparmor-dev/apparmor/apparmor-ubuntu-citrain.lp1295816 into
> lp:~apparmor-dev/apparmor/apparmor-ubuntu-citrain.

> Bug #1295816 in apparmor (Ubuntu): "/etc/init.d/apparmor reload
> complains if /var/lib/apparmor/profiles doesn't have profiles"
> https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/1295816

> === modified file 'debian/lib/apparmor/functions'
> --- debian/lib/apparmor/functions 2013-10-08 21:30:52 +0000
> +++ debian/lib/apparmor/functions 2014-03-21 20:17:10 +0000
> @@ -37,7 +37,7 @@
> if [ ! -d "$pdir" ]; then
> continue
> fi
> - num=`find "$pdir" -type f | wc -l`
> + num=`find "$pdir" -type f ! -regex '.*\.md5sums$' | wc -l`

I doubt a regex is needed. Instead, you could use
    find [...] -not -name '*.md5sum'

Regards,

Christian Boltz
--
[Serieller Anschluss] Mittlerweile ist (fast) alles USB. OK, es gibt
auch Adapter, aber die kosten wieder extra. Und der Rechner soll später
nicht aussehen wie ein Tannenbaum, mit einem Bündel herabhängender
Adaptern anstatt Lametta. [Ralph Müller in suse-linux]