Skip to content

Conversation

malfet
Copy link
Contributor

@malfet malfet commented Jul 9, 2021

It is needed only for ffmpeg integration, which is disabled anyway.

Also update setuptools to workaround:

AttributeError: 'Version' object has no attribute 'major'

Introduced by pytorch/pytorch#59665 and attempted to be mitigated by pytorch/pytorch#61053

It is needed only for ffmpeg integration, which is disabled anyway
To update pkg_resources.packaging to version 20.0+
@malfet malfet merged commit 8541c21 into master Jul 9, 2021
@malfet malfet deleted the malfet/disable-lame branch July 9, 2021 20:17
@github-actions
Copy link

github-actions bot commented Jul 9, 2021

Hey @malfet!

You merged this PR, but no labels were added.

facebook-github-bot pushed a commit that referenced this pull request Jul 12, 2021
Summary:
* Do not install lame when building torchvision

It is needed only for ffmpeg integration, which is disabled anyway

* Also upgrade setuptool

To update pkg_resources.packaging to version 20.0+

Reviewed By: fmassa

Differential Revision: D29659496

fbshipit-source-id: a3bbe76029c68d2cd521b9d360e0825873455448
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants