Re: [O] Markup problems when using references in source fragments (was: Markup on same line as text)
On 25.5.2011, at 09:41, Carsten Dominik wrote: You patch has not been accepted and merged. I mean it has *now* been accepted. Sorry for the typo. - Carsten - Carsten On 11.5.2011, at 21:41, Roland Kaufmann wrote: On February 27th I submitted a patch proposal to fix HTML export of line number references in source code fragments. The discussion leading up to the patch can be found in the thread: http://thread.gmane.org/gmane.emacs.orgmode/35810 As far as I can see, it is not in the source tree as of revision 62d4dd, but the problem still remains. I am grateful for feedback on its lackings, if the patch is deemed unsatisfactory in any way. In case it simply got lost amidst all the other traffic, I resubmit it, updated so it applies to the current revision. -- Roland. 0001-Fix-markup-problems-when-using-ref.patch
Re: [O] Markup problems when using references in source fragments (was: Markup on same line as text)
You patch has not been accepted and merged. - Carsten On 11.5.2011, at 21:41, Roland Kaufmann wrote: On February 27th I submitted a patch proposal to fix HTML export of line number references in source code fragments. The discussion leading up to the patch can be found in the thread: http://thread.gmane.org/gmane.emacs.orgmode/35810 As far as I can see, it is not in the source tree as of revision 62d4dd, but the problem still remains. I am grateful for feedback on its lackings, if the patch is deemed unsatisfactory in any way. In case it simply got lost amidst all the other traffic, I resubmit it, updated so it applies to the current revision. -- Roland. 0001-Fix-markup-problems-when-using-ref.patch
Re: [O] Markup problems when using references in source fragments
Bernt Hansen be...@norang.ca wrote: Roland Kaufmann rlndkfmn+orgm...@gmail.com writes: On 2011-05-11 21:51, Bernt Hansen wrote: Both the new and old patch are on the patchwork server waiting to be addressed. Oops, sorry my bad. I didn't realize that! However, only the discussion from the first thread: http://patchwork.newartisans.com/patch/525/ is revelant. The patch which is listed there should *not* be applied. A second and corrected patch was attached in the last message, but it seems to not have been picked up by the framework. How do I replace the patch above with the code from: http://patchwork.newartisans.com/patch/802/ -- Roland. I think you can create an account on the patchwork server and then invalidate your own patches. If this is possible, can somebody please give details? There is at least one post of mine that ended up as /patch/797/ that should certainly be invalidated, since it quotes the patch from an older commit. And while we are at it: is there any way to mark a post that contains a patch as not intended for the patchwork server? One would have to remember to do that, so it's not foolproof, but it might help avoid piling irrelevant stuff on patchwork. Thanks, Nick
Re: [O] Markup problems when using references in source fragments
Nick Dokos nicholas.do...@hp.com writes: Bernt Hansen be...@norang.ca wrote: Roland Kaufmann rlndkfmn+orgm...@gmail.com writes: . The patch which is listed there should *not* be applied. I think you can create an account on the patchwork server and then invalidate your own patches. If this is possible, can somebody please give details? And while we are at it: is there any way to mark a post that contains a patch as not intended for the patchwork server? If you think that a patch is not intended for the patchwork server you may write an e-mail to the patchwork maintainers: John Wiegley jo...@newartisans.com Carsten Dominik carsten.domi...@gmail.com Eric Schulte schulte.e...@gmail.com Bastien Guerry b...@altern.org Dan Davison dandavis...@gmail.com David Maus dm...@ictsoc.de Julien Danjou jul...@danjou.info with the subject: REMOVE patches and link the number of the patches to be removed. I think they can do that easily from the command line. I have done this way 3 or 4 times in the past. I also added some comments/justification: e.g. the link of the e-mail pointing out that it is a wrong patch or incomplete or... HTH Giovanni
Re: [O] Markup problems when using references in source fragments
Nick Dokos nicholas.do...@hp.com writes: Bernt Hansen be...@norang.ca wrote: Roland Kaufmann rlndkfmn+orgm...@gmail.com writes: On 2011-05-11 21:51, Bernt Hansen wrote: Both the new and old patch are on the patchwork server waiting to be addressed. Oops, sorry my bad. I didn't realize that! However, only the discussion from the first thread: http://patchwork.newartisans.com/patch/525/ is revelant. The patch which is listed there should *not* be applied. A second and corrected patch was attached in the last message, but it seems to not have been picked up by the framework. How do I replace the patch above with the code from: http://patchwork.newartisans.com/patch/802/ -- Roland. I think you can create an account on the patchwork server and then invalidate your own patches. If this is possible, can somebody please give details? There is at least one post of mine that ended up as /patch/797/ that should certainly be invalidated, since it quotes the patch from an older commit. Hi Nick, I think you just register an account on the patchwork server that matches your patch email address, then when you login and select your patch you get an option to invalidate it. And while we are at it: is there any way to mark a post that contains a patch as not intended for the patchwork server? One would have to remember to do that, so it's not foolproof, but it might help avoid piling irrelevant stuff on patchwork. I'm not aware of a way to do this currently - maybe just put [RFC] or something in the subject line of the patch so it's obviously not complete and ready for inclusion? - then you can just remove the patch from the patchwork server after posting it. I don't know if the patchwork server has any rules for skipping patches. Regards, Bernt
[O] Markup problems when using references in source fragments (was: Markup on same line as text)
On February 27th I submitted a patch proposal to fix HTML export of line number references in source code fragments. The discussion leading up to the patch can be found in the thread: http://thread.gmane.org/gmane.emacs.orgmode/35810 As far as I can see, it is not in the source tree as of revision 62d4dd, but the problem still remains. I am grateful for feedback on its lackings, if the patch is deemed unsatisfactory in any way. In case it simply got lost amidst all the other traffic, I resubmit it, updated so it applies to the current revision. -- Roland. From 98e2a586eb0e911ec6b5bedeec4af5f00ee2bf6c Mon Sep 17 00:00:00 2001 From: Roland Kaufmann rlndkfmn+orgm...@gmail.com Date: Sun, 27 Feb 2011 20:52:31 +0100 Subject: [PATCH] Fix markup problems when using references in source fragments * org-exp.el (org-export-format-source-code-or-example): fontify one line at the time to avoid partial overlap between fontification and reference markup. --- lisp/org-exp.el | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/lisp/org-exp.el b/lisp/org-exp.el index 9a35b00..dbcf105 100644 --- a/lisp/org-exp.el +++ b/lisp/org-exp.el @@ -2446,6 +2446,15 @@ in the list) and remove property and value from the list in LISTVAR. (defvar org-export-latex-listings-options) ;; defined in org-latex.el (defvar org-export-latex-minted-options) ;; defined in org-latex.el +(defun org-remove-formatting-on-newlines-in-region (beg end) + Remove formatting on newline characters + (interactive r) + (save-excursion +(goto-char beg) +(while (progn (end-of-line) ( (point) end)) + (put-text-property (point) (1+ (point)) 'face nil) + (forward-char 1 + (defun org-export-format-source-code-or-example (lang code optional opts indent caption) Format CODE from language LANG and return it formatted for export. @@ -2532,6 +2541,8 @@ INDENT was the original indentation of the block. (funcall mode) (fundamental-mode)) (font-lock-fontify-buffer) + ;; markup each line separately + (org-remove-formatting-on-newlines-in-region (point-min) (point-max)) (org-src-mode) (set-buffer-modified-p nil) (org-export-htmlize-region-for-paste -- 1.7.1
Re: [O] Markup problems when using references in source fragments
Roland Kaufmann rlndkfmn+orgm...@gmail.com writes: On February 27th I submitted a patch proposal to fix HTML export of line number references in source code fragments. The discussion leading up to the patch can be found in the thread: http://thread.gmane.org/gmane.emacs.orgmode/35810 As far as I can see, it is not in the source tree as of revision 62d4dd, but the problem still remains. I am grateful for feedback on its lackings, if the patch is deemed unsatisfactory in any way. In case it simply got lost amidst all the other traffic, I resubmit it, updated so it applies to the current revision. Hi Roland, Both the new and old patch are on the patchwork server waiting to be addressed. http://patchwork.newartisans.com/project/org-mode/list/ Regards, Bernt
Re: [O] Markup problems when using references in source fragments
On 2011-05-11 21:51, Bernt Hansen wrote: Both the new and old patch are on the patchwork server waiting to be addressed. Oops, sorry my bad. I didn't realize that! However, only the discussion from the first thread: http://patchwork.newartisans.com/patch/525/ is revelant. The patch which is listed there should *not* be applied. A second and corrected patch was attached in the last message, but it seems to not have been picked up by the framework. How do I replace the patch above with the code from: http://patchwork.newartisans.com/patch/802/ -- Roland.
Re: [O] Markup problems when using references in source fragments
Roland Kaufmann rlndkfmn+orgm...@gmail.com writes: On 2011-05-11 21:51, Bernt Hansen wrote: Both the new and old patch are on the patchwork server waiting to be addressed. Oops, sorry my bad. I didn't realize that! However, only the discussion from the first thread: http://patchwork.newartisans.com/patch/525/ is revelant. The patch which is listed there should *not* be applied. A second and corrected patch was attached in the last message, but it seems to not have been picked up by the framework. How do I replace the patch above with the code from: http://patchwork.newartisans.com/patch/802/ -- Roland. I think you can create an account on the patchwork server and then invalidate your own patches. -Bernt