[emms-help] wrong path to install-info on Cygwin

Post by Filipp Gunbin
Makefile uses `INSTALLINFO = /usr/sbin/install-info
--info-dir=$(INFODIR)' to set path to install-info program. On Cygwin
that does not work because install-info is located in /usr/bin.

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

--
"Cut your own wood and it will warm you twice"

Filipp Gunbin

2014-05-29 21:20:42 UTC

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

Thanks a lot! But I ran into another problem
( https://cygwin.com/ml/cygwin/2014-05/msg00515.html ), it prevents me
from checking this right now, I hope it will get fixed and I'll try this
one.

--
Filipp Gunbin

Filipp Gunbin

2014-05-29 22:08:58 UTC

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

Thanks a lot! But I ran into another problem
( https://cygwin.com/ml/cygwin/2014-05/msg00515.html ), it prevents me
from checking this right now, I hope it will get fixed and I'll try this
one.

These three lines

install -m 644 $(ALLSOURCE) $(SITELISP)
install -m 644 $(ALLCOMPILED) $(SITELISP)
install -m 0644 $(DOCDIR)emms.info $(INFODIR)/emms

call the wrong install from make on Cygwin. There should be
/usr/bin/install, too.

--
Filipp Gunbin

Filipp Gunbin

2014-05-29 22:13:07 UTC

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

Thanks a lot! But I ran into another problem
( https://cygwin.com/ml/cygwin/2014-05/msg00515.html ), it prevents me
from checking this right now, I hope it will get fixed and I'll try this
one.

These three lines
install -m 644 $(ALLSOURCE) $(SITELISP)
install -m 644 $(ALLCOMPILED) $(SITELISP)
install -m 0644 $(DOCDIR)emms.info $(INFODIR)/emms
call the wrong install from make on Cygwin. There should be
/usr/bin/install, too.

The first fix (ginstall-info) does not work:

/usr/bin/ginstall-info --info-dir=/usr/local/info doc/emms.info
make: /usr/bin/ginstall-info: Command not found

Probably it should not require ginstall-info.

--
Filipp Gunbin

Yoni Rabkin

2014-05-30 17:33:47 UTC

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

Thanks a lot! But I ran into another problem
( https://cygwin.com/ml/cygwin/2014-05/msg00515.html ), it prevents me
from checking this right now, I hope it will get fixed and I'll try this
one.

These three lines
install -m 644 $(ALLSOURCE) $(SITELISP)
install -m 644 $(ALLCOMPILED) $(SITELISP)
install -m 0644 $(DOCDIR)emms.info $(INFODIR)/emms
call the wrong install from make on Cygwin. There should be
/usr/bin/install, too.

/usr/bin/ginstall-info --info-dir=/usr/local/info doc/emms.info
make: /usr/bin/ginstall-info: Command not found
Probably it should not require ginstall-info.

We should be using GNU install-info, so this latest commit:
http://git.savannah.gnu.org/cgit/emms.git/commit/?id=90e8fcee1a5792446074f4b2a381877dbee6c7c6

...will only use it if it actually exists, otherwise just
/usr/bin/install-info

--
"Cut your own wood and it will warm you twice"

Filipp Gunbin

2014-05-30 23:59:00 UTC

Post by Yoni Rabkin
http://git.savannah.gnu.org/cgit/emms.git/commit/?id=90e8fcee1a5792446074f4b2a381877dbee6c7c6
...will only use it if it actually exists, otherwise just
/usr/bin/install-info

Now everything goes fine, thanks!

Filipp

Yoni Rabkin

2014-05-30 17:36:40 UTC

I've fixed this in the git repo. Can you please check? (I don't have
access to a machine with Cygwin.)

Thanks a lot! But I ran into another problem
( https://cygwin.com/ml/cygwin/2014-05/msg00515.html ), it prevents me
from checking this right now, I hope it will get fixed and I'll try this
one.

These three lines
install -m 644 $(ALLSOURCE) $(SITELISP)
install -m 644 $(ALLCOMPILED) $(SITELISP)
install -m 0644 $(DOCDIR)emms.info $(INFODIR)/emms
call the wrong install from make on Cygwin. There should be
/usr/bin/install, too.

/usr/bin/install is exactly what `install' points to. Which install does
it call on Cygwin? And where in Cygwin is the right `install' binary?

--
"Cut your own wood and it will warm you twice"

Filipp Gunbin

2014-05-30 23:57:55 UTC