Code review comment for ~ahasenack/ubuntu/+source/nfs-utils:lunar-nfs-utils-merge

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, Andreas.

Results: (from http://autopkgtest.ubuntu.com/results/autopkgtest-lunar-ahasenack-lunar-nfs-utils-merge/?format=plain)
  nfs-utils @ amd64:
    19.02.23 19:23:40 Log 🗒️ ✅ Triggers: nfs-utils/1:2.6.2-4ubuntu1~ppa1
  nfs-utils @ arm64:
    19.02.23 21:16:36 Log 🗒️ ✅ Triggers: nfs-utils/1:2.6.2-4ubuntu1~ppa1
  nfs-utils @ armhf:
    19.02.23 18:51:42 Log 🗒️ ✅ Triggers: nfs-utils/1:2.6.2-4ubuntu1~ppa1
  nfs-utils @ ppc64el:
    19.02.23 19:06:40 Log 🗒️ ✅ Triggers: nfs-utils/1:2.6.2-4ubuntu1~ppa1
  nfs-utils @ s390x:
    19.02.23 18:58:25 Log 🗒️ ✅ Triggers: nfs-utils/1:2.6.2-4ubuntu1~ppa1

Tests are OK. Package builds fine. The commits look good.

Regarding the unresponded Debian MRs, I believe it's worth filing bugs against the package too. Something I also do is ping the maintainer in the MR; sometimes they miss the notifications.

LGTM, +1.

review: Approve

« Back to merge proposal