Skip to content

Add nrepl-autoloads.el file #208

Closed
wants to merge 1 commit into from

3 participants

@vedang
vedang commented Jan 5, 2013

This file is a simple extraction of autoloaded functions from nrepl.el
(using the command update-file-autoloads)

The presence of this file makes loading nrepl faster since we can now
replace (require 'nrepl) with (require 'nrepl-autoloads)

@vedang vedang Add nrepl-autoloads file.
This file is a simple extraction of autoloaded functions from nrepl.el
(using the command `update-file-autoloads`)

The presence of this file makes loading nrepl faster since we can now
replace (require 'nrepl) with (require 'nrepl-autoloads)
2332917
@bbatsov
clojure-emacs member
bbatsov commented Jan 7, 2013

I don't think this is needed. The preferred way to install nrepl.el is via package.el which will generate the autoload file and load it automatically. I doubt that many people are installing nrepl.el manually and I don't think we should encourage the manual installation (since as the number of deps grows it will become very painful). And there's also the matter of keeping this file in sync with nrepl.el as autoloads change. These days distributing autoload files is generally frowned upon.

@vedang
vedang commented Jan 8, 2013

Okay, sounds good. I still prefer installing manually (vs through package.el etc) because then I have control over the commit I want to work with. But I understand that people are moving over to package.el. Please close the issue.

@kingtim kingtim closed this Jan 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.