From: "Steinar H. Gunderson" <sgunderson@bigfoot.com>
To: autofs@linux.kernel.org
Subject: Re: [ANNOUNCE] autofs 4.1.4 beta1 release
Date: Wed, 26 Jan 2005 16:13:44 +0100 [thread overview]
Message-ID: <20050126151344.GA841@uio.no> (raw)
In-Reply-To: <Pine.LNX.4.61.0501262151370.1716@donald.themaw.net>
On Wed, Jan 26, 2005 at 10:17:53PM +0800, raven@themaw.net wrote:
> So let me know when you find more and I'll fix`em as quick as I can!
Looks like you missed a patch or two from the Debian archive, at least:
- Quite recently, I uploaded a patch for the debian/event.d script, but this
is probably not relevant for the upstream distribution.
- There is a patch for rc.autofs.in missing, changing an "else" to en "elif".
The idea is that failure in loading the autofs4 module should not break
first-time installation (see http://bugs.debian.org/280276).
- There seem to have been a lot of changes in rc.autofs.in (especially
related to timeout parsing); I'm unable to follow them all, but I hope
you've done the right thing in merging in the Debian patches here :-)
- The rc.autofs.in searches for "^automount: ", which breaks if a tab is used
instead of a space (see http://bugs.debian.org/277320).
- I'm also having a hard time figuring out whether parse_sun.c does the right
thing wrt. handling whitespace at the end of maps or not; the code looks
different, at least, but I think it's broken for non-multimaps. Could you
verify? (Of course, the simplest thing is probably to test :-) )
- I can't see if lookup_file.c handles maps without a trailing newline now
(Debian patch 051_maps_without_trailing_newline.diff has a hackish fix
which is obviously not applied, but you might have implemented it
differently).
Apart from those, it appears like everything else from Debian's 4.1.3 package
is in, at least -- which removes the need for twenty or so local patches :-)
/* Steinar */
--
Homepage: http://www.sesse.net/
next prev parent reply other threads:[~2005-01-26 15:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-01-26 14:17 [ANNOUNCE] autofs 4.1.4 beta1 release raven
2005-01-26 15:13 ` Steinar H. Gunderson [this message]
2005-01-26 15:35 ` raven
2005-01-26 16:14 ` Steinar H. Gunderson
2005-01-27 1:21 ` Ian Kent
2005-01-27 2:02 ` Steinar H. Gunderson
2005-01-27 12:12 ` raven
-- strict thread matches above, loose matches on Subject: below --
2005-01-26 16:33 Rigler, Stephen C.
2005-01-27 1:24 ` Ian Kent
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20050126151344.GA841@uio.no \
--to=sgunderson@bigfoot.com \
--cc=autofs@linux.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.