Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update version, with new format #227

Merged
merged 5 commits into from
Oct 3, 2024

Conversation

itcarroll
Copy link
Contributor

@itcarroll itcarroll commented Aug 29, 2024

HDF5Group has introduced a "release number" after a hyphen into their distributions. I don't know what distinguishes this from a new patch number. I think we can accommodate by updating the conda-forge build number in case of a new release number: if HDF5 releases a 1.14.4-4, we can keep the version as 1.14.4 and update the build number.

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@ajelenak
Copy link

There will not be 1.14.4-4 or any -n. This is not a new version format but happened as the CI jobs for making release packages had some problems.

The next release will be 1.14.5. Hope this clarifies the situation.

@itcarroll
Copy link
Contributor Author

Thanks, @ajelenak. Would you be able to add a tarball with just "1.14.4" to https://support.hdfgroup.org/ftp/HDF5/releases/hdf5-1.14/hdf5-1.14.4/src/? Then there would be no need to change the conda recipe (and then change it back for 1.14.5).

@itcarroll
Copy link
Contributor Author

I'm probably not going to be able to resolve the build errors on the subset of platforms where it fails. If anybody wants to close this PR and open a new one for 1.14.4, please feel free.

@ajelenak
Copy link

Would you be able to add a tarball with just "1.14.4" to https://support.hdfgroup.org/ftp/HDF5/releases/hdf5-1.14/hdf5-1.14.4/src/? Then there would be no need to change the conda recipe (and then change it back for 1.14.5).

I suggest to wait for 1.14.5, which is due in a month or so.

@zklaus
Copy link
Contributor

zklaus commented Aug 30, 2024

Waiting is fine for me. Regarding the build artifacts (I came here via the discussion on element), @itcarroll, you have to rerender after changing the config.

@conda-forge-admin, please rerender

@zklaus
Copy link
Contributor

zklaus commented Aug 30, 2024

We can wait, but if you want to work on this, @itcarroll, here are some things that I found so far:

  • The patches need updating. They seem to be mostly fine, but there is a bit of an offset and in the case of osx-arm64, particularly for 0001-avoid-test-execution-when-cross-compiling.patch.native, they are just to big and the patching fails. Perhaps this is simply due to new tests. I would just try to re-create the patch manually.
  • For Windows, everything seems to work until we try to copy h5repart-shared.exe to h5repart.exe. However, there is already h5repart.exe, so maybe we just don't need to copy? In other words, I think we can simply remove everything beyond line 72 from our bld.bat.

@ajelenak, can you confirm that the windows cmake build was changed to not add -shared to every exe?

If this is not confirmed, we should double check that we indeed still avoid static linking...

The only linux failure so far is a fluke, that will be resolved by restarting.

@ajelenak
Copy link

@ajelenak, can you confirm that the windows cmake build was changed to not add -shared to every exe?

@zklaus Yes, see: https://github.com/HDFGroup/hdf5/blob/hdf5_1.14.4.3/release_docs/RELEASE.txt#L162-L168.

@zklaus zklaus mentioned this pull request Sep 4, 2024
5 tasks
@github-actions github-actions bot merged commit f44d764 into conda-forge:main Oct 3, 2024
12 of 19 checks passed
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.

3 participants