| Age | Commit message (Collapse) | Author | Files | Lines | |
|---|---|---|---|---|---|
| 2021-03-16 | Speed-up CI by reorganizing tests (#22247) | Massimiliano Culpo | 14 | -300/+367 | |
| * unit tests: mark slow tests as "maybeslow" This commit also removes the "network" marker and marks every "network" test as "maybeslow". Tests marked as db are maintained, but they're not slow anymore. * GA: require style tests to pass before running unit-tests * GA: make MacOS unit tests fail fast * GA: move all unit tests into the same workflow, run style tests as a prerequisite All the unit tests have been moved into the same workflow so that a single run of the dorny/paths-filter action can be used to ask for coverage based on the files that have been changed in a PR. The basic idea is that for PRs that introduce only changes to packages coverage is not necessary, this resulting in a faster execution of the tests. Also, for package only PRs slow unit tests are skipped. Finally, MacOS and linux unit tests are now conditional on style tests passing meaning that e.g. we won't waste a MacOS worker if we know that the PR has flake8 issues. * Addressed review comments * Skipping slow tests on MacOS for package only recipes * QA: make tests on changes correct before merging | |||||
| 2021-03-16 | py-aiobotocore: New package (#22258) | a-saitoh-fj | 1 | -0/+22 | |
| * py-aiobotocore: New package * py-aiobotocore: Added python dependencies, and Removed unnecessary whitespace | |||||
| 2021-03-16 | bugfix: allow imposed constraints to be overridden in special cases | Todd Gamblin | 1 | -6/+16 | |
| In most cases, we want condition_holds(ID) to imply any imposed constraints associated with the ID. However, the dependency relationship in Spack is special because it's "extra" conditional -- a dependency *condition* may hold, but we have decided that externals will not have dependencies, so we need a way to avoid having imposed constraints appear for nodes that don't exist. This introduces a new rule that says that constraints are imposed *unless* we define `do_not_impose(ID)`. This allows rules like dependencies, which rely on more than just spec conditions, to cancel imposed constraints. We add one special case for this: dependencies of externals. | |||||
| 2021-03-16 | bugfix: do not generate dep conditions when no dependency | Todd Gamblin | 1 | -10/+15 | |
| We only consider test dependencies some of the time. Some packages are *only* test dependencies. Spack's algorithm was previously generating dependency conditions that could hold, *even* if there was no potential dependency type. - [x] change asp.py so that this can't happen -- we now only generate dependency types for possible dependencies. | |||||
| 2021-03-16 | concretizer: unify logic for spec conditionals | Todd Gamblin | 3 | -183/+119 | |
| This builds on #20638 by unifying all the places in the concretizer where things are conditional on specs. Previously, we duplicated a common spec conditional pattern for dependencies, virtual providers, conflicts, and externals. That was introduced in #20423 and refined in #20507, and roughly looked as follows. Given some directives in a package like: ```python depends_on("foo@1.0+bar", when="@2.0+variant") provides("mpi@2:", when="@1.9:") ``` We handled the `@2.0+variant` and `@1.9:` parts by generating generated `dependency_condition()`, `required_dependency_condition()`, and `imposed_dependency_condition()` facts to trigger rules like this: ```prolog dependency_conditions_hold(ID, Parent, Dependency) :- attr(Name, Arg1) : required_dependency_condition(ID, Name, Arg1); attr(Name, Arg1, Arg2) : required_dependency_condition(ID, Name, Arg1, Arg2); attr(Name, Arg1, Arg2, Arg3) : required_dependency_condition(ID, Name, Arg1, Arg2, Arg3); dependency_condition(ID, Parent, Dependency); node(Parent). ``` And we handled `foo@1.0+bar` and `mpi@2:` parts ("imposed constraints") like this: ```prolog attr(Name, Arg1, Arg2) :- dependency_conditions_hold(ID, Package, Dependency), imposed_dependency_condition(ID, Name, Arg1, Arg2). attr(Name, Arg1, Arg2, Arg3) :- dependency_conditions_hold(ID, Package, Dependency), imposed_dependency_condition(ID, Name, Arg1, Arg2, Arg3). ``` These rules were repeated with different input predicates for requirements (e.g., `required_dependency_condition`) and imposed constraints (e.g., `imposed_dependency_condition`) throughout `concretize.lp`. In #20638 it got to be a bit confusing, because we used the same `dependency_condition_holds` predicate to impose constraints on conditional dependencies and virtual providers. So, even though the pattern was repeated, some of the conditional rules were conjoined in a weird way. Instead of repeating this pattern everywhere, we now have *one* set of consolidated rules for conditions: ```prolog condition_holds(ID) :- condition(ID); attr(Name, A1) : condition_requirement(ID, Name, A1); attr(Name, A1, A2) : condition_requirement(ID, Name, A1, A2); attr(Name, A1, A2, A3) : condition_requirement(ID, Name, A1, A2, A3). attr(Name, A1) :- condition_holds(ID), imposed_constraint(ID, Name, A1). attr(Name, A1, A2) :- condition_holds(ID), imposed_constraint(ID, Name, A1, A2). attr(Name, A1, A2, A3) :- condition_holds(ID), imposed_constraint(ID, Name, A1, A2, A3). ``` this allows us to use `condition(ID)` and `condition_holds(ID)` to encapsulate the conditional logic on specs in all the scenarios where we need it. Instead of defining predicates for the requirements and imposed constraints, we generate the condition inputs with generic facts, and define predicates to associate the condition ID with a particular scenario. So, now, the generated facts for a condition look like this: ```prolog condition(121). condition_requirement(121,"node","cairo"). condition_requirement(121,"variant_value","cairo","fc","True"). imposed_constraint(121,"version_satisfies","fontconfig","2.10.91:"). dependency_condition(121,"cairo","fontconfig"). dependency_type(121,"build"). dependency_type(121,"link"). ``` The requirements and imposed constraints are generic, and we associate them with their meaning via the id. Here, `dependency_condition(121, "cairo", "fontconfig")` tells us that condition 121 has to do with the dependency of `cairo` on `fontconfig`, and the conditional dependency rules just become: ```prolog dependency_holds(Package, Dependency, Type) :- dependency_condition(ID, Package, Dependency), dependency_type(ID, Type), condition_holds(ID). ``` Dependencies, virtuals, conflicts, and externals all now use similar patterns, and the logic for generating condition facts is common to all of them on the python side, as well. The more specific routines like `package_dependencies_rules` just call `self.condition(...)` to get an id and generate requirements and imposed constraints, then they generate their extra facts with the returned id, like this: ```python def package_dependencies_rules(self, pkg, tests): """Translate 'depends_on' directives into ASP logic.""" for _, conditions in sorted(pkg.dependencies.items()): for cond, dep in sorted(conditions.items()): condition_id = self.condition(cond, dep.spec, pkg.name) # create a condition and get its id self.gen.fact(fn.dependency_condition( # associate specifics about the dependency w/the id condition_id, pkg.name, dep.spec.name )) # etc. ``` - [x] unify generation and logic for conditions - [x] use unified logic for dependencies - [x] use unified logic for virtuals - [x] use unified logic for conflicts - [x] use unified logic for externals LocalWords: concretizer mpi attr Arg concretize lp cairo fc fontconfig LocalWords: virtuals def pkg cond dep fn refactor github py | |||||
| 2021-03-16 | amdlibflame: Fix build error (#22316) | Tomoyasu Nojiri | 1 | -0/+2 | |
| 2021-03-16 | libflame: Fix build error (#22315) | Tomoyasu Nojiri | 1 | -0/+2 | |
| 2021-03-16 | hwloc: add dependency on ncurses (#22300) | Harmen Stoppels | 1 | -0/+4 | |
| Co-authored-by: Michal Sudwoj <msudwoj@student.ethz.ch> | |||||
| 2021-03-15 | OpenCV: add new version, simplify package (#22292) | Adam J. Stewart | 1 | -7/+4 | |
| 2021-03-15 | Expand relative dev paths in environment files (#22045) | Harmen Stoppels | 7 | -20/+139 | |
| * Rewrite relative dev_spec paths internally to absolute paths in case of relocation of the environment file * Test relative paths for dev_path in environments * Add a --keep-relative flag to spack env create This ensures that relative paths of develop paths are not expanded to absolute paths when initializing the environment in a different location from the spack.yaml init file. | |||||
| 2021-03-15 | Propagate --test= for environments (#22040) | Harmen Stoppels | 8 | -28/+126 | |
| * Propagate --test= for environments * Improve help comment for spack concretize --test flag * Add tests for --test with environments | |||||
| 2021-03-15 | vtk-h: Fix missing trailing-colon in CMake version (#22302) | Chuck Atkins | 1 | -1/+1 | |
| 2021-03-15 | Make py-boto3 installable again with old concretizer (#22298) | Harmen Stoppels | 5 | -1/+10 | |
| 2021-03-15 | dos2unix: gettext needed for linking (#22288) | Adam J. Stewart | 1 | -1/+1 | |
| 2021-03-15 | Fix use of quotes in Python build system (#22279) | Adam J. Stewart | 1 | -1/+1 | |
| 2021-03-15 | py-tqdm: add new version, variants (#22290) | Adam J. Stewart | 1 | -2/+9 | |
| 2021-03-15 | PyTorch: add missing tqdm dependency (#22289) | Adam J. Stewart | 1 | -0/+1 | |
| 2021-03-15 | fortrilinos: Fix trilinos depend (#22295) | Tomoyasu Nojiri | 1 | -1/+1 | |
| 2021-03-15 | votca-*: add v2021 (#22282) | Christoph Junghans | 4 | -4/+8 | |
| 2021-03-15 | py-aioitertools: New package (#22256) | a-saitoh-fj | 1 | -0/+20 | |
| * py-aioitertools: New package * py-aioitertools: Fixed python dependencies, and Fixed style of code * switch to the source build | |||||
| 2021-03-14 | do not validate variants of concrete specs in solver setup (#22272) | Vanessasaurus | 1 | -10/+11 | |
| Currently, regardless of a spec being concrete or not, we validate its variants in `spec_clauses` (part of `SpackSolverSetup`). This PR skips the check if the spec is concrete. The reason we want to do this is so that the solver setup class (really, `spec_clauses`) can be used for cases when we just want the logic statements / facts (is that what they are called?) and we don't need to re-validate an already concrete spec. We can't change existing concrete specs, and we have to be able to handle them *even if they violate constraints in the current spack*. This happens in practice if we are doing the validation for a spec produced by a different spack install. Signed-off-by: vsoch <vsoch@users.noreply.github.com> | |||||
| 2021-03-14 | py-botocore: Update version to install py-s3fs (#22257) | a-saitoh-fj | 1 | -4/+10 | |
| * Update version to install py-s3fs * Update the required version of py-urllib3, and Set the dependent conditions of py-docutils | |||||
| 2021-03-13 | openblas: prevent microarch flags for generic targets (#22270) | Seth R. Johnson | 1 | -2/+4 | |
| As of OpenBLAS 0.3.13, leaving off `TARGET` by default optimizes most code for the host system -- adding flags that cause the resulting library to fail (SIGILL) on older systems. This change should ensure that a "x86_64" target for example will work across deployment systems. https://github.com/xianyi/OpenBLAS/issues/3139 | |||||
| 2021-03-13 | adding spack -c to set one off config arguments (#22251) | Vanessasaurus | 6 | -73/+133 | |
| This pull request will add the ability for a user to add a configuration argument on the fly, on the command line, e.g.,: ```bash $ spack -c config:install_tree:root:/path/to/config.yaml -c packages:all:compiler:[gcc] list --help ``` The above command doesn't do anything (I'm just getting help for list) but you can imagine having another root of packages, and updating it on the fly for a command (something I'd like to do in the near future!) I've moved the logic for config_add that used to be in spack/cmd/config.py into spack/config.py proper, and now both the main.py (where spack commands live) and spack/cmd/config.py use these functions. I only needed spack config add, so I didn't move the others. We can move the others if there are also needed in multiple places. | |||||
| 2021-03-12 | fixed ^python in py-rich, py-dvc (#22264) | Bryan Herman | 2 | -2/+2 | |
| 2021-03-12 | py-h5py: offline installation and deps fix (#22262) | Bryan Herman | 2 | -5/+36 | |
| * added h5py patch file for offline installs to work * h5py v3+ deps consistent with setup.py, add patch | |||||
| 2021-03-12 | ascent: Relax CMake version constraints (#22253) | Chuck Atkins | 2 | -6/+4 | |
| 2021-03-12 | [dd4hep] update env var (#22218) | Valentin Volkl | 1 | -0/+1 | |
| 2021-03-12 | bugfix: ensure spack test list still works (#22203) | Tamara Dahlgren | 2 | -2/+25 | |
| Was getting the following error: ``` $ spack test list ==> Error: issubclass() arg 1 must be a class ``` This PR adds a check in `has_test_method` (in case it is re-used elsewhere such as #22097) and ensures a class is passed to the method from `spack test list`. | |||||
| 2021-03-12 | Rivet: syntax fixes (#22225) | iarspider | 1 | -3/+2 | |
| 2021-03-12 | Version update to 1.9.4 (#22231) | Desmond Orton | 1 | -0/+1 | |
| 2021-03-12 | LBANN software stack: new versions and dependencies (#22234) | Brian Van Essen | 4 | -6/+11 | |
| Updated the versions for DiHydrogen and Aluminum. Added new constraints on versions of Aluminum that are used across the software stack. Cleaned up the dependency on DiHydrogen for LBANN. | |||||
| 2021-03-12 | add hints for dependency resolution (#22238) | Sinan | 1 | -0/+4 | |
| Co-authored-by: sbulut <sbulut@3vgeomatics.com> | |||||
| 2021-03-12 | py-fsspec: Update version to install py-s3fs (#22259) | a-saitoh-fj | 1 | -0/+1 | |
| 2021-03-12 | fixed when python dep in ftfy dep for py benedict (#22261) | Bryan Herman | 1 | -2/+2 | |
| 2021-03-12 | chai: Add 2.3.0 version (#22254) | Tomoyasu Nojiri | 1 | -0/+1 | |
| 2021-03-12 | py-chainer: Add test method for ChainerMN (continued #21848, #21940) (#22189) | a-saitoh-fj | 1 | -0/+49 | |
| * py-chainer: Add test method for ChainerMN (continued #21848, #21940) * py-chainer: Fixed the word in the message * py-chainer: Delete unnecessary imports * py-chainer: Incorporation of the measures pointed out in #21940 was insufficient. | |||||
| 2021-03-12 | mptensor: Add test method (#21712) | kuramoto-fj | 1 | -0/+41 | |
| * mptensor: Add test method * mptensor: Reverted the receipe on merged style * mptensor: Changed """ to # | |||||
| 2021-03-12 | superlu-dist: Add e4s testsuite-inspired smoke test (#22237) | Sergei Shudler | 1 | -0/+50 | |
| * Added a smoke test for superlu-dist recipe * Fixed small issues following a PR review | |||||
| 2021-03-11 | Add internal package options to Trilinos (#22205) | Paul Kuberry | 1 | -1/+34 | |
| Adds several EpetraExt_BUILD_* options as well as an Amesos2_ENABLE_Basker option. Adds `none` as an option to `gotype=`, which should be among the options since 'none' is specifically handled later in the package definition. Adds `stokhos` and `trilinoscouplings` as options in spack which already are available in CMake for Trilinos (e.g. Trilinos_ENABLE_Stokhos:BOOL=) | |||||
| 2021-03-12 | py-pytest-html recipe (#22221) | BerengerBerthoul | 2 | -0/+43 | |
| * py-pytest-html recipe * added missing deps + copyright * Update var/spack/repos/builtin/packages/py-pytest-html/package.py Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com> * Update var/spack/repos/builtin/packages/py-pytest-metadata/package.py Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com> Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com> | |||||
| 2021-03-12 | Add newest versions of py-h5py (#22245) | Thomas Kluyver | 1 | -5/+10 | |
| * Add newest versions of py-h5py * Update dependencies for py-h5py | |||||
| 2021-03-11 | r-rhdf5filters: Fix for aarch64 (#22074) | Tomoyasu Nojiri | 1 | -0/+7 | |
| 2021-03-11 | bind9: add missing dependencies (#22212) | Tomoyasu Nojiri | 1 | -1/+7 | |
| 2021-03-11 | ctffind: update url and version directive (#22039) | Tomoyasu Nojiri | 1 | -3/+3 | |
| 2021-03-11 | elpa: fix build of versions before 2020 with GCC >= 10 (#22182) | Ye Luo | 1 | -4/+11 | |
| 2021-03-11 | py-imageio: new version 2.9.0 (#22229) | Andrew W Elble | 1 | -1/+3 | |
| 2021-03-11 | eospac: add patch to fix selection of compiler flags (#21626) | Gabriel Rockefeller | 2 | -0/+26 | |
| Patch eospac's Makefile.-linux-gnu.hashes to consider only `$(notdir $(F90))` when constructing a key to look up compiler flags in the _F90-CPUINFO_COMP_FLAGS associative array. This patch was accepted into eospac itself after the release of 6.4.2beta, so apply it only to 6.4.2beta and earlier releases. | |||||
| 2021-03-10 | cctools: add v7.2.1 (#22193) | Benjamin Tovar | 1 | -1/+2 | |
| 2021-03-10 | gearshifft: fix patch, add support for mkl and rocm (#22195) | David Pape | 2 | -22/+31 | |
| - Fix faulty patch - Only use GEARSHIFFT_BACKEND_FFTW_PTHREADS if ~openmp - Explicitly disable float16 support - Use correct minimum required Boost version - Add variants for Intel MKL and ROCm rocfft | |||||
 |
index : spack | |
| A flexible package manager that supports multiple versions, configurations, platforms, and compilers. | root |
| summaryrefslogtreecommitdiff |