From: Waiyian Chong <waiyian.chong@gmail.com>
To: emacs-devel@gnu.org
Subject: Re: "make autoloads" fails
Date: Sun, 30 Jun 2013 23:30:00 +0800 [thread overview]
Message-ID: <87k3lbobuv.fsf@gmail.com> (raw)
In-Reply-To: 83vc4wda2i.fsf@gnu.org
[-- Attachment #1: Type: text/plain, Size: 1205 bytes --]
Eli Zaretskii <eliz@gnu.org> writes:
> Here's the error message:
>
> $ make autoloads
> EMACSLOADPATH=/d/gnu/bzr/emacs/trunk/lisp LC_ALL=C /d/gnu/bzr/emacs/trunk/src/emacs -batch --no-site-file --no-site-lisp -l autoload \
> --eval "(setq generate-autoload-cookie \";;;###cal-autoload\")" \
> --eval "(setq generated-autoload-file (unmsys--file-name \"/d/gnu/bzr/emacs/trunk/lisp/calendar/cal-loaddefs.el\"))" \
> --eval "(setq make-backup-files nil)" \
> -f batch-update-autoloads /d/gnu/bzr/emacs/trunk/lisp/calendar
> Wrong type argument: number-or-marker-p, nil
...
> Lisp Backtrace:
> "select-safe-coding-system" (0x82e494)
> "write-region" (0x82ed9c)
> "autoload-ensure-default-file" (0x82ef6c)
> "autoload-find-generated-file" (0x82f140)
> "update-directory-autoloads" (0x82f3e0)
> "apply" (0x82f3dc)
> "batch-update-autoloads" (0x82f5f0)
> "command-line-1" (0x82f7c8)
> "command-line" (0x82f9cc)
> "normal-top-level" (0x82fb50)
I encountered the same problem, and traced it back to the function
`autoload-ensure-default-file', which seems to call `write-region'
with wrong arguments. Build succeeded after applying attached patch.
--
waiyian
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 560 bytes --]
diff --git a/lisp/emacs-lisp/autoload.el b/lisp/emacs-lisp/autoload.el
index 22713c6..d1722d0 100644
--- a/lisp/emacs-lisp/autoload.el
+++ b/lisp/emacs-lisp/autoload.el
@@ -354,7 +354,7 @@ not be relied upon."
(defun autoload-ensure-default-file (file)
"Make sure that the autoload file FILE exists and if not create it."
(unless (file-exists-p file)
- (write-region (autoload-rubric file) nil file))
+ (with-temp-file file (insert (autoload-rubric file))))
file)
(defun autoload-insert-section-header (outbuf autoloads load-name file time)
next prev parent reply other threads:[~2013-06-30 15:30 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-29 18:51 "make autoloads" fails Eli Zaretskii
2013-06-30 1:02 ` Paul Eggert
2013-06-30 1:35 ` Juanma Barranquero
2013-06-30 2:43 ` Eli Zaretskii
2013-06-30 3:03 ` Juanma Barranquero
2013-06-30 2:43 ` Eli Zaretskii
2013-06-30 15:30 ` Waiyian Chong [this message]
2013-06-30 16:04 ` Eli Zaretskii
2013-06-30 17:45 ` Uwe Brauer
2013-06-30 17:50 ` Jambunathan K
2013-06-30 17:52 ` Uwe Brauer
2013-06-30 18:05 ` Jambunathan K
2013-06-30 18:12 ` Eli Zaretskii
2013-06-30 18:17 ` Uwe Brauer
2013-06-30 18:27 ` Eli Zaretskii
2013-06-30 18:29 ` Uwe Brauer
2013-06-30 19:20 ` Eli Zaretskii
2013-06-30 20:59 ` Uwe Brauer
-- strict thread matches above, loose matches on Subject: below --
2013-06-30 8:21 Jan Djärv
2013-06-30 14:37 ` Eli Zaretskii
2013-06-30 11:31 Angelo Graziosi
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.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k3lbobuv.fsf@gmail.com \
--to=waiyian.chong@gmail.com \
--cc=emacs-devel@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.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).