Skip to content

Commit

Permalink
Made RTD apt install FFMPEG instead of installing a Python binding (M…
Browse files Browse the repository at this point in the history
…animCommunity#1896)

* Try to apt install ffmpeg instead

* remove symlink

* try
  • Loading branch information
Darylgolden authored Aug 12, 2021
1 parent 748de4a commit 092555a
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 15 deletions.
1 change: 1 addition & 0 deletions .readthedocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ build:
image: latest
apt_packages:
- libpango1.0-dev
- ffmpeg
python:
version: 3.8
install:
Expand Down
1 change: 0 additions & 1 deletion docs/rtd-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
imageio-ffmpeg
jupyterlab
14 changes: 0 additions & 14 deletions docs/source/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,20 +20,6 @@
sys.path.insert(0, os.path.abspath("."))


if os.environ.get("READTHEDOCS") == "True":
site_path = get_python_lib()
# we need to add ffmpeg to the path
ffmpeg_path = os.path.join(site_path, "imageio_ffmpeg", "binaries")
# the included binary is named ffmpeg-linux..., create a symlink
[ffmpeg_bin] = [
file for file in os.listdir(ffmpeg_path) if file.startswith("ffmpeg-")
]
os.symlink(
os.path.join(ffmpeg_path, ffmpeg_bin), os.path.join(ffmpeg_path, "ffmpeg")
)
os.environ["PATH"] += os.pathsep + ffmpeg_path


# -- Project information -----------------------------------------------------

project = "Manim"
Expand Down

0 comments on commit 092555a

Please sign in to comment.