[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Merging feature/android
From: |
Robert Pluim |
Subject: |
Re: Merging feature/android |
Date: |
Tue, 21 Mar 2023 08:48:04 +0100 |
>>>>> On Mon, 20 Mar 2023 17:34:22 +0000, João Távora <joaotavora@gmail.com>
>>>>> said:
João> On Mon, Mar 20, 2023 at 2:20 PM Robert Pluim <rpluim@gmail.com> wrote:
>>
>> >>>>> On Mon, 20 Mar 2023 12:23:27 +0100, Robert Pluim
<rpluim@gmail.com> said:
>>
>> >>>>> On Mon, 20 Mar 2023 19:09:34 +0800, Po Lu <luangruo@yahoo.com>
said:
Robert> Po Lu> Do you see the clangd problem on master as well?
>>
Robert> I do now, although I could have sworn it wasnʼt failing earlier.
>>
Robert> Po Lu> I have a feeling this is some problem with clangd, but it
reliably
Robert> Po Lu> crashes for me while running this test.
>>
Robert> Po Lu> This is:
>>
Robert> Po Lu> clangd version 15.0.0 (Fedora 15.0.0-3.fc37)
>>
Robert> Debian clangd version 11.0.1-2
>>
Robert> here. Looks like itʼs an issue on master, I see if I can bisect.
>>
>> OK, so on master originally this test used rust, and was skipped on my
>> system, but now itʼs using clangd, and failing. João, any ideas?
>> Should we just add a clangd version test to
>> `eglot-test-diagnostic-tags-unnecessary-code'?
João> Hello,
João> Took me quite a while of rummaging to figure out what you are
João> talking about here. Consider changing the subject line or
João> adding a bit more context next time.
Sounds like you forgot to do (package-install 'telepathy) ;-)
João> _If_ I understand correctly, Robert, your run of
João> eglot-test-diagnostic-tags-unnecessary-code is failing
João> because of a face mismatch, presumably because only
João> clangd-11 is installed in your system, but if you install
João> a more recent clangd it goes away? If so, yes, I think a
João> version check is appropriate, yes.
Yes. Version 11 fails, version 17 works, I donʼt know where the cutoff
point is. Which version do you use?
Robert
--
- Re: Merging feature/android, (continued)
- Re: Merging feature/android, Po Lu, 2023/03/15
- Re: Merging feature/android, Robert Pluim, 2023/03/15
- Re: Merging feature/android, Po Lu, 2023/03/15
- Re: Merging feature/android, Po Lu, 2023/03/15
- Re: Merging feature/android, Robert Pluim, 2023/03/17
- Re: Merging feature/android, Po Lu, 2023/03/17
- Re: Merging feature/android, Po Lu, 2023/03/20
- Re: Merging feature/android, Robert Pluim, 2023/03/20
- Re: Merging feature/android, Robert Pluim, 2023/03/20
- Re: Merging feature/android, João Távora, 2023/03/20
- Re: Merging feature/android,
Robert Pluim <=
- eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Robert Pluim, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Robert Pluim, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, João Távora, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Robert Pluim, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Michael Albinus, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, João Távora, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Michael Albinus, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, João Távora, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, João Távora, 2023/03/21
- Re: eglot-test-diagnostic-tags-unnecessary-code fails with certain clangd versions, Michael Albinus, 2023/03/21