Re: [O] [PATCH] Override the default class name with a new one.

2013-06-18 Thread Rasmus
Nicolas Goaziou writes: >> 1. we can add a function `(defun org-latex-add-deriverd-class (class >> exist-class) ...) to menupulate `org-latex-classes >> 2. My patch's approach. >> 3. others. > > We can modify `org-latex-classes' docstring (e.g. by adding examples) so > it is easier to understand

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-18 Thread Nicolas Goaziou
Hello, Feng Shu writes: > Not all the org-mode users are expert of lisp and not all of the org-mode > users are well in English, so finding the variable `org-latex-classes can > customize latex class and adding a lisp block to it may be not a easy > things for many users. I understand, and I

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Feng Shu
Nicolas Goaziou writes: > Hello, > > Feng Shu writes: > >> #+LATEX_CLASS: article >> >> %%output: >> >> \documentclass[11pt]{article} >> >> ... >> >> -- >> >> #+LATEX_CLASS: article >> #+LATEX_CLASS_NAME: ctexart >> >> %%output: >> >> \documentclass[11pt]{ctex

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Nicolas Goaziou
Hello, Feng Shu writes: > #+LATEX_CLASS: article > > %%output: > > \documentclass[11pt]{article} > > ... > > -- > > #+LATEX_CLASS: article > #+LATEX_CLASS_NAME: ctexart > > %%output: > > \documentclass[11pt]{ctexart} > > Thanks for your patch. Though, i

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Suvayu Ali
On Fri, Jun 14, 2013 at 05:55:56PM +0800, Feng Shu wrote: > > CLASS and class-name are confusing, I think LATEX_CLASS should rename > to LATEX_CLASS_FORMAT or LATEX_CLASS_TEMPLATE, but this will break > compatibility. IMO compatibility is a big and important issue, and not to be taken lightly.

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Feng Shu
Rasmus writes: > Feng Shu writes: > >> #+LATEX_CLASS: article >> #+LATEX_CLASS_NAME: ctexart > > I'm not sure this is the right approach. . . I can certainly see the > value of generating classes on the fly, though. But somehow the > proposed syntax just seems to counter-intuitive. I'm not su

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Feng Shu
Nick Dokos writes: > Rasmus writes: > >> Feng Shu writes: >> >>> #+LATEX_CLASS: article >>> #+LATEX_CLASS_NAME: ctexart >> >> I'm not sure this is the right approach. . . I can certainly see the >> value of generating classes on the fly, though. But somehow the >> proposed syntax just seems t

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Rasmus
Nick Dokos writes: >> as e.g. the INCLUDE command. Of course (*) is somewhat different from >> how stuff like this is usually handled, cf. e.g. LATEX_CLASS_OPTIONS. >> > ...but this sounds a bit more promising: maybe LATEX_CLASS_OPTIONS can > be eliminated in favor of LATEX_CLASS with arguments:

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Nick Dokos
Rasmus writes: > Feng Shu writes: > >> #+LATEX_CLASS: article >> #+LATEX_CLASS_NAME: ctexart > > I'm not sure this is the right approach. . . I can certainly see the > value of generating classes on the fly, though. But somehow the > proposed syntax just seems to counter-intuitive. I'm not su

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-14 Thread Rasmus
Feng Shu writes: > #+LATEX_CLASS: article > #+LATEX_CLASS_NAME: ctexart I'm not sure this is the right approach. . . I can certainly see the value of generating classes on the fly, though. But somehow the proposed syntax just seems to counter-intuitive. I'm not sure how to overcome this, but

Re: [O] [PATCH] Override the default class name with a new one.

2013-06-13 Thread Feng Shu
Feng Shu writes: > #+LATEX_CLASS: article > > %%output: > > \documentclass[11pt]{article} > > ... > > -- > > #+LATEX_CLASS: article > #+LATEX_CLASS_NAME: ctexart > > %%output: > > \documentclass[11pt]{ctexart} Hi: If possible, please merge this patch to maste

[O] [PATCH] Override the default class name with a new one.

2013-06-12 Thread Feng Shu
#+LATEX_CLASS: article %%output: \documentclass[11pt]{article} ... -- #+LATEX_CLASS: article #+LATEX_CLASS_NAME: ctexart %%output: \documentclass[11pt]{ctexart} -- >From 338ce85c306ae400ba8c62bfaecaf8973346faa0 Mon Sep 17 00:00:00 2001 From: Feng