dimitri / el-get

Manage the external elisp bits and pieces upon which you depend!
http://tapoueh.org/emacs/el-get.html
1.64k stars 457 forks source link

Error installing magit (w32, makeinfo installed, fails to create magit.info) #2574

Open subhacom opened 6 years ago

subhacom commented 6 years ago

Using emacs 25.3 on Windows 7 with msys64 I encounter this error when trying install magit: Error (el-get): while installing magit: el-get: c:/msys64/usr/bin/install-info.exe el-get could not build magit [c:/msys64/usr/bin/install-info.exe c:\Users\UserName\AppData\Roaming.emacs.d\el-get\magit\Documentation\magit.info dir]

I can, however, install magit from the usual packages list via list-packages command.

npostavs commented 6 years ago

Are you using a cygwin/msys version of Emacs (i.e., not win32 native)? What is the full output from M-x emacs-version?

subhacom commented 6 years ago

GNU Emacs 25.3.1 (x86_64-w64-mingw32) of 2017-09-26

npostavs commented 6 years ago

Hmm, if you do M-: system-type do you not get windows-nt?

npostavs commented 6 years ago

Is there a *el-get-build: magit* buffer with some error messages?

subhacom commented 6 years ago

Yes, I do get windows-nt for system-type. Yes, el-get-build: magit buffer shows the same error:
install-info: No such file or directory for \Users\UserName\AppData\Roaming\.emacs.d\el-get\magit\Documentation\magit.info

And indeed, there is no magit.info in "\Users\UserName\AppData\Roaming.emacs.d\el-get\magit\Documentation\"

npostavs commented 6 years ago

Oh, right, el-get tries to install info if you have install-info in PATH, but the w32 build recipe doesn't create the .info file (it assumes there is no makeinfo).

You can add (:name magit :info nil) to your el-get-sources to prevent .info installation, or (:name magit :build/windows-nt nil) to use the standard build recipe which should create the .info file, assuming you have make and makeinfo installed.

subhacom commented 6 years ago

This entry in my .emacs worked: (add-to-list 'el-get-sources '(:name magit :info nil)) I guessed you meant :info instead of :documentation. Thank you for the help.

npostavs commented 6 years ago

I guessed you meant :info instead of :documentation.

Oops! Yeah, got confused, :info is correct.