Skip to content

Bugfix/stack write reference#12096

Merged
tgamblin merged 2 commits intodevelopfrom
bugfix/stack-write-reference
Aug 6, 2019
Merged

Bugfix/stack write reference#12096
tgamblin merged 2 commits intodevelopfrom
bugfix/stack-write-reference

Conversation

@becker33
Copy link
Copy Markdown
Member

Fixes #12095

Spack environment definitions were improperly considering definition references to be specs at Env.write() time, and were generating invalid character errors when attempting to create specs.

Now they don't.

@tgamblin tgamblin requested review from tgamblin and removed request for tgamblin August 4, 2019 00:50
@tgamblin tgamblin merged commit 3b4d6dd into develop Aug 6, 2019
likask added a commit to likask/spack that referenced this pull request Aug 7, 2019
…upsream_develop

* commit 'f7026a058b63f5a3109691e2c3871ee77c08f756': (1881 commits)
  Version 19.8.1 of PLASMA  (spack#12299)
  new package: py-exodus (spack#12291)
  ncurses: fix pic and opt flags (spack#12272)
  pumi: new version 2.2.1 (spack#12282)
  tests: explain and test dependency flattening routines (spack#11993)
  graphviz package: add MacOS fixes and quartz support (spack#11128)
  Overhaul numpy package (spack#12170)
  mirrors: mirror config should use spack variable expansions (spack#9027)
  stacks: fix reference handling in env.write() (spack#12096)
  fltk: fix about variable types (spack#12292)
  Avoid sending empty reports to codecov (spack#12293)
  Packages/musl (spack#12288)
  c-blosc package: Add -std=gnu99 flag for gcc (spack#11959)
  Move new packages from tutorial to builtin (spack#12289)
  Balay/amrex 19.08 (spack#12287)
  openPMD-api: pre-load depend libs (spack#12279)
  Add version 19.8.0 of PLASMA (spack#12275)
  Add version 2.5.1 of MAGMA released today (spack#12274)
  ginkgo: add maintainers (spack#12273)
  new package: py-backports-tempfile (spack#12261)
  ...

# Conflicts:
#	.travis.yml
#	var/spack/repos/builtin/packages/moab/package.py
#	var/spack/repos/builtin/packages/mofem-cephas/package.py
#	var/spack/repos/builtin/packages/mofem-fracture-module/package.py
#	var/spack/repos/builtin/packages/mofem-users-modules/package.py
#	var/spack/repos/builtin/packages/petsc/package.py
dev-zero pushed a commit to dev-zero/spack that referenced this pull request Aug 13, 2019
* stacks: Fix env.write to properly write references
* stacks: regression test for 12095
@tgamblin tgamblin deleted the bugfix/stack-write-reference branch September 3, 2019 05:30
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.

Invalid character error when referencing definitions in an environment's spec

2 participants