[debbugs-tracker] bug#30164: closed (load-history and modules)

[GNU bug Tracking System]

Your message dated Fri, 02 Feb 2018 19:58:44 +0000
with message-id <address@hidden>
and subject line Re: bug#30164: load-history and modules
has caused the debbugs.gnu.org bug report #30164,
regarding load-history and modules
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
30164: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=30164
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> load-history and modules
>
>
>
>
> Date: 
>
> Thu, 18 Jan 2018 17:49:49 -0500
>
>
>
>
> User-agent: 
>
> Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
>
>
>
>
> Package: emacs
> Version: 26.0.91
> Severity: minor
>
> As pointed out in https://debbugs.gnu.org/30106#25,
> Fmodule_load does not update load-history.
>
> As a result
>
> (require 'some-module)
>
> gives a confusing error message if some-module.so fails to provide a feature.
>
> Eg:
>
> Comment out the line in data/emacs-module/mod-test.c:
>   provide (env, "mod-test");
>
> and rebuild mod-test.so. Then do:
>
> ./src/emacs -Q -L $PWD/test/data/emacs-module
> (require 'mod-test)
>
> -> error "Loading file [...]elec-pair.elc failed to provide feature 'mod-test'")
>
>
>
> --- End Message ---

--- Begin Message --- Subject: Re: bug#30164: load-history and modules Date: Fri, 02 Feb 2018 19:58:44 +0000

Eli Zaretskii <address@hidden> schrieb am Fr., 2. Feb. 2018 um 16:24 Uhr:

> From: Philipp Stephani <address@hidden>
> Date: Sun, 28 Jan 2018 20:38:12 +0000
> Cc: address@hidden
>
>  Here is a patch.

Thanks, this LGTM, but I think it needs a NEWS entry.  Also, please
see if something we say about 'load' and load-history needs updating
due to this changeset.

Thanks, pushed to master as  0443411f5c.

The manual didn't say anything in particular about differences in behavior for loading modules (which makes sense, the previous behavior was clearly buggy), so no change to the manual was necessary.

--- End Message ---