Code review comment for ~ahasenack/ubuntu/+source/openldap:eoan-openldap-2.4.38-merge

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I'd expect to squash
Enable nss (and all it does)
with the later added
  49 - d/p/contrib-makefiles: given the change in 2.4.47+dfsg-3 regarding
  50 Debian bug #919136, we also have to patch the nssov makefile
  51 accordingly and thus update this patch.

Otherwise this will get ever longer.
I think we can squash the commits and jus omit the later message. maybe becoming one line
 "patch the nssov makefile"

There is enough Delta already, streamlining this a bit will make it more readable.
Also if we'd drop (or upstream) nss, then the nssov changes would belong to the same.

review: Needs Fixing

« Back to merge proposal