[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] New package xeft.el
From: |
Yuan Fu |
Subject: |
Re: [ELPA] New package xeft.el |
Date: |
Thu, 5 Jan 2023 22:46:15 -0700 |
> On Jan 4, 2023, at 12:24 PM, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
>
> Yuan Fu [2023-01-02 20:11:50] wrote:
>> I wrote xeft.el awhile ago, it allows you to search text notes really fast
>> by using a dynamic module. My initial plan is to somehow plug the dynamic
>> module into Deft some day and I’ll keep xeft to myself. Turns out it’s not
>> very feasible to make Deft use the dynamic module, so now I thought I might
>> as well publish xeft. After all I think it is really cool and I want more
>> people to use it :-)
>>
>> You can find xeft here: https://git.sr.ht/~casouri/xeft
>
> I'm a bit uncomfortable with the way this is set up:
> it's unusable without downloading further code first.
>
> Why not do it like the `pq` package, which includes the C source code in
> the GNU ELPA package? IOW merge your `xapian-lite` repository into the
> `xeft` repository. Or make a GNU ELPA package for `xapian-lite` if you
> think the two should not be so closely tied to one another?
Yeah I can do that. The download feature is probably over-engineering for not
much benefit :-)
> Other details I saw along the way:
> - You claim you have precompiled packages for GNU/Linux but that's only
> true for amd64, which is only a particular subset of GNU/Linux
> (most of my GNU/Linux machines (i386, armhf, arm64) are outside of
> that subset, for example).
That’s true, probably better to not have the download feature.
> - Your source code includes `emacs-module.h` which should not be
> necessary because that file is distributed with Emacs (the way `pq`
> finds it is not ideal, but this might be a good opportunity to look
> at improving the situation and providing a standardized way for
> ELisp packages to find that file).
Is it included in the distributed Emacs? I thought you would need the source of
Emacs to access that file.
>
> - `xapian-lite.cc` lacks a license blurb.
I can add it.
Yuan
- [ELPA] New package xeft.el, Yuan Fu, 2023/01/02
- Re: [ELPA] New package xeft.el, Stefan Monnier, 2023/01/04
- Re: [ELPA] New package xeft.el, Karl Fogel, 2023/01/04
- Re: [ELPA] New package xeft.el,
Yuan Fu <=
- Re: [ELPA] New package xeft.el, Stefan Monnier, 2023/01/06
- Re: [ELPA] New package xeft.el, Yuan Fu, 2023/01/10
- Re: [ELPA] New package xeft.el, Richard Stallman, 2023/01/12
- Re: [ELPA] New package xeft.el, Dr. Arne Babenhauserheide, 2023/01/13
- Re: [ELPA] New package xeft.el, Richard Stallman, 2023/01/15
- Re: [ELPA] New package xeft.el, Dr. Arne Babenhauserheide, 2023/01/16
- Re: [ELPA] New package xeft.el, Richard Stallman, 2023/01/16
- Re: [ELPA] New package xeft.el, Eli Zaretskii, 2023/01/17
- Re: [ELPA] New package xeft.el, Richard Stallman, 2023/01/22