Hello,

On Mon, Jul 10, 2017 at 3:54 PM Kaushal Modi <kaushal.modi@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