Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not work with hugo_extended_0.100.1_Linux-64bit #452

Closed
vossenjp opened this issue Jun 3, 2022 · 5 comments
Closed

Does not work with hugo_extended_0.100.1_Linux-64bit #452

vossenjp opened this issue Jun 3, 2022 · 5 comments

Comments

@vossenjp
Copy link

vossenjp commented Jun 3, 2022

A brand new git submodule add https://github.com/alex-shpak/hugo-book themes/hugo-book install with cp -R themes/hugo-book/exampleSite/content . fails to build:

$ ../hugo server -D
Start building sites … 
hugo v0.100.1-0afb4866e345d31cbbcbab4349e43f1d36122806+extended linux/amd64 BuildDate=2022-06-01T10:11:48Z VendorInfo=gohugoio
WARN 2022/06/03 16:54:43 Expand shortcode is deprecated. Use 'details' instead.
Error: Error building site: "/path/to/site/content/docs/shortcodes/mermaid.md:14:1": failed to render shortcode "columns": failed to process shortcode: "/path/to/site/themes/hugo-book/layouts/shortcodes/columns.html:4:11": execute of template failed: template: shortcodes/columns.html:4:11: executing "shortcodes/columns.html" at <markdownify>: error calling markdownify: "/path/to/site/content/_index.md:3:13": unrecognized character in shortcode action: U+005B '['. Note: Parameters with non-alphanumeric args must be quoted
Built in 216 ms

Downgrading to hugo_extended_0.99.1_Linux-64bit.tar.gz works as expected:

$ ../hugo server -D
Start building sites … 
hugo v0.99.1-d524067382e60ce2a2248c3133a1b3af206b6ef1+extended linux/amd64 BuildDate=2022-05-18T11:18:14Z VendorInfo=gohugoio
WARN 2022/06/03 16:59:31 Expand shortcode is deprecated. Use 'details' instead.

                   | EN  
-------------------+-----
  Pages            | 57  
  Paginator pages  |  0  
  Non-page files   |  0  
  Static files     | 80  
  Processed images |  0  
  Aliases          | 11  
  Sitemaps         |  1  
  Cleaned          |  0  

Built in 259 ms
Watching for changes in /path/to/site/{archetypes,content,data,layouts,static,themes}
Watching for config changes in /path/to/site/config.toml
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at http://localhost:1313/ (bind address 127.0.0.1)
Press Ctrl+C to stop
@alex-shpak
Copy link
Owner

Hi!
Reproducing for me as well, will need to dig what changes were done for rendering markdown with a function

@alex-shpak
Copy link
Owner

alex-shpak commented Jun 7, 2022

Could be related
gohugoio/hugo@9e904d7
gohugoio/hugo#6703

@alex-shpak
Copy link
Owner

Welp, I didn't find smooth solution, for nwo we have to use pre 100 version of hugo.
I will pin this issue.

@hammonba
Copy link

hammonba commented Oct 11, 2022

just playing with this as a hugo & go n00b:
apologies if am restating the bleedin obvious

Seems like the contents of the triple-backtick codeblock is NOT getting passed to as a single CLOB parameter in to shortcodes/columns.html , but instead its contents are getting passed
and the sortcode is then barfing on the [ inside

{{</* mermaid [class="text-center"]*/>}}
...

deleting the two offending files as mentioned in

#471 (comment)

is a good enough workaround for me though

@alex-shpak
Copy link
Owner

Well, all this time I looked at wrong place, but it was enough to escape nested shortcode with double /*/*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants