[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] orgtbl export to latex :fmt() fails
From: |
Thorsten Grothe |
Subject: |
Re: [O] orgtbl export to latex :fmt() fails |
Date: |
Thu, 17 Jul 2014 23:54:37 +0200 |
Nick,
> Nothing - there is a bug in org-table.el:org-table-clean-before-export
> where the regexp that matches special chars in the first column (see
>
> (info "(org)Advanced features")
>
> for the details) inadvertently matches "| | | | 3900|" and deletes the
> first column. The regexp is set like this:
>
> --8<---------------cut here---------------start------------->8---
> (let ((special (if maybe-quoted
> "^[ \t]*| *\\\\?[\#!$*_^/ ] *|"
> "^[ \t]*| *[\#!$*_^/ ] *|"))
> --8<---------------cut here---------------end--------------->8---
>
> and in each case I think that the space inside the second [...] is
> spurious.
>
> So try this patch:
>
> --8<---------------cut here---------------start------------->8---
> diff --git a/lisp/org-table.el b/lisp/org-table.el
> index d7ef615..864493e 100644
> --- a/lisp/org-table.el
> +++ b/lisp/org-table.el
> @@ -447,8 +447,8 @@ available parameters."
> "Check if the table has a marking column.
> If yes remove the column and the special lines."
> (let ((special (if maybe-quoted
> - "^[ \t]*| *\\\\?[\#!$*_^/ ] *|"
> - "^[ \t]*| *[\#!$*_^/ ] *|"))
> + "^[ \t]*| *\\\\?[\#!$*_^/] *|"
> + "^[ \t]*| *[\#!$*_^/] *|"))
> (ignore (if maybe-quoted
> "^[ \t]*| *\\\\?[!$_^/] *|"
> "^[ \t]*| *[!$_^/] *|")))
> --8<---------------cut here---------------end--------------->8---
>
> I think it's OK for the non-quoted case, but I'm not sure
> about the quoted case (maybe-quotes is t). If there are no
> objections, I'll push it later on tonight.
>
> Just to be sure: this is a bug, so it should be committed
> to the maint branch and then a merge should be done onto master -
> correct?
thank you very much for your response, well I'm not an emacs guru, so my simple
question is, how to apply this patch? I'm working with archlinux here and I
installed orgmode systemwide not locally, I guess I should first install it in
my local homedir and than patch it, but how?
Sorry about this simple question :-)
Regards
Th. Grothe
Re: [O] orgtbl export to latex :fmt() fails, Nick Dokos, 2014/07/18