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

Fix Calibre on macOS #2202

Merged
merged 1 commit into from
Dec 9, 2023
Merged

Fix Calibre on macOS #2202

merged 1 commit into from
Dec 9, 2023

Conversation

CryZFix
Copy link
Contributor

@CryZFix CryZFix commented Nov 24, 2023

Fix #2192
Changed path to calibre executable file on macOS

P.S. You may object that the installed Calibre from HomeBrew is also able to work directly with "ebook-convert", since a Symlink is created when it is installed. The solution in this commit, work for both installation options, from a DMG and HomeBrew.

@dipu-bd
Copy link
Owner

dipu-bd commented Dec 4, 2023

Can you not export a path variable to MacOS to make the ebook-convert globally accessible. like:

export PATH="$PATH:/Applications/calibre.app/Contents/MacOS"

@CryZFix
Copy link
Contributor Author

CryZFix commented Dec 5, 2023

export PATH="$PATH:/Applications/calibre.app/Contents/MacOS"

Yes, it will work, but it's a bit not obvious to macOS users who rarely use the terminal.

@dipu-bd
Copy link
Owner

dipu-bd commented Dec 7, 2023

What if the user has installed the Calibre in another folder other than /Applications/calibre.app/Contents/MacOS/ebook-convert?

This tool is not for general users anyway. Lets add some documentation in the README on how to add the ebook-convert to PATH variable, and be done with it.

Information about using Calibre on macOS has been added to the README
@CryZFix CryZFix reopened this Dec 8, 2023
@dipu-bd dipu-bd merged commit 854efc8 into dipu-bd:dev Dec 9, 2023
4 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants