Skip to content

Conversation

@eugeneswalker
Copy link
Contributor

@eugeneswalker eugeneswalker commented Jan 6, 2026

Adding new TAU version: 2.35.1 @sameershende @jordialcaraz
And removing maintainer who no longer works on the project!

@sameershende
Copy link
Contributor

I approve.

@eugeneswalker
Copy link
Contributor Author

Despite having a correct checksum, I see it is failing to verify the checksum in GHA:

Run . spack-core/share/spack/setup-env.sh
==> Error: Package tau does not have a valid URL for version 2.35.1
Error: Process completed with exit code 3.

@jordialcaraz
Copy link
Contributor

Was it updated?

sha256sum tau-2.35.1.tar.gz
6eadcf9e2a267d75d7dcae07cf24ab5975576de7ebd6eacb43e65a375eb8468e  tau-2.35.1.tar.gz

@eugeneswalker
Copy link
Contributor Author

Was it updated?

sha256sum tau-2.35.1.tar.gz
6eadcf9e2a267d75d7dcae07cf24ab5975576de7ebd6eacb43e65a375eb8468e  tau-2.35.1.tar.gz

Yes, updated! Thanks!

@spackbot-triage spackbot-triage bot requested a review from khuck January 7, 2026 17:06
@eugeneswalker
Copy link
Contributor Author

Let us update this PR also to include new configuration for +julia variant.

@nchaimov
Copy link

nchaimov commented Jan 7, 2026

The Julia-related changes look correct to me

@jordialcaraz
Copy link
Contributor

@spackbot fix style!

@spackbot-app
Copy link

spackbot-app bot commented Jan 7, 2026

Sorry jordialcaraz, I cannot do that for you. Only eugeneswalker and users with write can make this request!

@eugeneswalker
Copy link
Contributor Author

@spackbot fix style

@spackbot-app
Copy link

spackbot-app bot commented Jan 7, 2026

Let me see if I can fix that for you!

@spackbot-app
Copy link

spackbot-app bot commented Jan 7, 2026

I was able to run spack style --fix for you!

spack style --fix
repos/spack_repo/builtin/packages/asio/package.py
repos/spack_repo/builtin/packages/boost/package.py
repos/spack_repo/builtin/packages/capstone/package.py
repos/spack_repo/builtin/packages/cmake/package.py
repos/spack_repo/builtin/packages/cp2k/package.py
repos/spack_repo/builtin/packages/curl/package.py
repos/spack_repo/builtin/packages/dd4hep/package.py
repos/spack_repo/builtin/packages/edm4hep/package.py
repos/spack_repo/builtin/packages/esmf/package.py
repos/spack_repo/builtin/packages/fenics_ufcx/package.py
repos/spack_repo/builtin/packages/fzf/package.py
repos/spack_repo/builtin/packages/hdf_eos2/package.py
repos/spack_repo/builtin/packages/highfive/package.py
repos/spack_repo/builtin/packages/hpcx_mpi/package.py
repos/spack_repo/builtin/packages/hugo/package.py
repos/spack_repo/builtin/packages/libceed/package.py
repos/spack_repo/builtin/packages/libgeopm/package.py
repos/spack_repo/builtin/packages/libgeopmd/package.py
repos/spack_repo/builtin/packages/libvips/package.py
repos/spack_repo/builtin/packages/mapl/package.py
repos/spack_repo/builtin/packages/mercurial/package.py
repos/spack_repo/builtin/packages/mfem/package.py
repos/spack_repo/builtin/packages/mgard/package.py
repos/spack_repo/builtin/packages/neko/package.py
repos/spack_repo/builtin/packages/node_js/package.py
repos/spack_repo/builtin/packages/petsc/package.py
repos/spack_repo/builtin/packages/podio/package.py
repos/spack_repo/builtin/packages/py_amrex/package.py
repos/spack_repo/builtin/packages/py_dask/package.py
repos/spack_repo/builtin/packages/py_distributed/package.py
repos/spack_repo/builtin/packages/py_geopmdpy/package.py
repos/spack_repo/builtin/packages/py_geopmpy/package.py
repos/spack_repo/builtin/packages/py_mdanalysis/package.py
repos/spack_repo/builtin/packages/py_mdanalysistests/package.py
repos/spack_repo/builtin/packages/py_onnxruntime/package.py
repos/spack_repo/builtin/packages/py_pandas_stubs/package.py
repos/spack_repo/builtin/packages/py_petsc4py/package.py
repos/spack_repo/builtin/packages/py_rasterio/package.py
repos/spack_repo/builtin/packages/py_z3_solver/package.py
repos/spack_repo/builtin/packages/scorep/package.py
repos/spack_repo/builtin/packages/tau/package.py
repos/spack_repo/builtin/packages/vasp/package.py
repos/spack_repo/builtin/packages/verible/package.py
repos/spack_repo/builtin/packages/verilator/package.py
�[1;34m==> �[0mrunning flake8
repos/spack_repo/builtin/packages/tau/package.py:223: [E501] line too long (107 > 99 characters)
�[1;34m==> �[0mrunning isort
�[1;34m==> �[0mrunning black
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with style fixes.

@eugeneswalker eugeneswalker removed the request for review from khuck January 7, 2026 18:02
Copy link
Contributor

@kwryankrattiger kwryankrattiger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maintainers approved

@eugeneswalker eugeneswalker enabled auto-merge (squash) January 7, 2026 18:06
@eugeneswalker eugeneswalker merged commit 0942632 into spack:develop Jan 7, 2026
17 checks passed
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.

5 participants