210752 – app-editors/emacs-22.1-r3 bibtex-mode: bibtex-convert-alien always fails

Bug 210752 - app-editors/emacs-22.1-r3 bibtex-mode: bibtex-convert-alien always fails

Summary: app-editors/emacs-22.1-r3 bibtex-mode: bibtex-convert-alien always fails

Status:	RESOLVED UPSTREAM

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Emacs project

URL:	http://permalink.gmane.org/gmane.emac...
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2008-02-19 20:23 UTC by Andreas K. Hüttel
Modified:	2008-02-21 13:21 UTC (History)
CC List:	0 users

See Also:
Package list:
Runtime testing required:	---

Attachments
Patch for bibtex.el (emacs-22-bibtex.patch,963 bytes, patch) 2008-02-19 21:38 UTC, Ulrich Müller	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Andreas K. Hüttel archtester

2008-02-19 20:23:07 UTC

In bibtex mode, trying to reformat a bibtex file with the bibtex-convert-alien command always fails. The error message is  

Wrong type argument: number-or-marker-p, nil

Comment 1 Ulrich Müller gentoo-dev

2008-02-19 21:38:15 UTC

Created attachment 144015 [details, diff]
Patch for bibtex.el

It looks like bibtex.el was not fully converted to Emacs 22: function "sit-for" used to have 3 arguments in Emacs 21, now it has only 2.

Attached patch should fix the problem.

Comment 2 Andreas K. Hüttel archtester

2008-02-19 21:51:44 UTC

> It looks like bibtex.el was not fully converted to Emacs 22: function "sit-for"
> used to have 3 arguments in Emacs 21, now it has only 2.

I applied the patch to /usr/share/emacs/22.1/lisp/textmodes/bibtex.el and restarted emacs. Still the same problem. 

Grep tells me there's still a line in the file with 
 (sit-for 1)
i.e. only one argument. ->?

Comment 3 Ulrich Müller gentoo-dev

2008-02-19 22:05:33 UTC

(In reply to comment #2)
> I applied the patch to /usr/share/emacs/22.1/lisp/textmodes/bibtex.el and
> restarted emacs. Still the same problem. 

Please try again, and do an explicit "M-x load-library RET bibtex.el RET" before.

Comment 4 Ulrich Müller gentoo-dev

2008-02-19 22:20:09 UTC

Submitted upstream: <http://permalink.gmane.org/gmane.emacs.bugs/17543>.
Since Emacs 22.2 is already in pretest, we won't fix this in Gentoo for 22.1.

Thank you for reporting.

Comment 5 Andreas K. Hüttel archtester

2008-02-19 22:54:48 UTC

(In reply to comment #3)
> (In reply to comment #2)
> > I applied the patch to /usr/share/emacs/22.1/lisp/textmodes/bibtex.el and
> > restarted emacs. Still the same problem. 
> 
> Please try again, and do an explicit "M-x load-library RET bibtex.el RET"
> before.
> 

Thank you - now it works fine.

Comment 6 Ulrich Müller gentoo-dev

2008-02-21 13:21:22 UTC

This bug has been fixed in CVS upstream. emacs-cvs-22.1.9999 should be fine now, as well as the upcoming emacs-cvs-22.1.92 and emacs-22.2.