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

Regression causing Documenter to crash when compiling ApproxFun manual #1073

Closed
thisrod opened this issue Jul 25, 2019 · 3 comments · Fixed by #1075
Closed

Regression causing Documenter to crash when compiling ApproxFun manual #1073

thisrod opened this issue Jul 25, 2019 · 3 comments · Fixed by #1075
Milestone

Comments

@thisrod
Copy link

thisrod commented Jul 25, 2019

Documenter version v0.22.5 worked. Version 0.23.0 crashed.

The manual didn't change between those commits.

@thisrod thisrod changed the title Regression compiling ApproxFun manual Regression and crash compiling ApproxFun manual Jul 25, 2019
@thisrod thisrod changed the title Regression and crash compiling ApproxFun manual Regression causing Documenter to crash when compiling ApproxFun manual Jul 25, 2019
@thisrod
Copy link
Author

thisrod commented Jul 25, 2019

Here's a minimal example:

$ cd /tmp/A
$ ls -R
src

./src:
index.md
$ cat src/index.md 
$$
f
$$
$ julia -e 'using Documenter; makedocs()'
[ Info: SetupBuildDirectory: setting up build directory.
┌ Error: Markdown2.convert failed to convert src/index.md
└ @ Documenter.Documents ~/.julia/packages/Documenter/oEuOX/src/Documents.jl:62
ERROR: MethodError: no method matching _convert_block(::Symbol)
Closest candidates are:
  _convert_block(!Matched::Markdown.Admonition) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Utilities/Markdown2.jl:241
  _convert_block(!Matched::Markdown.Table) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Utilities/Markdown2.jl:234
  _convert_block(!Matched::Markdown.Footnote) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Utilities/Markdown2.jl:232
  ...
Stacktrace:
 [1] iterate at ./generator.jl:47 [inlined]
 [2] _collect(::Array{Any,1}, ::Base.Generator{Array{Any,1},typeof(Documenter.Utilities.Markdown2._convert_block)}, ::Base.EltypeUnknown, ::Base.HasShape{1}) at ./array.jl:619
 [3] map at ./array.jl:548 [inlined]
 [4] convert(::Type{Documenter.Utilities.Markdown2.MD}, ::Markdown.MD) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Utilities/Markdown2.jl:214
 [5] Documenter.Documents.Page(::String, ::String, ::String) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Documents.jl:60
 [6] addpage! at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Documents.jl:368 [inlined]
 [7] runner(::Type{Documenter.Builder.SetupBuildDirectory}, ::Documenter.Documents.Document) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Builder.jl:131
 [8] dispatch(::Type{Documenter.Builder.DocumentPipeline}, ::Documenter.Documents.Document) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Utilities/Selectors.jl:167
 [9] (::getfield(Documenter, Symbol("##3#8")))() at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Documenter.jl:319
 [10] cd(::getfield(Documenter, Symbol("##3#8")), ::String) at ./file.jl:96
 [11] #makedocs#1(::Bool, ::Documenter.Writers.HTMLWriter.HTML, ::Nothing, ::Nothing, ::Nothing, ::Nothing, ::Nothing, ::Nothing, ::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::typeof(makedocs)) at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Documenter.jl:318
 [12] makedocs() at /Users/rpolkinghorne/.julia/packages/Documenter/oEuOX/src/Documenter.jl:243
 [13] top-level scope at none:0
$ 

@thisrod
Copy link
Author

thisrod commented Jul 25, 2019

Interestingly, $$f$$ works, but adding the line breaks causes a crash.

@thisrod
Copy link
Author

thisrod commented Jul 31, 2019

Thanks! The ApproxFun manual is compiling again.

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

Successfully merging a pull request may close this issue.

2 participants