search

larrykollar / tines

Tines is a console-based outliner/planner/notebook. It is a fork of the hnb outliner, which has not been updated in >10 years.
Other
49 stars 7 forks source link

make install fails #10

Open peterjschroeder opened 7 years ago

peterjschroeder commented 7 years ago

make install fails with this error

install -D /usr/local/bin install: missing destination file operand after '/usr/local/bin' Try 'install --help' for more information. Makefile:23: recipe for target 'install' failed make: *** [install] Error 1

This is on Debian Sid, gcc 7.2.0

larrykollar commented 7 years ago

Workaround: change the first four lines under install: to read:

install $(INSTFLAGS) -d $(BINDIR)
install $(INSTFLAGS) -d $(SHAREDIR)
install $(INSTFLAGS) -d $(MANDIR)/man1
install $(INSTFLAGS) -d $(MANDIR)/man7

Of course, that will break on non-Linux installs. I have a patch that should make all this faffing around unnecessary, and will post it to the dev branch after I finish testing.

peterjschroeder commented 7 years ago

Thank you.

larrykollar commented 7 years ago

Just pushed an update that should fix this, and make it unnecessary to fiddle with the Makefile further. Please test!

peterjschroeder commented 7 years ago

It still fails. It looks like the capital D is for non-cygwin windows only and everything else uses the lowercase d? If so something like this works.

ifeq ($(OS),Windows_NT) INSTFLAGS=-D else INSTFLAGS=-d endif

aThorp96 commented 6 years ago

I just ran into this issue as well. In the Makefile's documentation it claims Linux needs -D while BSD and MacOS need -d. However I've checked Manjaro and Ubuntu and the install implimentation needs the -d flag for the use make wants.