Bugfix/netcdf cxx4 macos#4
Merged
climbfuji merged 16 commits intoclimbfuji:bugfix/netcdf_cxx4_macosfrom Mar 5, 2022
Merged
Conversation
This was a rudiment from the times when the package was fetched with git, which broke timestamp order of the automatically generated Autoconf files.
This makes it consistent with other packages from the NetCDF constellation: always build the static libraries and additionally build the shared ones when '+shared'.
This makes it consistent with other packages from the NetCDF constellation: build the shared libraries with the PIC flag and the static ones without it (the default for Autotools) when '~pic', and build the static libraries with PIC when '+pic' (to make them injectable into other shared libraries).
Owner
|
@skosukhin Thanks, I am testing this today, will let you know how it goes. |
Owner
|
@skosukhin Unfortunately using your version gives a ton of errors on macOS: These look a lot like what the patch in both our versions is supposed to fix, and it does say in the logs that the patch is applied. I checked however in the same directory the file is called |
Author
|
@climbfuji I'm sorry. I forgot that the patch implies running |
Owner
|
Yes, @skosukhin this works on macOS now. I am going to test it on one of our finicky HPC systems later today as well, because there are quite a few changes in your PR. |
climbfuji
pushed a commit
that referenced
this pull request
Aug 15, 2022
* Created package and added description * Add py-markdown-include * Created package * Finished creating package * Added py-md-environ * Added build dependencies * Added other deps * Add python-markdown-math (#4) * Created package and started to add info * Removed unneeded global/install options * Figured out version spec for markdown-math * Removed type=build from unnecessary dependencies * Removed unneeded install/global options, added version spec to dependency * Added wscullin as interim maintainer for packages * Fixed style issues * Took care of trailing whitespace * Removed comment line before imports * Changed file charset to match other packages * Update var/spack/repos/builtin/packages/py-ford/package.py Co-authored-by: Adam J. Stewart <[email protected]> * Update var/spack/repos/builtin/packages/py-ford/package.py Co-authored-by: Adam J. Stewart <[email protected]> * Removed test dependency after review feedback * Added new 6.1.12 version to py-ford Co-authored-by: Seth R. Johnson <[email protected]> Co-authored-by: Adam J. Stewart <[email protected]>
climbfuji
pushed a commit
that referenced
this pull request
Apr 28, 2023
1. support version 3.1.3, which now depends on sundials@6 2. support version 3.1.2:, which broke the two patch files and therefore the two patch files have been replaced by more flexible filter_file() commands inside a patch() function. 3. rename the variant for python extension from using the package name "+pyuqtk" to the more standard "+python" 4. add maintainers @omsai and the upstream developer @bjdebus who offered to help with the spack packaging. 5. swig should only be a build-time dependency. swig is only necessary until @:3.1.0 6. confirmed python dependencies are correct by inspecting imports, subset python dependencies type to build, run, and confirmed all 31 build-time tests pass including the 9 python tests: ```console $ spack env create uqtk-dev $ spack add [email protected] $ spack install --test root && cat $(spack location -i uqtk)/.spack/install-time-test-log.txt ==> Testing package uqtk-3.1.3-nok6fut ==> [2023-04-19-14:56:25.005361] Running build-time tests ==> [2023-04-19-14:56:25.005536] RUN-TESTS: build-time tests [check] ==> [2023-04-19-14:56:25.009543] '/home/omsai/src/spack/opt/spack/linux-pureos10-skylake/gcc-10.2.1/gmake-4.4.1-b6g4apmfvxz3bn4eabh37dehcrg65fj7/bin/make' '-j4' '-n' 'test' ==> [2023-04-19-14:56:25.014903] '/home/omsai/src/spack/opt/spack/linux-pureos10-skylake/gcc-10.2.1/gmake-4.4.1-b6g4apmfvxz3bn4eabh37dehcrg65fj7/bin/make' '-j4' 'test' Running tests... /home/omsai/src/spack/opt/spack/linux-pureos10-skylake/gcc-10.2.1/cmake-3.26.3-zjmsfz23j5l4ytniz26uzvxonlu5qebr/bin/ctest --force-new-ctest-process Test project /tmp/omsai/spack-stage/spack-stage-uqtk-3.1.3-nok6fut47h42cnaau7wkoohgqy5f2qqa/spack-build-nok6fut Start 1: ArrayReadAndWrite Start 2: ArrayDelColumn Start 3: Array1DMiscTest Start 4: Array2DMiscTest 1/31 Test #1: ArrayReadAndWrite ................ Passed 0.01 sec Start 5: ArraySortTest 2/31 Test #2: ArrayDelColumn ................... Passed 0.01 sec Start 6: MultiIndexTest 3/31 Test #3: Array1DMiscTest .................. Passed 0.01 sec Start 7: CorrTest 4/31 Test #4: Array2DMiscTest .................. Passed 0.01 sec Start 8: QuadLUTest 5/31 Test #5: ArraySortTest .................... Passed 0.02 sec Start 9: MCMC2dTest 6/31 Test #6: MultiIndexTest ................... Passed 0.01 sec Start 10: MCMCRandomTest 7/31 Test #8: QuadLUTest ....................... Passed 0.02 sec Start 11: MCMCNestedTest 8/31 Test spack#10: MCMCRandomTest ................... Passed 0.02 sec Start 12: Deriv1dTest 9/31 Test spack#12: Deriv1dTest ...................... Passed 0.01 sec Start 13: SecondDeriv1dTest 10/31 Test spack#13: SecondDeriv1dTest ................ Passed 0.01 sec Start 14: GradHessianTest 11/31 Test spack#11: MCMCNestedTest ................... Passed 0.03 sec Start 15: GradientPCETest 12/31 Test spack#14: GradHessianTest .................. Passed 0.01 sec Start 16: PCE1dTest 13/31 Test spack#15: GradientPCETest .................. Passed 0.01 sec Start 17: PCEImplTest 14/31 Test spack#16: PCE1dTest ........................ Passed 0.01 sec Start 18: PCELogTest 15/31 Test spack#18: PCELogTest ....................... Passed 0.01 sec Start 19: Hessian2dTest 16/31 Test spack#19: Hessian2dTest .................... Passed 0.01 sec Start 20: BCS1dTest 17/31 Test spack#20: BCS1dTest ........................ Passed 0.01 sec Start 21: BCS2dTest 18/31 Test spack#21: BCS2dTest ........................ Passed 0.01 sec Start 22: LowRankRegrTest 19/31 Test spack#22: LowRankRegrTest .................. Passed 0.01 sec Start 23: PyModTest 20/31 Test spack#17: PCEImplTest ...................... Passed 0.07 sec Start 24: PyArrayTest 21/31 Test spack#23: PyModTest ........................ Passed 0.08 sec Start 25: PyArrayTest2 22/31 Test spack#25: PyArrayTest2 ..................... Passed 0.30 sec Start 26: PyQuadTest 23/31 Test spack#24: PyArrayTest ...................... Passed 1.44 sec Start 27: PyBCSTest1D 24/31 Test spack#26: PyQuadTest ....................... Passed 1.68 sec Start 28: PyBCSTest2D 25/31 Test spack#27: PyBCSTest1D ...................... Passed 1.66 sec Start 29: PyBADPTest 26/31 Test #7: CorrTest ......................... Passed 3.43 sec Start 30: PyRegressionTest 27/31 Test spack#28: PyBCSTest2D ...................... Passed 1.50 sec Start 31: PyGalerkinTest 28/31 Test spack#9: MCMC2dTest ....................... Passed 3.90 sec 29/31 Test spack#29: PyBADPTest ....................... Passed 1.66 sec 30/31 Test spack#30: PyRegressionTest ................. Passed 1.72 sec 31/31 Test spack#31: PyGalerkinTest ................... Passed 1.63 sec 100% tests passed, 0 tests failed out of 31 Total Test time (real) = 5.35 sec ==> [2023-04-19-14:56:30.382797] '/home/omsai/src/spack/opt/spack/linux-pureos10-skylake/gcc-10.2.1/gmake-4.4.1-b6g4apmfvxz3bn4eabh37dehcrg65fj7/bin/make' '-j4' '-n' 'check' ==> [2023-04-19-14:56:30.385983] Target 'check' not found in Makefile ```
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@climbfuji could you, please, check whether this works for you? If it does, please, merge it into your branch (with
--ff-only) so that spack#29246 got updated automatically and we merged it there. If you don't like this approach for whatever reason, I can create a separate PR upstream.