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

Trying to install with natural bars patch #29

Closed
Xanyism opened this issue Apr 19, 2017 · 25 comments
Closed

Trying to install with natural bars patch #29

Xanyism opened this issue Apr 19, 2017 · 25 comments

Comments

@Xanyism
Copy link

Xanyism commented Apr 19, 2017

I am trying install emacs-plus after being redirected from the spacemacs guide to installing the prerequisites on OSX. https://github.com/syl20bnr/spacemacs#macos

brew install emacs-plus --HEAD --with-natural-title-bar

Lands me this output: https://pastebin.com/09MRsynS

I am rather new to managing packages with brew and also relatively new to configuring emacs, so I can't really tell where it's being hung up. For what it's worth, I used to have GNU Emacs installed through homebrew using the some options that I think d12frosted suggested in a comment (like --with-cocoa, --with-libxsrvg, etc.). I have since uninstalled it using brew uninstall emacs and it seemed to alleviate some of the conflicts except for the ones outlined above. I apologize in advance if this is mundance, but any help is much appreciated.

@idoo
Copy link

idoo commented Apr 20, 2017

it works well for me

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

Since #27, v-fork has been merged into emacs, so I reinstalled it with both --HEAD and --with-natural-title-bar and it also works for me.

I do not think its comes from the patch. I have you simply tried:

brew install emacs-plus --HEAD

Just to be sure ?

@d12frosted
Copy link
Owner

Hey @Xanyism

Unfortunately, I can't reproduce this issue. Could you please run brew doctor to see if there is something funky with your setup. Also, try to brew install emacs-plus --HEAD as @jwintz suggested or brew install -s emacs-plus (to avoid bottle).

BTW, does it work when you brew install emacs --HEAD?

@d12frosted
Copy link
Owner

Can reproduce it. Will look into it.

@d12frosted
Copy link
Owner

Ok, it fails when I brew install emacs --HEAD as well. Could anyone please try to install Emacs from sources out of brew, please?

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

Just done so. Used github's mirror (https://github.com/emacs-mirror/emacs) since savannah seemed down. Last commit is 9 hours ago so I guess the mirror is up to date.

It builds fine.

screen shot 2017-04-20 at 13 35 14

I'll take the opportunity to work on #30.

@d12frosted
Copy link
Owner

@jwintz thanks a lot! I've fired an issue in brew repo for emacs formula. Maybe they'll help to find the problem.

I'll take the opportunity to work on #30.

Will be great 😸 🙇

@idoo
Copy link

idoo commented Apr 20, 2017

@jwintz can you share your emacs config please? :)

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

There you go: https://github.com/jwintz/prolusion.

Clone it as .prolusion.d in your HOME folder, and symlink .emacs.d to it.

Feel free to fork, you may not be interested in all layers.

Also, do not use the installer, it only fits my needs ;-)

@d12frosted
Copy link
Owner

@Xanyism
Could you please run brew doctor and fix as many as possible issues and try again afterwards? Also try updating Xcode. I've heard it fails for some people with outdated Xcode.

@Xanyism
Copy link
Author

Xanyism commented Apr 20, 2017

Sorry for the delay, will do brew doctor and Xcode. For what it's worth, it I was able to install with brew install emacs-plus --with-natural-title-bar and I just tried to run brew install emacs-plus --HEAD after doing a bit of doctoring yielded errors as well.

https://pastebin.com/qX8gESa9

@d12frosted
Copy link
Owner

Ok, please try installing it using HOMEBREW_MAKE_JOBS=1 brew install emacs-plus --HEAD --with-natural-title-bar. It will take very long (on my computer it took 14 minutes). But works 😸

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

As for --with-natural-title-bar, I checked, it still works with --HEAD, as nsterm.m has not been changed.

And just for the comment, I successfully built it with 16 cores, but I do not know about HOMEBREW_MAKE_JOBS default value.

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

@Xanyism, could you post the brew doctor output ?

@Xanyism
Copy link
Author

Xanyism commented Apr 20, 2017

Here it is, https://pastebin.com/UjCUxX0j

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

Ok, this basically means you have installed Tcl/Tk with /usr/local as a prefix, and that your python does not come from brew, all that is no big deal when it comes to installing emacs.

@d12frosted
Copy link
Owner

@jwintz what is your version of xcode?

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

screen shot 2017-04-20 at 20 10 56

@jwintz
Copy link
Contributor

jwintz commented Apr 20, 2017

Also, make sure xcode-select is setup correctly.

screen shot 2017-04-20 at 20 14 57

@d12frosted
Copy link
Owner

Hey @Xanyism, any luck on installing with --HEAD?

@JonaMX
Copy link

JonaMX commented Apr 25, 2017

I'm getting the following error while trying to install:

brew install emacs-plus --HEAD --with-natural-title-bars
==> Installing emacs-plus from d12frosted/emacs-plus
Warning: d12frosted/emacs-plus/emacs-plus: this formula has no --with-natural-title-bars option so it will be ignored!
==> Cloning https://github.com/emacs-mirror/emacs.git
Updating /Users/Jonatan/Library/Caches/Homebrew/emacs-plus--git
==> Checking out branch master
==> Downloading https://gist.githubusercontent.com/aatxe/260261daf70865fbf1749095de9172c5/raw/214b50c62450be1cbee9f11cecba846dd66c7d06/patch-multicolor-font.diff
Already downloaded: /Users/Jonatan/Library/Caches/Homebrew/emacs-plus--patch-5af2587e986db70999d1a791fca58df027ccbabd75f45e4a2af1602c75511a8c.diff
==> Patching
==> Applying patch-multicolor-font.diff
patching file src/macfont.m
Hunk #1 succeeded at 2360 (offset -13 lines).
==> ./autogen.sh
==> ./configure --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-plus/HEAD-5b0fdef/share/info/emacs --prefix=/usr/local/Cellar/emacs-plus/HE
==> make
Last 15 lines from /Users/Jonatan/Library/Logs/Homebrew/emacs-plus/03.make:
make[2]: *** [../lisp/leim/ja-dic/ja-dic.el] Error 255
make[1]: *** [../lisp/leim/leim-list.el] Error 2
make[1]: *** Waiting for unfinished jobs....
Directories for loaddefs: . ./calc ./calendar ./cedet ./cedet/ede ./cedet/semantic ./cedet/semantic/analyze ./cedet/semantic/bovine ./cedet/semantic/decorate ./cedet/semantic/symref ./cedet/semantic/wisent ./cedet/srecode ./emacs-lisp ./emulation ./erc ./eshell ./gnus ./image ./international ./language ./leim ./leim/ja-dic ./leim/quail ./mail ./mh-e ./net ./nxml ./org ./play ./progmodes ./textmodes ./url ./vc
EMACSLOADPATH= '../src/bootstrap-emacs' -batch --no-site-file --no-site-lisp -l autoload \
	    --eval '(setq autoload-ensure-writable t)' \
	    --eval '(setq autoload-builtin-package-versions t)' \
	    --eval '(setq generated-autoload-file (expand-file-name (unmsys--file-name "./loaddefs.tmp")))' \
	    -f batch-update-autoloads . ./calc ./calendar ./cedet ./cedet/ede ./cedet/semantic ./cedet/semantic/analyze ./cedet/semantic/bovine ./cedet/semantic/decorate ./cedet/semantic/symref ./cedet/semantic/wisent ./cedet/srecode ./emacs-lisp ./emulation ./erc ./eshell ./gnus ./image ./international ./language ./leim ./leim/ja-dic ./leim/quail ./mail ./mh-e ./net ./nxml ./org ./play ./progmodes ./textmodes ./url ./vc
Not registering prefix "lo" from completion.  Affects: ("locate-completion-entry" "locate-completion-entry-retry" "locate-completion-db-error" "load-completions-from-file")
Not registering prefix "*" from ielm.  Affects: ("*" "**" "***" "*1" "*2" "*3")
Not registering prefix "is" from ind-util.  Affects: ("is13194-default-repertory" "is13194-repertory-to-ucs-script" "is13194-to-ucs-devanagari-hashtbl" "is13194-to-ucs-devanagari-regexp" "is13194-to-ucs-bengali-hashtbl" "is13194-to-ucs-bengali-regexp" "is13194-to-ucs-assamese-hashtbl" "is13194-to-ucs-assamese-regexp" "is13194-to-ucs-gurmukhi-hashtbl" "is13194-to-ucs-gurmukhi-regexp" "is13194-to-ucs-gujarati-hashtbl" "is13194-to-ucs-gujarati-regexp" "is13194-to-ucs-oriya-hashtbl" "is13194-to-ucs-oriya-regexp" "is13194-to-ucs-tamil-hashtbl" "is13194-to-ucs-tamil-regexp" "is13194-to-ucs-telugu-hashtbl" "is13194-to-ucs-telugu-regexp" "is13194-to-ucs-malayalam-hashtbl" "is13194-to-ucs-malayalam-regexp" "is13194-to-ucs-kannada-hashtbl" "is13194-to-ucs-kannada-regexp" "iscii-to-ucs-region")
Not registering prefix "se" from page-ext.  Affects: ("search-pages" "set-page-delimiter")
../build-aux/move-if-change ./loaddefs.tmp loaddefs.el
make: *** [src] Error 2

If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
https://github.com/d12frosted/homebrew-emacs-plus/issues

Here's the output of my brew doctor:

brew doctor
Your system is ready to brew.

@d12frosted
Copy link
Owner

d12frosted commented Apr 25, 2017 via email

@d12frosted
Copy link
Owner

Closing this issue for inactivity. Let me know if you need any assistance! 😸

@danpolanco
Copy link

I may have run into the same issue. It was caused by installing via brew install emacs-plus first, and then brew install emacs-plus --HEAD --with-natural-title-bars. It didn't like that I had it installed already. And the error was a bit technical/hard to follow.

I did brew rm emacs-plus && brew cleanup && brew install emacs-plus --HEAD --with-natural-title-bars. The cleanup is likely not necessary. But that fixed it.

Hopefully this helps someone else. If I get a chance, I'll submit a pull request to syl20bnr/spacemacs. I'll try to make the documentation easier to follow correctly the first time.

@d12frosted
Copy link
Owner

Personally, I think that this is something not clear on the brew side.

If you have installed a package with a specific set of options, in order to install it with another you have to brew uninstall first and then install it again. If you brew install instead, you'll get an error saying that this package is already installed. If you brew reinstall it starts reinstallation process, but with the previous set of options. The last one is confusing me.

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

6 participants