On Mon, 20 Sep 2010, Tim Cutts wrote: > On 14 Sep 2010, at 9:34 pm, Don Armstrong wrote: > > On Tue, 14 Sep 2010, Julien Cristau wrote: > >> On Mon, Sep 13, 2010 at 23:09:55 -0700, Don Armstrong wrote: > > [...] > >>> + * Non Maintainer Upload > >>> + * Include linux/nfs_mount.h in conf/mount/linux_mount.c (Closes: > >>> #560528) > > [...] > >>> + #endif /* HAVE_CONFIG_H */ > >>> + #include <am_defs.h> > >>> + #include <amu.h> > >>> ++#ifndef NFS_MOUNT_VERSION > >>> ++#define NFS_MOUNT_VERSION 4 > >>> ++#endif > >> > >> The changelog and patch seem to say different things? > > > > Oops. Yeah, I fixed it one way to start with, then backed out and > > fixed it the other way, and forgot to change the changelog. > > I will try to get a maintainer upload done this week. It looks like > this may have caused older problems to re-emerge, though, judging by > Richard Kettlewell's subsequent bug report. The problem historically > was that because am-utils is essentially dead upstream, it's never > been able to cope with NFS_MOUNT_VERSION higher than 4; it doesn't > populate the extra struct fields, and weird things start happening.
Unless I'm totally misreading the code, the above change doesn't alter the fix made for #496060. [That's why I set NFS_MOUNT_VERSION to 4, not the actual current value of 7.] Don Armstrong -- In all matters of government, the correct answer is usually: "Do nothing" -- Robert Heinlein _Time Enough For Love_ p428 http://www.donarmstrong.com http://rzlab.ucr.edu -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
