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: if statement breaking copy() and ccopy() on macos #395

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tjex
Copy link

@tjex tjex commented Dec 11, 2023

Resolves issue #361.

Problem: ccopy and copy return an error on mac, "copy: supported only in the Desktop version"

This is because the cht.sh script checks for presence of the $DISPLAY environment variable, which
is no longer a part of MacOS as it comes from the X11 days. So, calling the function would always
return the error on mac.

pr366 attempted to fix this with a similar approach.

The solution of this PR reuses the $is_macos boolean variable, which is already a part of this
script.

It now only returns the error if $DISPLAY is null/empty and the OS is not MacOS.

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.

None yet

1 participant