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

Emacs 29 fix - make-directory behavior #2895

Closed nevsan closed 2 weeks ago

nevsan commented 1 year ago

The behavior of make-directory changed such that it will return t if the directory exists, rather than always returning nil. Here, we update the conditional so that it ensures that ps is not a list, instead of just checking if null.

renard commented 7 months ago

Thanks for this patch. Wouldn't something like

((or (and (< emacs-major-version 29) (null ps)) (not (consp ps)) .... )

be more appropriate?

Long story short: keep current behaviour up to emacs below 29 or use your trick otherwise?

And can you also please add a comment in git commit message referring to emacs commit bef1edc9cacb976120dff73b4d7bbdce6ade982b which introduce that change and the reason for your patch? That would be appreciated.

TIA

saffroy commented 2 weeks ago

Fixed in #2890

alexott commented 2 weeks ago

closing as fixed