[BUG] Semicolon in juliaconsole environment no longer suppresses output

[mossr]

When using the juliaconsole environment from pythontex, semicolons no longer suppress outputs in Weave versions 0.10+ (v0.9.4 is the last working version).

Any pointers to help track this down, I'd be happy to help get this fixed.

[mykelk]

I'm able to reproduce this issue and can confirm that v0.9.4 works.

[torfjelde]

Seems like this line (which was introduced after v0.10) is the issue:

Weave.jl/src/run.jl

Line 227 in 174585e

obj = include_string(mod, code, path) # TODO: fix line number

include_string doesn't respect ;, e.g.

julia> include_string(Main, "1 + 2;")
3

[aviatesk]

@torfjelde thanks for tracking the problem. I introduced include_string in this commit just because it's easier way to respect filename and line numbers than to use Core.eval solution (, which needs bit of expression manipulation).

I feel a special casing that adds ; nothing when the input line ends with semicolon would be enough for handling this bug.

[torfjelde]

How about

Weave.jl/src/run.jl

Lines 227 to 228 in 174585e

	obj = include_string(mod, code, path) # TODO: fix line number
	!isnothing(obj) && display(obj)

becomes

obj = include_string(mod, code, path) # TODO: fix line number
!isnothing(obj) && !REPL.ends_with_semicolon(code) && display(obj)

?

[aviatesk]

ah sounds nice. wanna open a PR ?

[torfjelde]

Sure thing 👍

[aviatesk]

will be fixed in v0.10.4