Skip to content

Commit

Permalink
Add support for OS X
Browse files Browse the repository at this point in the history 
Since OS X doesn't support the `-D` install option, we use `-d` to create a directory, followed by the file that needs to be moved there.
  • Loading branch information
@christianbundy
christianbundy committed Jul 4, 2014
1 parent 3c6fe80 commit 211692d
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions Makefile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,10 @@ all:

.PHONY: install
install:
install -Dm755 sshcd -- "$(DESTDIR)$(BINDIR)/$(COMMAND)"
install -Dm644 LICENSE -- "$(DESTDIR)$(DATADIR)/licenses/$(PKGNAME)/LICENSE"
install -dm755 "$(DESTDIR)$(BINDIR)"
install -m755 sshcd "$(DESTDIR)$(BINDIR)/$(COMMAND)"
install -dm644 "$(DESTDIR)$(DATADIR)/licenses/$(PKGNAME)"

This comment has been minimized.

Sign in to view

Copy link
@maandree

maandree Jul 8, 2014

Contributor

It should be -dm755, the x (bit 1) permission lets users ls the directory.

This comment has been minimized.

Sign in to view

Copy link
@christianbundy

christianbundy Jul 8, 2014

Author Member

Thanks @maandree, I totally missed that. Fixed in c914512.
install -m644 LICENSE "$(DESTDIR)$(DATADIR)/licenses/$(PKGNAME)/LICENSE"

.PHONY: uninstall
uninstall:
Expand All @@ -25,4 +27,3 @@ uninstall:
.PHONY: clean
clean:
@echo Nothing to clean

0 comments on commit 211692d

Please sign in to comment.