Hi,
I've attached a patch to do the lockmbox() after open()ing and
lockf()ing the mailbox, and conversely to unlockmbox() before
close()ing it. Looking at the strace, it now goes...
open("/var/mail/edam", O_WRONLY|O_APPEND) = 4
fcntl64(4, F_SETLKW, {type=F_WRLCK, whence=SEEK_CUR, start=0, len=0}) = 0
link("/var/mail/.lk06598ddebian-dev", "/var/mail/edam.lock") = 0
...
unlink("/var/mail/edam.lock") = 0
close(4) = 0
Someone with more experience of locking should verify there are no
problems with this. :o)
--
Tim Marston
ed.am
Index: en/upgrading.dbk
===================================================================
--- en/upgrading.dbk (revision 9557)
+++ en/upgrading.dbk (working copy)
@@ -1227,6 +1227,37 @@
may be required, either before or during the upgrade; these are detailed
below on a per-package basis.
</para>
+<section id="issues-sudo">
+<title>Sudo</title>
+<para>
+If you have modified <filename>/etc/sudoers</filename> then you should be aware
+of changes made to how <systemitem role="package">sudo</systemitem>
+configuration is handled. The default <filename>/etc/sudoers</filename> now
+includes the following two directives:
+</para>
+<programlisting>Defaults
secure_path="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"</programlisting>
+<programlisting>#includedir /etc/sudoers.d</programlisting>
+<para>
+Neither of these entries are added to your <filename>/etc/sudoers</filename>
+automatically during the upgrade. (Although you will still be able to run
+<command>sudo</command> commands by specifying their fully-qualified path.) So
+you might wish to consider migrating your changes to the new
+<filename>/etc/sudoers.d</filename> directory and using the default
+<filename>/etc/sudoers</filename> file. For example:
+</para>
+<screen>
+# mv /etc/sudoers /etc/sudoers.d/mychanges
+# mv /etc/sudoers.dpkg-new /etc/sudoers
+</screen>
+<para>
+You may also need to edit your <filename>/etc/sudoers.d/mychanges</filename> to
+remove unwanted <literal>Defaults</literal> and <literal>#includedir</literal>
+entries. You should use <command>visudo</command> for this:
+</para>
+<screen>
+# visudo -f /etc/sudoers.d/mychanges
+</screen>
+</section>
<section id="issues-evolution">
<title>Evolution</title>
<para>
