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

Opam package calls the 'install' tool with an illegal option on OSX #469

Closed
beurdouche opened this issue Feb 1, 2016 · 5 comments
Closed

Comments

@beurdouche
Copy link
Member

When installing the fstar opam package on OSX $ opam install fstar
there is yet another boring incompatibility between Linux and Unix tools...

# install -m 755 -D -T fstar.exe /Users/bbeurdouche/.opam/system/bin/fstar.exe
### stderr ###
# [...]
# Warning 26: unused variable for1_sig.
# File "FStar_Extraction_ML_ExtractExp.ml", line 547, characters 8-10:
# Warning 26: unused variable t0.
# install: illegal option -- D
# usage: install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
#                [-o owner] file1 file2
#        install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
#                [-o owner] file1 ... fileN directory
#        install -d [-v] [-g group] [-m mode] [-o owner] directory ...
# make: *** [install] Error 64
@msprotz
Copy link
Collaborator

msprotz commented Feb 1, 2016

Urgh. Any ideas Benjamin?

@beurdouche
Copy link
Member Author

Unfortunately, I am not sure there is a generic way to circumvent these kind of problems...

That said, everyone that has coreutils installed through Homebrew or Macports has all those utilities in the GNU version available by prefixing the tools with g (ginstall -D will work properly while install -D fails).

A fix could be to simply search if ginstall exists and use it if it is available.
It's not generic, still it will be a good fix a lot of people...

@beurdouche
Copy link
Member Author

@msprotz, I think, I'll patch the Makefile this way in order to fix (#468) ...

@msprotz
Copy link
Collaborator

msprotz commented Feb 1, 2016

Yes that would be good.

That sounds like one more argument in favor of a configure... see the last few lines of https://github.com/protz/mezzo/blob/master/configure to see how to detect this kind of stuff

@beurdouche
Copy link
Member Author

Fixed by 85f3ba4

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants