>>>>> "Timo" == Timo Juhani Lindfors <[email protected]> writes:

> Hi,
> [ Sending a copy to upstream mailing list for further comments ]

> I'm bit puzzled here. Isn't this by design? 

>    "It's not auto-loaded so users will have to "(require
>     'magit-simple-keys)" in their .emacs explicitly to use it anyway."

> -- http://comments.gmane.org/gmane.comp.version-control.git.magit/1156

Hi,

this is indeed a bug.
The problem is that 50magit.el has not been properly updated. If you
instead install magit via package.el instead, you would see that
extension code is autoloaded properly.

The proper fix is to generate this file (which is fixed in the 'master'
git branch but didn't hit any release yet).

magit-simple-keys.el is a totally different problem, as it's not part of
magit itself (only a convenient contrib). So in any case, it will have
to be loaded manually (on purpose).

Yann.

> -----Original Message-----
> From: Braun Gábor <[email protected]>
> Sent: Tue, 08 May 2012 09:37:35 +0200
> To: Debian Bug Tracking System <[email protected]>
> Subject: Bug#671922: magit: Does not autoload extensions

> Package: magit
> Version: 1.1.1-1
> Severity: normal


> Hi,

> With the original configuration file magit does not autoload
> functions defined by the extensions,
> ie the functions

> magit-svn-mode
> turn-on-magit-svn
> magit-topgit-mode
> turn-on-magit-topgit
> magit-stgit-mode
> turn-on-magit-stgit

> These functions are marked to be autoloaded in the source,
> and the documentation explicitly mentions them as available
> (eg as examples to add to magit-mode-hook).

> The modified configuration file below autoloads them
> (it is a mix of the original config file and the autoloads
> generated by update-directory-autoloads).

> Kind regards,

>       Gábor Braun

> -- System Information:
> Debian Release: 6.0.4
>   APT prefers stable
>   APT policy: (990, 'stable'), (650, 'testing'), (600, 'unstable')
> Architecture: amd64 (x86_64)

> Kernel: Linux 2.6.32-5-amd64 (SMP w/2 CPU cores)
> Locale: LANG=hu_HU.UTF-8, LC_CTYPE=hu_HU.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash

> Versions of packages magit depends on:
> ii  dpkg                      1.15.8.12      Debian package management system
> ii  emacs23                   23.4+1-3       The GNU Emacs editor (with GTK+ 
> us
> ii  git [git-core]            1:1.7.2.5-3    fast, scalable, distributed 
> revisi
> ii  git-core                  1:1.7.2.5-3    fast, scalable, distributed 
> revisi
> ii  install-info              4.13a.dfsg.1-6 Manage installed documentation 
> in 

> magit recommends no packages.

> magit suggests no packages.

> -- Configuration Files:
> /etc/emacs/site-start.d/50magit.el changed:
> ;;; Autoloads for magit
> (if (member debian-emacs-flavor '(emacs23 emacs-snapshot))
>     (if (file-exists-p "/usr/share/emacs/site-lisp/magit/magit.el")
>       (progn
> ;;; --- automatically extracted autoloads
> ;;
> ;;; Code:
> 
> ;;;### (autoloads (magit-status) "magit" 
> "../../usr/share/emacs/site-lisp/magit/magit.el"
> ;;;;;;  (20261 17500))
> ;;; Generated autoloads from ../../usr/share/emacs/site-lisp/magit/magit.el
> (autoload 'magit-status "magit" "\
> Open a Magit status buffer for the Git repository containing
> DIR.  If DIR is not within a Git repository, offer to create a
> Git repository in DIR.
> Interactively, a prefix argument means to ask the user which Git
> repository to use even if `default-directory' is under Git control.
> Two prefix arguments means to ignore `magit-repo-dirs' when asking for
> user input.
> \(fn DIR)" t nil)
> ;;;***
> 
> ;;;### (autoloads (turn-on-magit-stgit magit-stgit-mode) "magit-stgit"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-stgit.el" (20261
> ;;;;;;  17500))
> ;;; Generated autoloads from 
> ../../usr/share/emacs/site-lisp/magit/magit-stgit.el
> (autoload 'magit-stgit-mode "magit-stgit" "\
> StGit support for Magit
> \(fn &optional ARG)" t nil)
> (autoload 'turn-on-magit-stgit "magit-stgit" "\
> Unconditionally turn on `magit-stgit-mode'.
> \(fn)" nil nil)
> ;;;***
> 
> ;;;### (autoloads (turn-on-magit-svn magit-svn-mode) "magit-svn"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-svn.el" (20261
> ;;;;;;  17500))
> ;;; Generated autoloads from 
> ../../usr/share/emacs/site-lisp/magit/magit-svn.el
> (autoload 'magit-svn-mode "magit-svn" "\
> SVN support for Magit
> \(fn &optional ARG)" t nil)
> (autoload 'turn-on-magit-svn "magit-svn" "\
> Unconditionally turn on `magit-svn-mode'.
> \(fn)" nil nil)
> ;;;***
> 
> ;;;### (autoloads (turn-on-magit-topgit magit-topgit-mode) "magit-topgit"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-topgit.el" (20261
> ;;;;;;  17500))
> ;;; Generated autoloads from 
> ../../usr/share/emacs/site-lisp/magit/magit-topgit.el
> (autoload 'magit-topgit-mode "magit-topgit" "\
> Topgit support for Magit
> \(fn &optional ARG)" t nil)
> (autoload 'turn-on-magit-topgit "magit-topgit" "\
> Unconditionally turn on `magit-topgit-mode'.
> \(fn)" nil nil)
> ;;;***
> 
> ;;;### (autoloads nil nil 
> ("../../usr/share/emacs/site-lisp/magit/magit-bisect.el"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-classic-theme.el"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-key-mode.el"
> ;;;;;;  "../../usr/share/emacs/site-lisp/magit/magit-simple-keys.el")
> ;;;;;;  (20390 50830 406308))
> )
> ;;; End --- automatically extracted autoloads
>       (message "Package magit removed but not purged. Skipping setup.")))
> ;; Local Variables:
> ;; no-byte-compile: t
> ;; no-update-autoloads: t
> ;; coding: utf-8
> ;; End:


> -- no debconf information

-- 
"Once more the drama begins."

  -- The Emperor Paul Muad'dib on his ascension to the Lion Throne

Reply via email to