Re: [O] [PATCH] org-rss-headline

2015-01-21 Thread Nicolas Goaziou
Nicolas Petton writes: > Ahh, sorry about that. I'm not used to contributing through > emails. Would cloning the repository and create branches that can be > merged into the main repository be more convenient? Cloning the repository would help. You can create branches for large patches and use f

Re: [O] [PATCH] org-rss-headline

2015-01-20 Thread Nicolas Petton
Nicolas Goaziou writes: > Nicolas Petton writes: > >> Hi guys, >> >> Any update on this? > > Applied, thank you. > > I forgot to mention it, but next time, please provide patch with "git > format-patch" and a proper commit message. Ahh, sorry about that. I'm not used to contributing through em

Re: [O] [PATCH] org-rss-headline

2015-01-20 Thread Nicolas Goaziou
Nicolas Petton writes: > Hi guys, > > Any update on this? Applied, thank you. I forgot to mention it, but next time, please provide patch with "git format-patch" and a proper commit message. Regards,

Re: [O] [PATCH] org-rss-headline

2015-01-20 Thread Nicolas Petton
Hi guys, Any update on this? Cheers, Nico Nicolas Petton writes: > Indeed, sorry about that. > Here's a new patch with a short description of the RSS_TITLE property. > > > Cheers, > Nico -- Nicolas Petton http://nicolas-petton.fr

Re: [O] [PATCH] org-rss-headline

2015-01-18 Thread Nicolas Petton
Indeed, sorry about that. Here's a new patch with a short description of the RSS_TITLE property. diff --git a/contrib/lisp/ox-rss.el b/contrib/lisp/ox-rss.el index fddaa1d..6681336 100644 --- a/contrib/lisp/ox-rss.el +++ b/contrib/lisp/ox-rss.el @@ -42,6 +42,9 @@ ;; PUBDATE property. If `org-rss

Re: [O] [PATCH] org-rss-headline

2015-01-16 Thread Nicolas Goaziou
Nicolas Petton writes: > Here's a new version that uses :RSS_TITLE. Do you think it's good enough > for a first patch? Thanks. It would be nice to document the feature somewhere in the library. > +(title (or (org-element-property :EXPORT_TITLE headline) I think you sent the wrong patch

Re: [O] [PATCH] org-rss-headline

2015-01-14 Thread Nicolas Petton
Hi, Here's a new version that uses :RSS_TITLE. Do you think it's good enough for a first patch? I'll work on the export back-end, but that'll take some more time. diff --git a/contrib/lisp/ox-rss.el b/contrib/lisp/ox-rss.el index fddaa1d..5617d26 100644 --- a/contrib/lisp/ox-rss.el +++ b/contrib/

Re: [O] [PATCH] org-rss-headline

2015-01-13 Thread Nicolas Petton
Nicolas Goaziou writes: > Nicolas Petton writes: > >> If the headline contains *bold* text for example, the RSS item title >> will have the extras * characters, since it's using the :raw-value. > > OK. But what is expected instead? HTML markup? Nothing at all? I'd say the content of headline w

Re: [O] [PATCH] org-rss-headline

2015-01-13 Thread Nicolas Goaziou
Nicolas Petton writes: > If the headline contains *bold* text for example, the RSS item title > will have the extras * characters, since it's using the :raw-value. OK. But what is expected instead? HTML markup? Nothing at all? > Ok. What about an :RSS_TITLE property? Or should I have a look at

Re: [O] [PATCH] org-rss-headline

2015-01-13 Thread Nicolas Petton
Nicolas Goaziou writes: > Hello, > > Nicolas Petton writes: > >> I'm a happy user of ox-rss.el, but was annoyed the other day when I >> wanted to export an article with a headline containing markup. > > What problem did you encounter? If the headline contains *bold* text for example, the RSS i

Re: [O] [PATCH] org-rss-headline

2015-01-13 Thread Nicolas Goaziou
Hello, Nicolas Petton writes: > I'm a happy user of ox-rss.el, but was annoyed the other day when I > wanted to export an article with a headline containing markup. What problem did you encounter? > The problem is that org-rss-headlines uses the raw-value of the headline > to export the title