From: Achim Gratz <Stromeko@nexgo.de>
To: emacs-orgmode@gnu.org
Subject: Re: Web site bug
Date: Sun, 28 Oct 2012 08:36:33 +0100 [thread overview]
Message-ID: <87625vhwu6.fsf@Rainer.invalid> (raw)
In-Reply-To: 87hapgnv18.fsf@bzg.ath.cx
[-- Attachment #1: Type: text/plain, Size: 885 bytes --]
Bastien writes:
> No, (require 'org-loaddefs) is never needed IMHO.
It is if you are using Org from Git and you only want the autoload
definitions pulled in on startup and not the whole of Org plus most of
its dependencies. This is what a
(require 'org-install)
did before the change to org-loaddefs.el. You can't rely on some older
Org version in Emacs or whereever to supply the correct autoload
definitions, so you eitehr need to
(require 'org-loaddefs)
which is horribly wrong in case you're not sing Org from Git; or
(require 'org)
which is pulling in too much stuff on startup.
Org from Git is currently missing the first-level autoloads file that
was implicitly introduced for all other use-cases with that change
(loaddefs.el for vanilla Emacs and org-autoloads.el for package
manager). I propose to re-introduce org-install.el for this purpose for
standalone Org.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Re-introduce-org-install.el.patch --]
[-- Type: text/x-patch, Size: 3017 bytes --]
From 6b2d3f7689078f836cf1dbd8ee508131e8b22ef5 Mon Sep 17 00:00:00 2001
From: Achim Gratz <Stromeko@Stromeko.DE>
Date: Sun, 28 Oct 2012 08:23:22 +0100
Subject: [PATCH] Re-introduce org-install.el
* lisp/org-install.el: Delete from version control, is autogenerated
again. An empty file should be provided in Emacs and for ELPA as
before.
* mk/org-fixup.el (org-make-org-loaddefs): Arrange for org-install to
collect the "first-level" autoload definitions for standalone
Org (like loaddefs.el for Emacs and org-autoloads.el for package
manager). The autoloads in org-loaddefs.el will be identical
"second-level" for all installations. The "first-level" autoload
definitions are automatically loaded by EMacs or package manager,
respectively. For standalone Org, the user has to "(require
'org-install)", as has been customary.
FIXME: this implementation does not support XEmacs.
---
lisp/org-install.el | 13 -------------
mk/org-fixup.el | 15 ++++-----------
2 files changed, 4 insertions(+), 24 deletions(-)
delete mode 100644 lisp/org-install.el
diff --git a/lisp/org-install.el b/lisp/org-install.el
deleted file mode 100644
index a31d8b7..0000000
--- a/lisp/org-install.el
+++ /dev/null
@@ -1,13 +0,0 @@
-;;; org-install.el --- autogenerated file, do not edit
-;;
-;;; Code:
-(warn "The file org-install is obsolete.
-Please change your configuration to (require 'org) instead.")
-\f
-(provide 'org-install)
-\f
-;; Local Variables:
-;; no-byte-compile: t
-;; coding: utf-8
-;; End:
-;;; org-install.el ends here
diff --git a/mk/org-fixup.el b/mk/org-fixup.el
index 7b59efb..e5dee9a 100644
--- a/mk/org-fixup.el
+++ b/mk/org-fixup.el
@@ -67,17 +67,10 @@ (defun org-make-org-loaddefs ()
be used by foreign build systems or installers to produce this
file in the installation directory of org-mode. Org will not
work correctly if this file is not up-to-date."
- (with-temp-buffer
- (set-visited-file-name "org-loaddefs.el")
- (insert ";;; org-loaddefs.el --- autogenerated file, do not edit\n;;\n;;; Code:\n")
- (let ((files (directory-files default-directory nil "^\\(org\\|ob\\)\\(-.*\\)?\\.el$")))
- (mapc (lambda (f) (generate-file-autoloads f)) files))
- (insert "\f\n(provide 'org-loaddefs)\n")
- (insert "\f\n;; Local Variables:\n;; version-control: never\n")
- (insert ";; no-byte-compile: t\n;; no-update-autoloads: t\n")
- (insert ";; coding: utf-8\n;; End:\n;;; org-loaddefs.el ends here\n")
- (toggle-read-only 0)
- (save-buffer)))
+ (write-region (autoload-rubric "org-install.el" nil "org-install") nil "org-install.el")
+ (write-region (autoload-rubric "org-loaddefs.el" nil "org-loaddefs") nil "org-install.el")
+ (let ((generated-autoload-file (expand-file-name "org-install.el" default-directory)))
+ (update-directory-autoloads (file-name-directory generated-autoload-file))))
(defun org-make-autoloads (&optional compile force)
"Make the files org-loaddefs.el and org-version.el in the install directory.
--
1.8.0
[-- Attachment #3: Type: text/plain, Size: 187 bytes --]
Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+
Samples for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldSamplesExtra
next prev parent reply other threads:[~2012-10-28 7:36 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-24 20:36 Web site bug Simon Thum
2012-10-24 22:42 ` Bastien
2012-10-26 19:57 ` Simon Thum
2012-10-27 7:33 ` Bastien
2012-10-27 8:54 ` Sebastien Vauban
2012-10-27 9:06 ` Bastien
2012-10-28 7:36 ` Achim Gratz [this message]
2012-10-28 10:29 ` Bastien
2012-10-28 16:12 ` Achim Gratz
2012-10-29 6:57 ` Bastien
2012-10-29 7:29 ` Achim Gratz
2012-10-29 7:33 ` Bastien
2012-10-29 7:48 ` Achim Gratz
2012-10-29 8:56 ` Eric S Fraga
2012-10-29 6:56 ` Bastien
2012-10-29 7:02 ` Achim Gratz
2012-10-29 7:07 ` Bastien
2012-10-27 16:15 ` Simon Thum
2012-10-27 19:37 ` Bastien
2012-10-27 8:01 ` Achim Gratz
2012-10-27 8:28 ` Bastien
2012-10-27 11:11 ` Achim Gratz
-- strict thread matches above, loose matches on Subject: below --
2010-07-20 16:11 web " Robert Klein
2010-07-25 16:26 ` David Maus
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
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87625vhwu6.fsf@Rainer.invalid \
--to=stromeko@nexgo.de \
--cc=emacs-orgmode@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 public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).