Skip to content

Specify self for lua internal setup method#14008

Merged
adamjstewart merged 1 commit intospack:developfrom
paulbry:lua-internal-methods
Dec 6, 2019
Merged

Specify self for lua internal setup method#14008
adamjstewart merged 1 commit intospack:developfrom
paulbry:lua-internal-methods

Conversation

@paulbry
Copy link
Copy Markdown
Contributor

@paulbry paulbry commented Dec 6, 2019

I noticed this when deploying software with the 0.13.2 release:

==> Installing lua-luaposix
==> Searching for binary cache of lua-luaposix
==> Warning: No Spack mirrors are currently configured
==> No binary for lua-luaposix found: installing from source
==> Error: NameError: name '_setup_dependent_env_helper' is not defined

/opt/spack/var/spack/repos/builtin/packages/lua/package.py:121, in setup_dependent_build_environment:
        120    def setup_dependent_build_environment(self, env, dependent_spec):
  >>    121        lua_patterns, lua_cpatterns = _setup_dependent_env_helper(
        122            self, env, dependent_spec)
        123
        124        env.set('LUA_PATH', ';'.join(lua_patterns), separator=';')

I've included a fix that worked in my case but it could very well be something I simply overlooked.

I tested this on the recent commit (a288449) using the spack/ubuntu-bionic container.

@adamjstewart adamjstewart merged commit 67e4509 into spack:develop Dec 6, 2019
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