Skip to content

Various changes to spack create template#945

Closed
adamjstewart wants to merge 2 commits intospack:developfrom
adamjstewart:features/create
Closed

Various changes to spack create template#945
adamjstewart wants to merge 2 commits intospack:developfrom
adamjstewart:features/create

Conversation

@adamjstewart
Copy link
Copy Markdown
Member

The main reason I opened this PR is because:

from spack import *

class ${class_name}(Package):

causes flake8 to be unhappy. I made it two empty lines.

While I was at it, I also changed:

configure('--prefix=%s' % prefix)

to

configure('--prefix={0}'.format(prefix))

While the former only works in Python 2, the latter works in Python 2.6, 2.7, and 3. The sooner we include this in the template, the sooner we can get people used to using it. Then, converting Spack to Python 3 (hopefully in the distant future) will be a little bit easier.

Let me know if you don't want to include the latter.

@hegner
Copy link
Copy Markdown

hegner commented May 12, 2016

@adamjstewart - great! With #926 I started doing a similar thing. Waiting there for make, cmake, etc to be defined early enough so that the from spack import * can go.
For the format I fully agree.

@adamjstewart
Copy link
Copy Markdown
Member Author

Ah sorry, didn't see that. But it doesn't look like we overlap too much aside from that line.

I think the from spack import * line can be kept. We added an exception to that rule, so it doesn't cause problems anymore. Sometimes it feels a little magic, and I would like to know where these methods are coming from, but it definitely makes life simpler for package developers.

@adamjstewart
Copy link
Copy Markdown
Member Author

Closing this PR as I will be including these changes in another, bigger PR.

@adamjstewart adamjstewart deleted the features/create branch May 27, 2016 17:53
olupton pushed a commit to olupton/spack that referenced this pull request Feb 7, 2022
olupton pushed a commit to olupton/spack that referenced this pull request Feb 7, 2022
* Update bbp-packages.yaml

Adding my currentscape module to bbp-packages.

* Update modules.yaml

Adding my currentscape package to the whitelist.

* Create package.py

Added my currentscape package.

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update package.py

Modified the version. Is it better now?

* update bglibpy to 4.4.6 (spack#937)

* Update package.py

Changing tag to match the one from source on gerrit.

* Bglibpy 4.4.10 (spack#938)

* update bglibpy to 4.4.6

* updated to bglpy 4.4.10

v 4.4.10 does not have the pandas, pyrsistent or the bluepy-configfile dependencies

* Updates Libsonata package to include readers improvements (spack#940)

* update glm to 0.9.9.3 (spack#943)

* Bump neurodamus for most recent patches (spack#942)

Bump neurodamus-py 2.0.0 to 2.0.2:
 - Fixing replay to work with multiple populations
 - Ensure data dir when skipping model build
 - Fix skipping synapse creation when weight is 0 (BBPBGLIB-673)
 - Fix deadlock when an exception is thrown from NEURON (BBPBGLIB-678)
 - Logging colors only for tty

Bump neurodamus-core 3.0.0 to 3.0.1:
 - Avoid getting nilSecRef from objects (HPCTM-1381)

* Update py-sonata-network-reduction dependencies: [email protected], [email protected] (spack#930)

* Update bbp-packages.yaml

Updated currentscape version.

* Update package.py

Updated version & tag.

* Brion and Brayns are dependent on GLM (spack#944)

* Steps updates (spack#941)

* gmsh: add version 4.6.0
* omega-h: new version 9.32.5.dev3
* steps: new test requirements

* libsonata-report: Improves initialization performance (spack#945)

* adapt brion test to a new python module name (spack#946)

* Adding nvidia-hpc-sdk based on upstream PR (spack#935)

* New compiler: nvhpc (NVIDIA HPC SDK) (spack#19294)
* Add nvhpc compiler definition: "spack compiler add" will now look
  for instances of the NVIDIA HPC SDK compiler executables
  (nvc, nvc++, nvfortran) in supplied paths
* Add the nvhpc package which installs the nvhpc compiler
* Add testing for nvhpc detection and C++-standard/pic flags

Based on spack#19294

* Add CUDA@11 required for latest NVIDIA-HPC-SDK
* Fix legacy apis : setup_environment to setup_run_environment

* NEURON and CoreNEURON should use legacy units for BBP/HBP deployment (spack#947)

* Update bbp-packages.yaml

Adding my currentscape module to bbp-packages.

* Update modules.yaml

Adding my currentscape package to the whitelist.

* Create package.py

Added my currentscape package.

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update var/spack/repos/builtin/packages/py-currentscape/package.py

Co-authored-by: Matthias Wolf <[email protected]>

* Update package.py

Modified the version. Is it better now?

* Update package.py

Changing tag to match the one from source on gerrit.

* Update bbp-packages.yaml

Updated currentscape version.

* Update package.py

Updated version & tag.

Co-authored-by: Matthias Wolf <[email protected]>
Co-authored-by: anilbey <[email protected]>
Co-authored-by: Sergio <[email protected]>
Co-authored-by: ppodhajski <[email protected]>
Co-authored-by: Fernando Pereira <[email protected]>
Co-authored-by: asanin-epfl <[email protected]>
Co-authored-by: Nadir Román Guerrero <[email protected]>
Co-authored-by: Tristan Carel <[email protected]>
Co-authored-by: Pramod Kumbhar <[email protected]>
Co-authored-by: Jaquier Aurélien Tristan <[email protected]>
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.

2 participants