[Debian-med-packaging] Bug#1134956: routine-update: implementation of check for optional/extra priorities

[Santiago Vila]

Package: routine-update
Version: 0.2.4
Tags: patch

Hello again.

Related with my previous report, I think the check itself is wrong:

if grep -q -e '^Priority:.*optional' -e '^Priority:.*extra' debian/control ; then
   sed -i '/^Priority/d' debian/control

Using such code, this is what it happens: if the control file contains any package
of optional or extra priorities, then all priority fields will be removed,
including, if any, standard, required or important, not what we really want.

I believe the check should be like this instead:

if ! grep '^Priority:' debian/control | grep -q -v -e 'optional' -e 'extra'; then

i.e. if *all* priorities are optional or extra, then remove all priority fields.

Thanks.