Skip to content

Conversation

@SereneRuby12
Copy link
Contributor

@SereneRuby12 SereneRuby12 commented Jan 15, 2026

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

I have been having issues with LUA_CPATH causing trouble, because it was being set to a wrong value, so I investigated why it even was added to the lua manifest here:

It was first added in #2931 and I can't find a reason why, the directories for it don't make any sense too, since LUA_PATH and LUA_CPATH (which replace package.path and package.cpath default values) must include the current directory (.\?.lua and .\?.dll) to be able to load modules via require "my_module", which is the most common way to include files.

We can see those env vars didn't make sense because it caused issues in #3829, but the PR for it wasn't a proper fix, it just renamed LUA_PATH to a random environment path that isn't used anywhere (in the stack overflow page mentioned, the person that renamed it, was because they wanted to have an env var to the lua executable, not because it was used anywhere by any program), it kinda solved the issue though, because LUA_PATH isn't being set to wrong values anymore by scoop.

The proper solution is to just remove the LUA_PATH and LUA_CPATH so lua can use its default paths which work normally.

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

lua

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@Lutra-Fs Lutra-Fs merged commit c20e0f0 into ScoopInstaller:master Jan 15, 2026
4 checks passed
@Lutra-Fs
Copy link
Member

Thanks for your contribution

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants