Summary: | app-emacs/vm-8.2.0_beta with app-editors/emacs-26.1_rc1 - Variable name missing after &optional in vm-pcrisis.el | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | milan hodoscek <milan> |
Component: | Current packages | Assignee: | GNU Emacs project <gnu-emacs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | UPSTREAM |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://launchpad.net/bugs/1763377 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
fix vm-pcrisis.el for emacs-26 elisp
build.log |
Description
milan hodoscek
2018-04-11 05:53:54 UTC
Created attachment 527128 [details, diff]
fix vm-pcrisis.el for emacs-26 elisp
ading epatch_user to ebuild and put this file in /etc/portage/patches/app-emacs/vm-8.2.0_beta enables vm package to compile with emacs-26
Found this in the /usr/share/emacs/26.1/etc/NEWS ** Using '&rest' or '&optional' incorrectly is now an error. For example giving '&optional' without a following variable, or passing '&optional' multiple times: (defun foo (&optional &rest x)) (defun bar (&optional &optional x)) Previously, Emacs would just ignore the extra keyword, or give incorrect results in certain cases. Please attach the entire build log to this bug report. Created attachment 527138 [details]
build.log
Attaching the build log. It fails only if bbdb use flag is used. Without bbdb the vm-pcrisis.el is not compiled (patched out from Makefile)
Since the issue is neither fixed in 8.2.x nor in trunk, can you please report it upstream, too: <https://launchpad.net/vm> (In reply to Ulrich Müller from comment #5) > Since the issue is neither fixed in 8.2.x nor in trunk, can you please > report it upstream, too: <https://launchpad.net/vm> I wanted to look at the louncpad bug reports but it was asking for login, etc, so I got discouraged. But now that you are asking for it I just did it. Sorry for the delay. I wonder about this change in your patch: -(defun vmpc-tab-header-or-tab-stop (&optional backward) +(defun vmpc-tab-header-or-tab-stop (&rest backward) backward is a simple flag there (can be nil or non-nil), so &optional looks right to me. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5e3ee0a508a71cbc4090d86e09e3a5c124566034 commit 5e3ee0a508a71cbc4090d86e09e3a5c124566034 Author: Ulrich Müller <ulm@gentoo.org> AuthorDate: 2018-08-10 06:22:02 +0000 Commit: Ulrich Müller <ulm@gentoo.org> CommitDate: 2018-08-10 06:22:41 +0000 app-emacs/vm: Fix function definition in vm-pcrisis.el. Bug: https://bugs.gentoo.org/652952 Package-Manager: Portage-2.3.45, Repoman-2.3.10 app-emacs/vm/files/vm-8.2.0_beta-optional-args.patch | 13 +++++++++++++ .../{vm-8.2.0_beta.ebuild => vm-8.2.0_beta-r1.ebuild} | 18 +++++++++--------- 2 files changed, 22 insertions(+), 9 deletions(-) |