emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Matt Lundin <mdl@imapmail.org>
To: emacs-orgmode@gnu.org
Subject: Re: Allowing loose ordering in Org files
Date: Tue, 10 Nov 2015 11:51:28 -0600	[thread overview]
Message-ID: <87h9ktoi8v.fsf@fastmail.fm> (raw)
In-Reply-To: <m2y4e6y61o.fsf@Vulcan.attlocal.net> (John Wiegley's message of "Mon, 09 Nov 2015 17:52:35 -0800")

John Wiegley <jwiegley@gmail.com> writes:

>>>>>> Aaron Ecay <aaronecay@gmail.com> writes:
>
>> Adding knobs to this parser increases the burden of those who have to build
>> and maintain it.
>
> Thank you for your reply, Aaron, I found it most illuminating.
>
> If the answer from the maintainers is "It's more work than we want to
> do", that's completely acceptable. I've been operating under the
> premise that it wouldn't be difficult to add such an option (just the
> hook, mind you, not the functionality behind it).

After looking quickly at the code, my impression is that without
substantial refactoring, a hook/variable pointing to an alternate "find
property drawer" function is not a very clean option, since org makes
the assumption in several places that the property drawer comes
immediately after the planning info. See, for instance,
org-insert-drawer, org-end-of-meta-data, org-get-property-block.

If you were to hack something on your own, I suppose your could
rewrite/advise org-get-property-block, but I have no idea what else that
would break.

> If my request is fulfilled, I would expect that changing the "find
> properties function" would imply that one's Org file is no longer
> usable by secondary interpreters. I.e., "If you change this, you do so
> at your own risk".
[ ... quoted from another email ] 

I would vote against creating a hook or variable that comes with a
warning "change this at your own risk." I am concerned about the
precedent this would set. This seems to me what defadvice is for. Would
this really be a simpler option than posting a hack on emacswiki or
github? After all, anyone customizing the variable would still have to
grab the custom function from github, etc.

IMO, to add a customization option that might (and probably will) break
other parts of org mode threatens to add complexity to the maintenance
of org, since it might create a base of users who are relying on a
"semi-officially supported hack" rather than the official parsing logic
supported by org. Despite the disclaimer, maintainers will still be
forced to keep the possibility of this customization in mind when
rewriting functions that parse org syntax. And even if a user accepts
the full risk of the customization, it is still possible that he/she
might accidentally report a bug to the mailing list when something
breaks (not realizing it is related to the customization).

In short, if we allow for an alternative parsing logic, I think it
should be one that is officially supported/maintained.

Best,
Matt

  parent reply	other threads:[~2015-11-10 17:51 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-03  0:11 bug in org-habits Mark A. Hershberger
2015-11-03  9:56 ` Marco Wahl
2015-11-03 11:16   ` Puneeth Chaganti
2015-11-03 13:11     ` John Wiegley
2015-11-03 13:46       ` Marco Wahl
2015-11-03 14:20         ` Stelian Iancu
2015-11-03 16:31           ` Nicolas Goaziou
2015-11-03 19:20             ` John Wiegley
2015-11-03 19:35               ` Nicolas Goaziou
2015-11-03 20:17                 ` John Wiegley
2015-11-03 20:52                   ` Nicolas Goaziou
2015-11-03 20:55                     ` John Wiegley
2015-11-03 21:31                       ` Achim Gratz
2015-11-03 21:36                         ` John Wiegley
2015-11-03 21:48                         ` Jonathan Leech-Pepin
2015-11-03 21:56                           ` John Wiegley
2015-11-03 22:36                             ` Achim Gratz
2015-11-03 22:45                               ` John Wiegley
2015-11-04 13:01                                 ` Bastien Guerry
2015-11-04 20:26                                   ` John Wiegley
2015-11-09 15:13                                     ` Allowing loose ordering in Org files (Was: bug in org-habits) John Wiegley
2015-11-09 17:47                                       ` Allowing loose ordering in Org files Rasmus
2015-11-09 18:15                                         ` John Wiegley
2015-11-09 19:28                                           ` Rasmus
2015-11-09 19:57                                             ` John Wiegley
2015-11-09 19:12                                       ` Achim Gratz
2015-11-09 19:24                                         ` John Wiegley
2015-11-09 20:04                                           ` Achim Gratz
2015-11-09 21:13                                             ` Stelian Iancu
2015-11-09 21:30                                               ` John Wiegley
2015-11-10  1:40                                       ` Allowing loose ordering in Org files (Was: bug in org-habits) Aaron Ecay
2015-11-10  1:52                                         ` Allowing loose ordering in Org files John Wiegley
2015-11-10  5:31                                           ` Thomas S. Dye
2015-11-10 17:37                                             ` Nicolas Goaziou
2015-11-10 19:20                                               ` Thomas S. Dye
2015-11-10 20:02                                               ` John Wiegley
2015-11-10 20:42                                                 ` Matt Lundin
2015-11-10 20:44                                                   ` Matt Lundin
2015-11-10 17:51                                           ` Matt Lundin [this message]
2015-11-10 18:19                                           ` Matt Lundin
2015-11-10 19:49                                           ` Achim Gratz
2015-11-10 20:11                                             ` John Wiegley
2015-11-10 20:38                                               ` Achim Gratz
2015-11-10 22:35                                                 ` John Wiegley
2015-11-11 16:13                                               ` Karl Voit
2015-11-10 11:30                                         ` Allowing loose ordering in Org files (Was: bug in org-habits) Stelian Iancu
2015-11-03 23:43                       ` bug in org-habits Nicolas Goaziou
2015-11-04  1:01                         ` John Wiegley
2015-11-04  9:02                           ` Stelian Iancu
2015-11-04  9:16                           ` Eric S Fraga

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.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h9ktoi8v.fsf@fastmail.fm \
    --to=mdl@imapmail.org \
    --cc=emacs-orgmode@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/org-mode.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).