Re: Re: Bug: #+ATTR_LaTeX broken in LaTeX export [6.36trans (release_6.36.461.g798e)]

["Eric Schulte" <schulte.eric@gmail.com>]

Hi Nicolas,

Nicolas Goaziou <n.goaziou@gmail.com> writes:

> Hi Eric,
>
> I do not think this is the right fix. You are only allowing lines
> like :
>
> ---
>      # a line of text
> ---
>
> to be treated as comments, when they are not! The first regex is the
> one matching comments definition (# on column 0 and '#+ ' everywhere).
> Handling of lines like #+ATTR_LaTeX should be processed outside of
> comments handling functions to not add confusion.
>
> Moreover, I mailed a fix earlier (see thread with Sebastian Rose:
> unwanted #+attr_backend and #+backend stuff) which already solves that
> problem, among others.
>

Yes, this patch has been floundering on the patchwork server for a while
now.  I've been reticent to apply it hoping that someone with more
familiarity with the export engine would take care of it.  At this point
however I think delaying it's application is doing more hard than good.

I'd like to apply this, however one last question.  This patch doesn't
address Sébastien's table issue, namely

--8<---------------cut here---------------start------------->8---
** table comment issue
#+BEGIN_changemargin {-4.2cm}{0cm}
  #+TBLNAME: AutresFPNVE
  #+ATTR_LaTeX: align=lrrrrr
  |   |                                          | Montant total (\EUR) | Taux amort (\%) | Part pro. (\%) | Déduc (\%) | NVE (\EUR) |
  |---+------------------------------------------+----------------------+-----------------+----------------+------------+------------|
  |   | Documentation et formation               |                51.05 |                 |                |            |       0.00 |
  |   | Communications GSM                       |               831.16 |             100 |             25 |        100 |     207.79 |
  |   | Internet (Dommel)                        |               167.88 |             100 |             33 |        100 |      55.40 |
  |   | Fournitures à amortir (ordinateur + GSM) |               762.51 |              33 |             80 |        100 |     201.30 |
  |   | Restaurant                               |               304.70 |             100 |            100 |         69 |     210.24 |
  |---+------------------------------------------+----------------------+-----------------+----------------+------------+------------|
  |   | Total                                    |                      |                 |                |            |    1062.02 |
  | ^ |                                          |                      |                 |                |            |      Total |
  #+TBLFM: $7=$3*$4*$5*$6/1000000;%.2f::@2$3=51.05::@3$3=9.00+184.88+51.22+201.82+45.67+69.03+62.93+54.16+38.87+39.77+36.35+37.46::@4$3=12*13.99::@6$3=146.50+158.20;%.2f::@7$7=vsum(@-I..@-II);%.2f
#+END_changemargin
--8<---------------cut here---------------end--------------->8---

Before I apply, should the patch be updated to handle this case?  Let me
know and I'll either apply the current version of the patch, or an
updated version.

Thanks for your patience -- Eric

>
> Regards,
>
> -- Nicolas