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

Install script for Mac #27

Closed
nadrad opened this issue Sep 17, 2022 · 7 comments
Closed

Install script for Mac #27

nadrad opened this issue Sep 17, 2022 · 7 comments

Comments

@nadrad
Copy link
Owner

nadrad commented Sep 17, 2022

Can someone check whether or not the install script works on Mac, and if not, come with adjustments to it to work on both platforms, or prepare a separate install script for Mac?

@otakumesi
Copy link

Although I ran the install script on M1 Mac, I was able to install this tool if I had already installed PHP and xclip.

@gloof11
Copy link

gloof11 commented Sep 23, 2022

There are additional dependencies that need installing, at least on Intel macs. I'm gonna write some OS detect into the install script and do a pull request.

@airingursb
Copy link

brew install php
brew install xclip
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/nadrad/h-m-m/main/install.sh)"

@nadrad
Copy link
Owner Author

nadrad commented Oct 24, 2022

brew install xclip

It uses pbcopy and pbpaste in Mac; xclip is not needed.

@raf2000
Copy link

raf2000 commented Jan 28, 2023

it works after I commented this part

#Test if xclip, xsel or wl-clipboard are on PATH
#if ! command -v xclip > /dev/null 2>&1 && ! command -v xsel > /dev/null 2>&1 && ! command -v wl-clipboard > /dev/null 2>&1 ;then
#printf "ERROR: xclip, xsel or wl-clipboard must exist on PATH. Installation cancelled.\n";
#printf "%s\n" "$MINIMUM_REQUIREMENTS"
#exit 1
#fi

then, the pre-reqs "brew install xclip" and it worked like a charm

@ynac
Copy link

ynac commented Feb 22, 2023

I just downloaded the zip and tossed it in a folder and opened it up. Beautiful program. Nice work!

@nadrad
Copy link
Owner Author

nadrad commented Jul 23, 2023

I think 35a6bae is enough for now.

@nadrad nadrad closed this as completed Jul 23, 2023
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

No branches or pull requests

6 participants