Skip to content

Issue 37: Explicitly export term to reproduce bug#39

Closed
Renegatto wants to merge 1 commit into
purescript-lua:yura/issue-37-reprofrom
Renegatto:patch-1
Closed

Issue 37: Explicitly export term to reproduce bug#39
Renegatto wants to merge 1 commit into
purescript-lua:yura/issue-37-reprofrom
Renegatto:patch-1

Conversation

@Renegatto

@Renegatto Renegatto commented Oct 16, 2025

Copy link
Copy Markdown

No description provided.

@Renegatto
Explicitly export term to reproduce the bug
24a5124 
Signed-off-by: Renegatto <46404781+Renegatto@users.noreply.github.com>
@Unisay Unisay self-assigned this Nov 3, 2025
@Unisay Unisay mentioned this pull request Jun 12, 2026
Merged
Unisay added a commit that referenced this pull request Jun 12, 2026
@Unisay
test: reproduce issue #37 (undefined variable in compiled code)
47b3316 
Golden test from PR #38 with the explicit export list from PR #39:
with 'module ... (baz) where' the non-exported bindings get inlined
and the generated Lua references an undefined variable Bind11.
The luacheck golden suite fails with W113 until the bug is fixed.
@Unisay Unisay mentioned this pull request Jun 12, 2026
Closed
@Unisay

Unisay commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

You were right, the explicit export list is exactly what triggers the bug. #43 includes your change on top of #38's test and fixes the underlying scoping defect in the inliner. Closing in favor of that PR, and thanks again.

@Unisay Unisay closed this Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Unisay Unisay

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Renegatto @Unisay