Code review comment for ~ahasenack/ubuntu/+source/autofs:cosmic-merge-5.1.2-3

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

Hi,
tags LGTM
dropped changes: ack
retained changes: correctly retained

So this is all good, but has some optional things we might want to look into on this merge.

I wonder on the two older kept changes, both about 3 years old:
+ - Add LDFLAGS to avoid issues with accessing global variables in
+ shared libraries (LP #1470687, LP #1472115).
+ - d/p/fix-lookup-ldap-crash.patch: fix crash on startup due to uninitialized
+ config.
There should be either of the following:
a) dropping the code, build it and retest the old bugs - still triggers -> submit to Debian as well
b) dropping the code, build it and retest the old bugs - no more triggers -> Drop Delta

On the crash I'd assume it still appears and needs to be reported.
On the LDFLAGS instead I doubt this still really works.
First our build log [1] still has symbolic-functions while Debian [2] has not.
So the fix might be incomplete these days or is it no more important?

Anyway I'd appreciate if you would look into those two Deltas.

Also you mentioned segfaults when building that on Ubuntu - was that riddle solved as well?

Finally I have two super-minor changelog nit picks that you can consider (or not) depending on your mood.

[1]: https://launchpadlibrarian.net/365726198/buildlog_ubuntu-bionic-amd64.autofs_5.1.2-1ubuntu3_BUILDING.txt.gz
[2]: https://buildd.debian.org/status/fetch.php?pkg=autofs&arch=amd64&ver=5.1.2-3&stamp=1518905129&raw=0

review: Needs Information

« Back to merge proposal