Skip to content

Commit

Permalink
Changed mkvmerge.exe to mkvmerge to support Non-Windows Operating Sys…
Browse files Browse the repository at this point in the history
…tems
  • Loading branch information
Xonshiz committed Jul 24, 2017
1 parent 91e78a5 commit dc35950
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 5 deletions.
3 changes: 2 additions & 1 deletion Changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,5 @@
- Muxing All The Subtitles [2017.07.03]
- Fix for special characters and #15 [2017.07.05]
- Episode Download Range supprt Added [2017.07.07]
- Added support to include fonts in the muxed videos [2017.07.09]
- Added support to include fonts in the muxed videos [2017.07.09]
- Changed mkvmerge.exe to mkvmerge to support Non-Windows Operating Systems [2017.07.24]
2 changes: 1 addition & 1 deletion ReadMe.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ Anime-dl is a Command-line program to download anime from CrunchyRoll and Funima
You can check the list of supported websites [**`HERE`**](https://github.com/Xonshiz/anime-dl/blob/master/Supported_Sites.md).

## Dependencies Installation
This script can run on multiple Operating Systems. But, the script depends on some external binaries or libs. We need `FFmpeg` and `Node.js` in our paths. There are some old streams on Crunchyroll which only support `rtmpe` streams, as noted from Issue #9. For this, you need `rtmpdump`.
This script can run on multiple Operating Systems. But, the script depends on some external binaries or libs. We need `FFmpeg`, `mkvmerge` and `Node.js` in our paths. There are some old streams on Crunchyroll which only support `rtmpe` streams, as noted from Issue #9. For this, you need `rtmpdump`.

You also need [mkvmerge.exe](https://mkvtoolnix.download/downloads.html) in your `PATH` or `Working Directory`.

Expand Down
2 changes: 1 addition & 1 deletion anime_dl/sites/crunchyroll.py
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ def singleEpisode(self, url, cookies, token, resolution):
if len(font_files) == 0:
fonts = ''

mkv_merge_command = 'mkvmerge.exe --ui-language en --output "%s" ' % str(file_name).replace(
mkv_merge_command = 'mkvmerge --ui-language en --output "%s" ' % str(file_name).replace(
".mp4",
".mkv") + '"' + str(
file_name) + '" ' + ' '.join(subs_files) + ' ' + str(fonts)
Expand Down
3 changes: 2 additions & 1 deletion docs/Changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,5 @@
- Muxing All The Subtitles [2017.07.03]
- Fix for special characters and #15 [2017.07.05]
- Episode Download Range supprt Added [2017.07.07]
- Added support to include fonts in the muxed videos [2017.07.09]
- Added support to include fonts in the muxed videos [2017.07.09]
- Changed mkvmerge.exe to mkvmerge to support Non-Windows Operating Systems [2017.07.24]
2 changes: 1 addition & 1 deletion docs/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ Anime-dl is a Command-line program to download anime from CrunchyRoll and Funima
You can check the list of supported websites [**`HERE`**](https://github.com/Xonshiz/anime-dl/blob/master/Supported_Sites.md).

## Dependencies Installation
This script can run on multiple Operating Systems. But, the script depends on some external binaries or libs. We need `FFmpeg` and `Node.js` in our paths. There are some old streams on Crunchyroll which only support `rtmpe` streams, as noted from Issue #9. For this, you need `rtmpdump`.
This script can run on multiple Operating Systems. But, the script depends on some external binaries or libs. We need `FFmpeg`, `mkvmerge` and `Node.js` in our paths. There are some old streams on Crunchyroll which only support `rtmpe` streams, as noted from Issue #9. For this, you need `rtmpdump`.

You also need [mkvmerge.exe](https://mkvtoolnix.download/downloads.html) in your `PATH` or `Working Directory`.

Expand Down

0 comments on commit dc35950

Please sign in to comment.