diff options
author | Ashley Duckworth | 2021-01-23 12:51:22 -0600 |
---|---|---|
committer | Ashley Duckworth | 2021-01-23 12:51:22 -0600 |
commit | 91b56c70a0a78b47c318c75eaf7ef5673077224e (patch) | |
tree | 17b6db6062512e6007da082dd8dd69d9662e8f87 | |
parent | Add auth-sources config (diff) | |
download | emacs-91b56c70a0a78b47c318c75eaf7ef5673077224e.tar.gz emacs-91b56c70a0a78b47c318c75eaf7ef5673077224e.zip |
Fork and correct cl dependency in font-utils and list-utils
See config.org for details.
-rw-r--r-- | config.org | 64 |
1 files changed, 57 insertions, 7 deletions
diff --git a/config.org b/config.org index 930b145..8323986 100644 --- a/config.org +++ b/config.org | |||
@@ -314,20 +314,70 @@ underline below all the text. | |||
314 | (setq-default x-underline-at-descent-line t) | 314 | (setq-default x-underline-at-descent-line t) |
315 | #+end_src | 315 | #+end_src |
316 | 316 | ||
317 | *** COMMENT Unicode fonts :package: | 317 | *** Unicode fonts :package: |
318 | 318 | ||
319 | =unicode-fonts= pulls in some other packages that still require the | 319 | =unicode-fonts= pulls in some other packages that still require the |
320 | deprecated =cl= library. I'm including this stuff here because it's | 320 | deprecated =cl= library. So, I've forked those libraries to require |
321 | good to have, ready to use, but I'm holding off on actually including | 321 | =cl-lib= instead. |
322 | it in my config -- thus the =COMMENT= in the header. | 322 | |
323 | *** First: un-fuck =font-utils= and =list-utils= | ||
324 | |||
325 | **** List-utils first | ||
326 | |||
327 | Since =font-utils= depends on =list-utils=, if I load the former first, it | ||
328 | pulls in the unpatched latter. /So/ I need to do =list-utils= first. | ||
329 | (=*straight-process*= is your friend, y'all!) | ||
330 | |||
331 | Since =list-utils= requires =cl= in line 259 (see [[https://github.com/rolandwalker/list-utils/issues/6][this issue]], apparently | ||
332 | just changing it breaks many tests, but I'll run with it until Emacs | ||
333 | complains), I need to fork and change that to a =cl-lib=. | ||
323 | 334 | ||
324 | #+begin_src emacs-lisp :noweb-ref packages | 335 | #+begin_src emacs-lisp :noweb-ref packages |
325 | (straight-use-package 'unicode-fonts) | 336 | (straight-use-package '(list-utils |
337 | :host github | ||
338 | :repo "rolandwalker/list-utils" | ||
339 | :fork (:repo "duckwork/list-utils"))) | ||
340 | #+end_src | ||
341 | |||
342 | **** Font-utils | ||
343 | |||
344 | I was able to actually create a [[https://github.com/rolandwalker/font-utils/pull/2][PR]] for this one, so fingers crossed. | ||
345 | Since the last update on =font-utils= was in 2015, I'm not super hopeful | ||
346 | that my fix will get merged upstream, but I'm using a =:fork= argument | ||
347 | to stay hopeful. | ||
348 | |||
349 | #+begin_src emacs-lisp :noweb-ref packages | ||
350 | (straight-use-package '(font-utils | ||
351 | :host github | ||
352 | :repo "rolandwalker/font-utils" | ||
353 | :fork (:repo "duckwork/font-utils"))) | ||
354 | #+end_src | ||
355 | |||
356 | **** Unicode-fonts | ||
357 | |||
358 | /Okay/ ... pull requests in, time to load =unicode-fonts=. | ||
359 | |||
360 | #+begin_src emacs-lisp :noweb-ref packages | ||
361 | (straight-use-package '(unicode-fonts | ||
362 | :host github | ||
363 | :repo "rolandwalker/unicode-fonts")) | ||
326 | (require 'unicode-fonts) | 364 | (require 'unicode-fonts) |
327 | #+end_src | 365 | #+end_src |
328 | 366 | ||
329 | #+begin_src emacs-lisp :noweb-ref modes | 367 | According to [[https://github.com/rolandwalker/unicode-fonts/issues/3][Issue #3]], there can be problems with =unicode-fonts-setup= |
330 | (unicode-fonts-setup) | 368 | when using a daemon. Instead of forking this repo and merging [[https://github.com/rolandwalker/unicode-fonts/pull/4][PR #4]] |
369 | into my personal fork, I'll use the workaround described in the | ||
370 | issue. Trying to un-fuck these packages by Roland Walker have been | ||
371 | enough work today. | ||
372 | |||
373 | #+begin_src emacs-lisp :noweb-ref hooks | ||
374 | (defun hook--unicode-fonts-setup (frame) | ||
375 | "Run `unicode-fonts-setup', then remove the hook." | ||
376 | (select-frame frame) | ||
377 | (unicode-fonts-setup) | ||
378 | (remove-hook 'after-make-frame-functions #'hook--unicode-fonts-setup)) | ||
379 | |||
380 | (add-hook 'after-make-frame-functions #'hook--unicode-fonts-setup) | ||
331 | #+end_src | 381 | #+end_src |
332 | 382 | ||
333 | ** Theming | 383 | ** Theming |