Closed abcdw closed 2 years ago
Please attach the file which breaks here. I haven't seen uncompression errrors on my system.
Thank you very much for the fix, it works for me.
Not sure if making marginalia depend on gzip and coreutils is a good idea in long-term, but it works. I'll report separately if I find it problematic.
Not sure if making marginalia depend on gzip and coreutils is a good idea in long-term, but it works. I'll report separately if I find it problematic.
Emacs itself uses these commands behind the scene in jka-compr, otherwise I wouldn't have gone that route. But it could be that we have to make the library command configurable in the long term or that we have to auto detect different compression programs. I doubt that the current approach works on Windows, but Windows doesn't have high priority for me since I also don't have convenient access to it. The issues are not serious though, you will just not see any annotations there. :shrug:
I must also say that Consult has a few long standing Windows issues but so far nobody has come forward proposing a fix in the form of a proper PR. So I assume that there are just not that many users of Emacs on Windows, who also use the Marginalia, Consult, Embark, Vertico, etc. ecosystem.
I found this change didn't work for me on macos, no library had their doc string included in the annotations. The problem was the zcat
command on macos failed as described here. One of the answers there, using zcat < %s
instead of zcat -f %s
only partially worked (some doc strings appeared, not all). What did work for me was installing the gnu gzip packages via homebrew, brew install gzip
. I don't think marginalia needs to change for this, but someone with this problem finding this bug can find an answer here.
Is something else available ootb in Mac? For example gunzip -c
? Or gzcat?
There is but it behaved the same as zcat < %s
, only working for some libraries (I assume the difference is those that are .el vs .el.gz but didn't check). The problem is what's installed on macos is very old versions of gnu packages because they wouldn't go to gplv3.
@hmelman please check https://github.com/minad/marginalia/commit/26f2bd9ee7b63bcad6604108e2f565b34bc6083b
With marginalia-mode enabled, on
find-library
I get the following error:It happens automatically with vertico and on TAB press, when using default completion buffer.
Happens on 097bd81743026c9b4889b860efd0283b26e64ccf and probably earlier versions too.