From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Chong Yidong Newsgroups: gmane.emacs.bugs Subject: bug#10970: 24.0.93; package.el doesn't load autoloads before compiling Date: Sun, 11 Mar 2012 17:26:29 +0800 Message-ID: <87d38jv67u.fsf@gnu.org> References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1331458049 18014 80.91.229.3 (11 Mar 2012 09:27:29 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Sun, 11 Mar 2012 09:27:29 +0000 (UTC) Cc: 10970@debbugs.gnu.org To: Donald Ephraim Curtis Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sun Mar 11 10:27:28 2012 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1S6f3l-0006el-LF for geb-bug-gnu-emacs@m.gmane.org; Sun, 11 Mar 2012 10:27:25 +0100 Original-Received: from localhost ([::1]:43971 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S6f3k-0006QR-MY for geb-bug-gnu-emacs@m.gmane.org; Sun, 11 Mar 2012 05:27:24 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:50695) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S6f3g-0006Q9-SG for bug-gnu-emacs@gnu.org; Sun, 11 Mar 2012 05:27:22 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S6f3e-0002zc-UP for bug-gnu-emacs@gnu.org; Sun, 11 Mar 2012 05:27:20 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:37635) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S6f3e-0002zT-Ql for bug-gnu-emacs@gnu.org; Sun, 11 Mar 2012 05:27:18 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1S6fWQ-0006so-Ei for bug-gnu-emacs@gnu.org; Sun, 11 Mar 2012 05:57:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Chong Yidong Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 11 Mar 2012 09:57:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 10970 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 10970-submit@debbugs.gnu.org id=B10970.133145979626426 (code B ref 10970); Sun, 11 Mar 2012 09:57:02 +0000 Original-Received: (at 10970) by debbugs.gnu.org; 11 Mar 2012 09:56:36 +0000 Original-Received: from localhost ([127.0.0.1]:44467 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1S6fVz-0006sA-DI for submit@debbugs.gnu.org; Sun, 11 Mar 2012 05:56:35 -0400 Original-Received: from fencepost.gnu.org ([208.118.235.10]:36242) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1S6fVn-0006rs-Tu for 10970@debbugs.gnu.org; Sun, 11 Mar 2012 05:56:34 -0400 Original-Received: from bb116-14-103-36.singnet.com.sg ([116.14.103.36]:56971 helo=ulysses) by fencepost.gnu.org with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1S6f2y-0002ld-Ay; Sun, 11 Mar 2012 05:26:37 -0400 In-Reply-To: (Donald Ephraim Curtis's message of "Thu, 8 Mar 2012 09:08:36 -0600") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.94 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:57664 Archived-At: Donald Ephraim Curtis writes: > However, before compiling, the autoloads file (pkg-autoloads.el) is not > loaded, so anything that updates the load-path is also not sourced. > > It may be best that packages simply put all their lisp files in the main > package directory, but it also may be worthwhile to simply add, > > (load (expand-file-name (concat (symbol-name name) "-autoloads") pkg-dir) nil t) > > in the `package-unpack' function after the autoloads are generated. I see the problem. OTOH I don't see any actual bug that would arise from loading the autoloads file before byte compilation. But on the other hand, it's kind of conceptually nice that the autoloads file is only loaded when the package is activated, as a separate step from unpacking. Maybe it's better for the affected files to use `eval-when-compile' if they need to alter the load path. Anyone have another opinion?