Skip to content

Conversation

@digital-carver
Copy link
Contributor

On Julia nightly, trying to run anything in pager mode gives an error:

julia> VERSION
v"1.13.0-DEV.690"

pager> x = 1
ERROR: UndefVarError: `eval_with_backend` not defined in `REPL`
Suggestion: check for spelling errors or missing imports.
Stacktrace:
 [1] getproperty
   @ ./Base_compiler.jl:50 [inlined]
 [2] _tp_mode_do_cmd(repl::REPL.LineEditREPL, input::String)
   @ TerminalPager ~/foss/TerminalPager.jl/src/repl.jl:245

This is due to JuliaLang/julia#58414 that removed the REPL.eval_with_backend function. The method for evaluating ASTs is now called eval_on_backend.

This change is also being backported to Julia 1.12 and will appear with v1.12-beta4: JuliaLang/julia@c585f7e (#58369), so it seems like this error will start happening on 1.12 also.

(The original PR 58414 is tagged as backport 1.11 also, but there don't seem to be any commits that do that backport. The version number checks will get a bit messier if that does happen in the future.)

This PR uses the correct method name according to the Julia version. With this change:

julia> VERSION
v"1.13.0-DEV.690"

julia> using TerminalPager

pager> x = 1
1

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 24.94%. Comparing base (9e61638) to head (7375372).
Report is 24 commits behind head on main.

Files with missing lines Patch % Lines
src/repl.jl 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   24.67%   24.94%   +0.27%     
==========================================
  Files          15       15              
  Lines         912      914       +2     
==========================================
+ Hits          225      228       +3     
+ Misses        687      686       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@digital-carver
Copy link
Contributor Author

(A bit funny that my PR yesterday removed a @static if VERSION check, and now this one adds a new @static if VERSION check to the same file 😄 )

@ronisbr
Copy link
Owner

ronisbr commented Jun 5, 2025

Thanks!

(A bit funny that my PR yesterday removed a @static if VERSION check, and now this one adds a new @static if VERSION check to the same file 😄 )

This happens a lot 😄

@ronisbr ronisbr merged commit de64136 into ronisbr:main Jun 5, 2025
10 checks passed
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

Successfully merging this pull request may close these issues.

3 participants