[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in comm
From: |
Viktor Rosenfeld |
Subject: |
Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]) |
Date: |
Sat, 25 May 2013 19:03:22 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Hi Rasmus,
Rasmus wrote:
> The following message is a courtesy copy of an article
> that has been posted to gmane.emacs.orgmode as well.
>
> Alan Schmitt <address@hidden> writes:
>
> > Hello,
> >
> > Viktor Rosenfeld writes:
> >
> >> Hi Robert,
> >>
> >> Robert Klein wrote:
> >>
> >>> Hi,
> >>>
> >>> FWIW, from a users view it would be nice if:
> >>>
> >>> - Use Author/Email information from org file
> >>> - If not present use information from LCO file
> >>> - if neither org file nor LCO file has any information use
> >>> user-full-name and user-email-address
> >>>
> >>> Could this be solved by having several e.g. `setkomavar{fromname}'
> >>> and so on in the tex file, so is created as follows:
>
> I'd go with 'no'. It's not aesthetically pleasing and I don't want my
> output to look like LyX. When feasible we should go for beautiful
> output. This isn't always the case at the moment, but still.
I agree that there should not be multiple instances of, e.g.,
\setkomavar{fromname} in the TeX file. I must have overlooked that bit
in the original mail.
> On a side-note, Viktor: this seems to be the default in scrletter
> anyway:
> >>> add \setkomavar{signature}{\usekomavar{fromname}}
> Could we remove it? I'd like us to get to a more clean template (C-e
> # koma-letter RET).
I think so, yes.
> >> This is what is implemented by the latest patch
> >> (http://thread.gmane.org/gmane.emacs.orgmode/72430/focus=72525).
> >
> > I'm waiting for Rasmus's confirmation that it works for him before
> > committing it.
>
> Thanks and sorry for the wait. No it didn't work for me. My user
> name was always overwritten by "". . . I couldn't figure out why.
Hmm, that's too bad. I tested it pretty thoroughly. Could you maybe
trace the contents of the variable by adding calls to message in various
places?
> I've attached a patch that work for me (it goes on top of Viktor's
> patch 148c737ae79f3a98d8e93147c2d0ec0db3a2389a). It allows for nil
> and it gets up-to-date default values by default. In my book it's a
> bit more clean 'cause it doesn't rely on hooks. It does, introduce a
> new helper function to distinguish between a function value (which are
> default for the two variables) and a string value (and nil for that
> matter). I don't know if this is undesirable. It would crash if you
> set the variables to a symbol that isn't nil and isn't a function.
Did you send the patch? I did not receive it and it's not available on
gmane.
Cheers,
Viktor
>
> It seems to work in mine and Viktor's use-case (to the best of my
> testing ability).
>
> –Rasmus
>
> --
> ⠠⠵
>
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], (continued)
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], Viktor Rosenfeld, 2013/05/21
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], Rasmus, 2013/05/21
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], Viktor Rosenfeld, 2013/05/21
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], Rasmus, 2013/05/21
- Re: [O] [patch] ox-koma-letter.el: clean-up/semantic bug [4/4], Viktor Rosenfeld, 2013/05/22
- [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]), Viktor Rosenfeld, 2013/05/22
- Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]), Robert Klein, 2013/05/25
- Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]), Viktor Rosenfeld, 2013/05/25
- Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]), Alan Schmitt, 2013/05/25
- Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]), Rasmus, 2013/05/25
- Re: [O] [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults (was Re: [patch] ox-koma-letter.el: clean-up/semantic bug [4/4]),
Viktor Rosenfeld <=
- [O] [PATCH][ox-koma-letter]: sender, email and cleanup (was: [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults), Rasmus, 2013/05/25
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Rasmus, 2013/05/25
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Viktor Rosenfeld, 2013/05/26
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Rasmus, 2013/05/26
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup (was: [PATCH] ox-koma-letter.el: Reintroduce variables removed in commit 832c6fd with proper defaults), Viktor Rosenfeld, 2013/05/26
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Rasmus, 2013/05/26
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Alan Schmitt, 2013/05/27
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Xavier Garrido, 2013/05/27
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Rasmus, 2013/05/27
- Re: [O] [PATCH][ox-koma-letter]: sender, email and cleanup, Viktor Rosenfeld, 2013/05/27