all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Glenn Morris <rgm@gnu.org>
To: Bastien Guerry <bzg@altern.org>
Cc: Justus-bulk@Piater.name, emacs-devel@gnu.org
Subject: Re: password-cache for smtpmail.el
Date: Wed, 13 Feb 2008 17:23:48 -0500	[thread overview]
Message-ID: <3qbq6klcln.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <87bq6ko75a.fsf@bzg.ath.cx> (Bastien Guerry's message of "Wed, 13 Feb 2008 21:53:21 +0000")

Bastien Guerry wrote:

> I have tested this, looks fine.

I don't see how it can be, without notice being paid to this comment

  The patch also applies against the latest CVS (MAIN/HEAD) version
  1.97, except that "password-cache" should be autoloaded instead of
  "password" for recent development emacsen.

> I applied the patch.

The ChangeLog entry should be under the name of the patch author, not
yours. You should check for a copyright assignment before installing
code written by others. (See "Summary for the impatient" in
admin/notes/copyright.) Less than 10-15 lines (cumulative, over all
patches) is OK to install without legal papers, but should be marked
as "(tiny change)". See ChangeLog for examples.

And again, you probably want to wait more than 12 hours to allow
people to comment.




  reply	other threads:[~2008-02-13 22:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-13 10:04 password-cache for smtpmail.el Justus-bulk
2008-02-13 21:53 ` Bastien Guerry
2008-02-13 22:23   ` Glenn Morris [this message]
2008-02-13 23:56     ` Juanma Barranquero
2008-02-14  1:58       ` Glenn Morris
2008-02-14  1:59         ` Juanma Barranquero
2008-02-14  2:08           ` Glenn Morris
2008-02-14  2:00       ` Bastien
2008-02-14  2:10         ` Glenn Morris
2008-02-14  1:58     ` Bastien
2008-02-14  8:55     ` Justus-bulk
2008-02-14 15:01       ` Bastien
2008-02-14 14:21 ` Simon Josefsson

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=3qbq6klcln.fsf@fencepost.gnu.org \
    --to=rgm@gnu.org \
    --cc=Justus-bulk@Piater.name \
    --cc=bzg@altern.org \
    --cc=emacs-devel@gnu.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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

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.