Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-04-07 Thread Rasmus
Myles English writes: > A patch is attached that fixes some naughty brackets in the recent commit. Don't worry: I probably added those by accident when rebasing. Anyway, I pushed both patches and updated the Worg page. Let me know if everything is OK. Rasmus -- Bang

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-04-03 Thread Myles English
>From 2d9b513e14f6d34fe81a5970fd4cee24fc253a31 Mon Sep 17 00:00:00 2001 From: Myles English Date: Mon, 4 Apr 2016 01:11:19 +0100 Subject: [PATCH] ox-koma-letter: Fixed bug in filling 'location' field * contrib/lisp/ox-koma-letter.el: Fix code apparently introduced in

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-04-03 Thread Rasmus
Myles English writes: > Thanks for looking at this again. I'm just sorry about the delay, but I hope you can appreciate that sometimes time is at a premium. >> Is :with-location something you feel strongly about? I'm not sure I find >> that it makes sense... >> >> I

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-03-31 Thread Myles English
Hi Rasmus, Thanks for looking at this again. Rasmus Pank Roulund writes: > Myles, > > Is :with-location something you feel strongly about? I'm not sure I find > that it makes sense... > > I would like to apply the following patch on top of yours. I think that's alright. Just to recap: I use

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-03-31 Thread Rasmus Pank Roulund
Myles, Is :with-location something you feel strongly about? I'm not sure I find that it makes sense... I would like to apply the following patch on top of yours. Thanks, Rasmus -- I almost cut my hair, it happened just the other day >From 43a26688870f6861747a1d333a5844348d3a1749 Mon Sep 17

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-02-17 Thread Rasmus
Myles English writes: > Rasmus writes: > >> I will try to merge your patch soon, this weekend. Again, unless someone >> beats me to it. > > Please would someone apply this patch? I will *try* to write the remaining code to maintain feature parity between keywords and

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-02-15 Thread Myles English
Rasmus writes: > I will try to merge your patch soon, this weekend. Again, unless someone > beats me to it. Please would someone apply this patch? Thanks, Myles >From 4495abd39d57abc0dd64361f852c3a638f86915b Mon Sep 17 00:00:00 2001 From: Myles English Date: Wed, 27

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-29 Thread Rasmus
Hi Myles, Myles English writes: >> But is it much neater than: >> >> ** bank details :location: >> Acc. no.: 12345 >> Big Bank Corp >> Sort code: 1122 > > Err, yes it is? And anyway, that data is in my LCO file, I don't want > to duplicate it. That's

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-29 Thread Myles English
Rasmus, Rasmus writes: > I’m arguing for a solution that supports BOTH "one-off" and permanent > data. OK, sorry, I thought you were arguing for a heading tagged :location: and against an option #+LOCATION or property :EXPORT_LOCATION:. How about applying my patch please and the tagged

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-29 Thread Rasmus
Myles English writes: >> I’m arguing for a solution that supports BOTH "one-off" and permanent >> data. > > OK, sorry, I thought you were arguing for a heading tagged :location: > and against an option #+LOCATION or property :EXPORT_LOCATION:. How > about applying my

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-28 Thread Eric S Fraga
On Wednesday, 27 Jan 2016 at 23:05, Rasmus wrote: > It always makes me happy when someone is using ox-koma-letter. Just to let you know that I use ox-koma-letter all the time! It took a while before I started using it but that was mostly because I need to figure out how to do my organisation's

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-28 Thread Myles English
Rasmus writes: > Myles English writes: > >> Thanks for making it! > > It's Nicolas’ creation, not mine. I merely contribute a few patches > to it. Thanks to Nicolas too. >>> Aside: it sounds like what you need is "yourref", maybe "yourmail", >>> "myref", "customer",

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-28 Thread Rasmus
Myles English writes: > Thanks for making it! It's Nicolas’ creation, not mine. I merely contribute a few patches to it. >> Aside: it sounds like what you need is "yourref", maybe "yourmail", >> "myref", "customer", and "invoice". These are typed in the same line as

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-28 Thread Myles English
Hi Rasmus, Rasmus writes: > Thanks for your patch. > > It always makes me happy when someone is using ox-koma-letter. Thanks for making it! >> Often when writing a letter I want to include "Your Ref: xyz" or >> "Account No.: 1234" and a good place to do this is below the sender's >> address

Re: [O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-27 Thread Rasmus
Hi Myles, Thanks for your patch. It always makes me happy when someone is using ox-koma-letter. Myles English writes: > Often when writing a letter I want to include "Your Ref: xyz" or > "Account No.: 1234" and a good place to do this is below the sender's > address

[O] [PATCH] ox-koma-letter.el: Add support for 'location' koma variable

2016-01-27 Thread Myles English
Hello, Often when writing a letter I want to include "Your Ref: xyz" or "Account No.: 1234" and a good place to do this is below the sender's address field and above the date (and 'place') field(s). This space becomes filled with the contents of the koma variable called 'location'. This patch