emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: patch to include EasyPG to Emacs


From: Dan Nicolaescu
Subject: Re: patch to include EasyPG to Emacs
Date: Sun, 10 Feb 2008 14:36:09 -0800

Daiki Ueno <address@hidden> writes:


  > ! @c To install, just follow the standard CMMI installation instructions.

CMMI?


  > ! @c @cartouche
  > ! @c @example
  > ! @c $ ./configure
  > ! @c $ sudo make install
  > ! @c @end example
  > ! @c @end cartouche

Not needed given that it's installed by default now.

  > ! @c @noindent
  > ! @c Then, add the following line to your @file{~/.emacs}
  > ! 
  > ! Add the following line to your @file{~/.emacs}.
  >   
  >   @cartouche
  >   @lisp
  > ! (epa-mode 1)

Is this still needed now that epa is part of emacs?
If yes, can't it be done with customize instead of editing .emacs?
Ideally the user should have to do nothing by default...



  >   @end lisp
  >   @end cartouche
  >   
  >   @noindent
  > ! Restart emacs and type @kbd{M-x epa- @key{TAB}}, 


  > and you will see a lot of commands available.
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Omit, not very useful information for a user.

+ (defconst epa-menu-items
+   '("EasyPG Assistant"

Please replace this with something more obvious to someone that does not
know what "EasyPG Assistant" is.  Maybe something like "Encryption/Decryption"? 


  > + ;;;###autoload
  > + (define-minor-mode epa-menu-mode
  > +   "Minor mode to hook EasyPG into the menu-bar."

Isn't this overkill, a global minor mode just to hook into the menu-bar?

Hope this helps...




reply via email to

[Prev in Thread] Current Thread [Next in Thread]