projects / brevno-suite / hugo / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: bdca972)
helpers: Call rst2html directly on *nix
authorShreyansh Khajanchi <shreyanshk@users.noreply.github.com>
Thu, 11 Oct 2018 20:46:10 +0000 (20:46 +0000)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Thu, 11 Oct 2018 20:46:10 +0000 (22:46 +0200)
Initially, rst2html was called via the python interpreter which would
fail if the script was wrapped in a launcher as on NixOS.
Ideally, on *nix, binaries should be invoked directly to ensure that
shebangs work properly as is being done now.
Handle the case of windows as it doesn't do shebangs.

helpers/content.go patch | blob | history

diff --git a/helpers/content.go b/helpers/content.go
index 55d8ce202be75ee38c6badebb11fe5ae24e602af..f8479cd1b9ae9b8810e3c86dfe26b8a720c87963 100644 (file)
--- a/helpers/content.go
+++ b/helpers/content.go
@@ -22,6 +22,7 @@ import (
        "fmt"
        "html/template"
        "os/exec"
+       "runtime"
        "unicode"
        "unicode/utf8"
 
@@ -678,7 +679,6 @@ func getPythonExecPath() string {
 // getRstContent calls the Python script rst2html as an external helper
 // to convert reStructuredText content to HTML.
 func getRstContent(ctx *RenderingContext) []byte {
-       python := getPythonExecPath()
        path := getRstExecPath()
 
        if path == "" {
@@ -688,8 +688,19 @@ func getRstContent(ctx *RenderingContext) []byte {
 
        }
        jww.INFO.Println("Rendering", ctx.DocumentName, "with", path, "...")
-       args := []string{path, "--leave-comments", "--initial-header-level=2"}
-       result := externallyRenderContent(ctx, python, args)
+       var result []byte
+       // certain *nix based OSs wrap executables in scripted launchers
+       // invoking binaries on these OSs via python interpreter causes SyntaxError
+       // invoke directly so that shebangs work as expected
+       // handle Windows manually because it doesn't do shebangs
+       if runtime.GOOS == "windows" {
+               python := getPythonExecPath()
+               args := []string{path, "--leave-comments", "--initial-header-level=2"}
+               result = externallyRenderContent(ctx, python, args)
+       } else {
+               args := []string{"--leave-comments", "--initial-header-level=2"}
+               result = externallyRenderContent(ctx, path, args)
+       }
        // TODO(bep) check if rst2html has a body only option.
        bodyStart := bytes.Index(result, []byte("<body>\n"))
        if bodyStart < 0 {
Unnamed repository; edit this file 'description' to name the repository.