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

ocrmypdf: bump to 13.7.0 & migrate to [email protected] #107080

Closed
wants to merge 2 commits into from

Conversation

unitof
Copy link
Contributor

@unitof unitof commented Aug 1, 2022

Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added bump-formula-pr PR was created using `brew bump-formula-pr` python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Aug 1, 2022
@unitof unitof marked this pull request as draft August 2, 2022 00:26
@unitof
Copy link
Contributor Author

unitof commented Aug 2, 2022

Seems to be same build error as #106648 had

@unitof
Copy link
Contributor Author

unitof commented Aug 2, 2022

From the build logs:

  Successfully installed cffi-1.15.1 pycparser-2.21 semantic-version-2.10.0 setuptools-63.3.0 setuptools-rust-1.4.1 typing-extensions-4.3.0 wheel-0.37.1
  WARNING: There was an error checking the latest version of pip.
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Running command Getting requirements to build wheel

          =============================DEBUG ASSISTANCE==========================
          If you are seeing an error here please try the following to
          successfully install cryptography:

          Upgrade to the latest pip and try again. This will fix errors for most
          users. See: https://pip.pypa.io/en/stable/installing/#upgrading-pip
          =============================DEBUG ASSISTANCE==========================

  Traceback (most recent call last):
    File "/opt/homebrew/Cellar/ocrmypdf/13.7.0/libexec/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
      main()
    File "/opt/homebrew/Cellar/ocrmypdf/13.7.0/libexec/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File "/opt/homebrew/Cellar/ocrmypdf/13.7.0/libexec/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 130, in get_requires_for_build_wheel
      return hook(config_settings)
    File "/opt/homebrew/Cellar/[email protected]/3.9.13_1/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/build_meta.py", line 177, in get_requires_for_build_wheel
      return self._get_build_requires(
    File "/opt/homebrew/Cellar/[email protected]/3.9.13_1/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/build_meta.py", line 159, in _get_build_requires
      self.run_setup()
    File "/opt/homebrew/Cellar/[email protected]/3.9.13_1/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/build_meta.py", line 174, in run_setup
      exec(compile(code, __file__, 'exec'), locals())
    File "setup.py", line 14, in <module>
      from setuptools_rust import RustExtension
    File "/private/tmp/pip-build-env-0d_6ubew/overlay/lib/python3.9/site-packages/setuptools_rust/__init__.py", line 1, in <module>
      from .build import build_rust
    File "/private/tmp/pip-build-env-0d_6ubew/overlay/lib/python3.9/site-packages/setuptools_rust/build.py", line 20, in <module>
      from setuptools.command.build import build as CommandBuild  # type: ignore[import]
  ModuleNotFoundError: No module named 'setuptools.command.build'
  error: subprocess-exited-with-error

Any way to update the pip used to build? Happy for help from anyone else! Out of my depth here in python packageland.

@carlocab
Copy link
Member

carlocab commented Aug 2, 2022

setuptools in [email protected] needs updating.

@cho-m
Copy link
Member

cho-m commented Aug 2, 2022

Alternatively, can you try switching to [email protected]? I think the dependency tree is ready now.

@cho-m cho-m added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Aug 2, 2022
@unitof unitof marked this pull request as ready for review August 2, 2022 01:27
@unitof
Copy link
Contributor Author

unitof commented Aug 2, 2022

[email protected] worked locally! (on M1 Mac)

@unitof unitof changed the title ocrmypdf 13.7.0 ocrmypdf: bump to 13.7.0 & migrate to [email protected] Aug 2, 2022
Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Thanks!

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

This was referenced Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. python Python use is a significant feature of the PR or issue python-3.10-migration rust Rust use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants