Hello,
On Mon, Jul 10, 2017 at 3:54 PM Kaushal Modi <kaushal.m...@gmail.com> wrote:
> Hi Nicolas,
>
> I tried out the patch, and my comments are in the MWE below.
>
> I have this MWE that explains the "would like to have" vs "what this patch
> gives us":
>
> =====
> #+TITLE: ox-html root dir
> #+OPTIONS: toc:nil
> #+STARTUP: inlineimages
> #+HTML_LINK_ROOT: /tmp/site/static/
>
> An =image.png= exists as =/tmp/site/static/images/image.png=.
>
> * This is how the image links should have worked with the link root
> already specified
> [[/images/image.png]]
> - The =HTML_LINK_ROOT= should to appended to the front of the above link
> if that
> concatenated path is a valid one.
> - Would also like =C-x C-o=, inline image display, etc. to work.
> - So a thought.. should the link root specification be made exporter
> agnostic?
>
To expand on my MWE:
I have this patch on top of Nicolas's patch:
=====
diff --git a/lisp/ox-html.el b/lisp/ox-html.el
index 9e710e3e0a..410bb42eec 100644
--- a/lisp/ox-html.el
+++ b/lisp/ox-html.el
@@ -3011,11 +3011,12 @@ INFO is a plist holding contextual information. See
((file-name-absolute-p raw-path)
(let ((root (plist-get info :html-link-root)))
(setq raw-path
- (if (and root (file-in-directory-p raw-path root))
- (concat "/"
- (file-relative-name
- (expand-file-name raw-path)
- root))
+ (if (and root
+ (file-exists-p
+ (concat (file-name-as-directory root)
+ (replace-regexp-in-string
+ "\\`/" "" raw-path))))
+ raw-path
(org-export-file-uri raw-path)))))
((and home use-abs-url)
(setq raw-path (expand-file-name raw-path home))))
=====
This adds the root and link instead of having the root be a subset of link
(hope that makes sense).
With that, here is the updated MEW:
=====
#+TITLE: ox-html root dir
#+OPTIONS: toc:nil
#+STARTUP: inlineimages
#+HTML_LINK_ROOT: /tmp/site/static/
An =image.png= exists at =/tmp/site/static/images/image.png=.
* TODO [1/2] This is how the image links should have worked with the link
root already specified
[[/images/image.png]]
- [X] The =HTML_LINK_ROOT= should to appended to the front of the above
link if
that concatenated path is a valid one.
- [ ] Would also like =C-x C-o=, inline image display, etc. to work.
- So a thought.. should the link root specification be made exporter
agnostic?
=====
--
Kaushal Modi
