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

Does one need to check that the ABI changed from X to Y. #3368

Open
hmaarrfk opened this issue Sep 3, 2022 · 4 comments
Open

Does one need to check that the ABI changed from X to Y. #3368

hmaarrfk opened this issue Sep 3, 2022 · 4 comments
Labels

Comments

@hmaarrfk
Copy link
Contributor

hmaarrfk commented Sep 3, 2022

Comment:

I find the the question:

- [ ]  I checked that the ABI changed from X to Y.

doesn't really explain to the user what should happen if the ABI didn't change.

The challenge is two fold:

  1. The bot is kinda an "easy solution". It is also the "safe" solution.
  2. Patching requires more manual (error prone) work that simply click merge on the bot rebuild.

So really, I would like to propose some clarification on what should happen if the pin doesn't change. I find "rebuilding" to be a good solution, even if the ABI doesn't change.

@hmaarrfk
Copy link
Contributor Author

From #3269 it seems that isuruf recommends:

  1. Not merging the migrator.
  2. Issuing a repo-data patch expanding the pinnings.

@hmaarrfk
Copy link
Contributor Author

For example:

        if record.get('timestamp', 0) < 1662860143000:
            if any(dep.startswith("libxml2 >=2.9") for dep in deps):
                _pin_looser(fn, record, "libxml2", upper_bound="2.11.0")

which was done as part of conda-forge/conda-forge-repodata-patches-feedstock#316

@hmaarrfk
Copy link
Contributor Author

hmaarrfk commented Dec 9, 2023

For example to release the pin from 1.14.2 to allow compatibility with 1.14.3

  1. Make a repodata patch that includes variants of the build too
# While not guaranteed
# hdf5 1.14.2 and 1.14.3 have the same ABI
# https://gamma.hdfgroup.org/ftp/pub/outgoing/hdf5/prelim_reports/hdf5-1.14.2-vs-hdf5-1.14.3-1-interface_compatibility_report.html
if:
  has_depends: hdf5 >=1.14.2,<1.14.3.0a0?( *)
  timestamp_lt: 1702066879000

then:
  - loosen_depends:
      name: hdf5
      upper_bound: 1.14.4
  1. Update the pin and any current migrations
    Update pin for hdf5 #5243

@hmaarrfk
Copy link
Contributor Author

hmaarrfk commented Dec 9, 2023

@jakirkham i feel like you had a bullet point list but i lost it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant