sqlite dbhash sqldiff sqlite-analyzer 3.39.0

[owine]

Created with brew bump-formula-pr.

[danielnachun]

This is failing only because of python@3.7. Is it time to deprecate it? It only has two dependents both of which are deprecated.

[cho-m]

This is failing only because of python@3.7. Is it time to deprecate it? It only has two dependents both of which are deprecated.

From EOL standpoint, python@3.7 is supported until June 2023.

However, I think the problem here is that importlib-metadata has a setup_requires dependency on setuptools_scm, but setuptools_scm>=7 has a runtime dependency on importlib-metadata. It picks up the vendor copy within setuptools and fails.

Upstream has specifically mentioned they consider Homebrew's way of handling package installation unsupported (pypa/setuptools-scm#722 (comment)).

We luckily got a fix for Python 3.8 and later as importlib-metadata is only needed on Python 3.7, but it is possible that we will hit more of these issues in future. For example, we see a build loop in jupyterlab #104097.

---

If we want to keep python@3.7, then one option is to locally inreplace importlib-metadata/setup.cfg to restrict setuptools_scm[toml] requirement to < 7.

If we want to deprecate, we could use :unsupported reason as that is pretty much what upstream told us.

[carlocab]

I suggest doing inreplace. Possibly also consider deprecating it, but I think that's just going to end up teaching users not to take deprecation warnings seriously.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

[owine]

This is blocked by python@3.7

[carlocab]

This is blocked by python@3.7

Actually, it's not. brew install python@3.7 fails with the same error even without this PR. (Which makes sense, because the failure looks to be happening inside pure Python code rather than in anything sqlite-related.)

[BrewTestBot]

@carlocab has triggered a merge.