[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] Package proposal: EBDB
From: |
Eric Abrahamsen |
Subject: |
Re: [ELPA] Package proposal: EBDB |
Date: |
Tue, 15 Aug 2017 08:30:08 -0700 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
Stefan Monnier <address@hidden> writes:
>> Huh, interesting. Is that function completely deprecated, or still
>> usable for class-allocated slots?
>
> It's fine for class-allocated slots.
>
>>> - ebdb-vm and ebdb-mu4e will break the compilation of the package if the
>>> user doesn't have VM and mu4e installed.
>>> The appended patch tries to fix those two, but please take a look at it
>>> to make sure it still works correctly (especially the ebdb-vm part is
>>> quick&dirty, leaving a lot of warnings when VM is not installed, some
>>> of them may be real bugs).
>> I thought about extracting those two into separate packages, and
>> probably should have done that.
>
> That's one way to solve the problem, but it's not necessary.
>
>> Actually, I do know: vm and mu4e aren't installable via the package
>> manager. If I make separate packages, and put a package-requires header
>> in them for vm and mu4e, will package.el be satisfied if a user has
>> installed them via other means? Or will it bark?
>
> It will bark :-(
>
>> Another option would be a boatload of (autoload 'function "ext:vm")
>> statements.
>
> For ebdb-mu4e, I think the patch I send leads to correct results, so
> I think it's an acceptable solution.
>
> For VM, I'm not sure if the result is correct. It basically depends on
> whether part of the VM code are needed during compilation (most common
> case is if your code uses VM macros).
>
> Fixing the warnings is easy enough (rather than autoloads, you can use
> `declare-function`).
Okay, I'll apply your solution to ebdb-mu4e, then take a closer look at
vm.
Thanks,
Eric
- Re: [ELPA] Package proposal: EBDB, (continued)
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/09
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/12
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/13
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/13
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/14
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/14
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/14
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/14
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/15
- Re: [ELPA] Package proposal: EBDB,
Eric Abrahamsen <=
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/17
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/17
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/17
- Re: [ELPA] Package proposal: EBDB, Stefan Monnier, 2017/08/17
- Re: [ELPA] Package proposal: EBDB, Eric Abrahamsen, 2017/08/17