Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

trying prev version #4615

Merged
merged 4 commits into from
Jun 18, 2022
Merged

trying prev version #4615

merged 4 commits into from
Jun 18, 2022

Conversation

Golovneva
Copy link
Contributor

@Golovneva Golovneva commented Jun 18, 2022

Patch description
Reverted setuptools to the version that didn't fail
Reverted removal for the tensorflow and protobuf dependencies that didn't help to resolve the failure, but somehow happened to clear tests

In v62.4.0 version of setuptools they added a new command "setuptools.command.build" to match distutils.command.build . For some reason, our pipeline fails to find corresponding scripts. With this commit, including temporary workaround that reverts setuptools to its previous state. Ideally we would need to figure why circleCI can't see updates, although logs indicate that newest version was installed.

Failures:
Screen Shot 2022-06-17 at 11 08 16 PM

Corresponding version:
Screen Shot 2022-06-17 at 11 08 54 PM

@Golovneva Golovneva requested a review from klshuster June 18, 2022 03:15
@Golovneva Golovneva marked this pull request as ready for review June 18, 2022 03:19
Copy link
Contributor

@stephenroller stephenroller left a comment

Choose a reason for hiding this comment

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

tyvm

@stephenroller stephenroller merged commit a84d28c into main Jun 18, 2022
@stephenroller stephenroller deleted the olggol/fix-build branch June 18, 2022 13:45
@Golovneva Golovneva restored the olggol/fix-build branch June 27, 2022 12:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants