From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Ecay Subject: Re: Citation syntax: a revised proposal Date: Tue, 03 Mar 2015 14:25:26 +0000 Message-ID: <87vbii2m5l.fsf@gmail.com> References: <87k2zjnc0e.fsf@berkeley.edu> <87bnkvm8la.fsf@berkeley.edu> <87zj8co3se.fsf@berkeley.edu> <87ioezooi2.fsf@berkeley.edu> <87mw4bpaiu.fsf@nicolasgoaziou.fr> <8761aznpiq.fsf@berkeley.edu> <87twyjnh0r.fsf@nicolasgoaziou.fr> <87oaopx24e.fsf@berkeley.edu> <87k2zd4f3w.fsf@nicolasgoaziou.fr> <87egpkv8g9.fsf@berkeley.edu> <877fv6xfaq.fsf@gmail.com> <87twya2ak0.fsf@berkeley.edu> <87zj81aa97.fsf@nicolasgoaziou.fr> <87ioep2r6p.fsf@berkeley.edu> <87y4ngbgm7.fsf@nicolasgoaziou.fr> <87bnkbi61v.fsf@gmail.com> <87lhjfb0yv.fsf@nicolasgoaziou.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35415) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSnlY-0006lW-NK for emacs-orgmode@gnu.org; Tue, 03 Mar 2015 09:25:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YSnlU-0006pb-DU for emacs-orgmode@gnu.org; Tue, 03 Mar 2015 09:25:44 -0500 Received: from mail-we0-x22e.google.com ([2a00:1450:400c:c03::22e]:42839) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSnlU-0006pC-3S for emacs-orgmode@gnu.org; Tue, 03 Mar 2015 09:25:40 -0500 Received: by wesw62 with SMTP id w62so40179564wes.9 for ; Tue, 03 Mar 2015 06:25:39 -0800 (PST) In-Reply-To: <87lhjfb0yv.fsf@nicolasgoaziou.fr> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Nicolas Goaziou , Richard Lawrence Cc: emacs-orgmode@gnu.org, John Kitchin Hi Nicolas, 2015ko martxoak 2an, Nicolas Goaziou-ek idatzi zuen: >=20 > Hello, >=20 > Aaron Ecay writes: >=20 >> I decided to go ahead and see what I could make of it. The result has >> been pushed to the org mode repo to the branch wip-cite-awe. (I didn=E2= =80=99t >> want to push to your branch without asking, but if you prefer I=E2=80=99= ll do >> that and delete my own.) >=20 > This is not *my* branch. However, I suggest to push only consensual > features with documentation and tests there, and experiment features in > other branches. OK, I wasn=E2=80=99t sure what the etiquette was. >=20 >> The first issue is that the parser includes trailing punctuation in >> =E2=80=9Cbare=E2=80=9D @key citations. So the following does not work a= s expected (the >> :key includes the period): =E2=80=9CThis was demonstrated most recently = by >> @Smith2015.=E2=80=9D I=E2=80=99m not sure what the right approach is = =E2=80=93 one option >> would be to say that keys can contain punctuation, but must end (and >> begin) with an alphanumeric character. >=20 > I'll update the parser once there is a new syntax for keys. At the > moment, it is correct wrt syntax. >=20 >> The second issue is that the :key property of the citation element >> includes the @. This is not right IMO: it=E2=80=99s a detail of the syn= tax. >> And it means that consumers of the syntax, who might want to look up >> the key in a database, will always have to remember to strip the @. >> I=E2=80=99ve pushed a provisional fix for this in my branch. >=20 > Please apply it to wip-cite. A dedicated test would be nice, too. OK. >=20 >> The code is very rough and ready, has lots of TODO comments in it, is >> missing tests, documentation in the manual, etc. Nonetheless, I want to >> get feedback on it early, given that many people have already contributed >> so much useful information to this discussion. >=20 > I didn't look closely at the code, but I suggest to use "org-cite.el" > instead of "ox-cite.el". Even though this is only related to export at > the moment, this library will also contain commands to manipulate > citation objects. It also shorten prefix for these functions. OK. It will certainly save some typing. --=20 Aaron Ecay