[BRLTTY] 5.4 has been released.

[Didier Spaier]

On 29/06/2016 16:27, Dave Mielke wrote:
> [quoted lines by Didier Spaier on 2016/06/29 at 10:50 +0200]
> 
>> On 29/06/2016 09:27, Didier Spaier wrote:
>>> On 29/06/2016 08:59, Dave Mielke wrote:
>>>> [quoted lines by Didier Spaier on 2016/06/29 at 08:45 +0200]
>>>>
>>>>> In dmesg's output and /var/log/messages I see:
>>>>>
>>>>> Jun 29 08:10:03 didier kernel: <27>[    6.536711] udevd[581]: invalid key/value pair in file /lib/udev/rules.d/40-usb-brltty.rules on line 450, starting at character 1 ('R')
>>>>
>>>> Line 450 says:
>>>>
>>>>    RUN+="/bin/brltty -E
>>>>
>>>> I didn't catch this since I do use systemd so, here, that line isn't executed. 
>>>> It's missing a closing quote. Could you pldease add the closing quote and let 
>>>> us know if that resolVes your problem? The line, in other wrods, should look 
>>>> like this:
>>>>
>>>>    RUN+="/bin/brltty -E"
>>>
>>> Yes Dave I will try.
>>>
>>> @Philippe: please wait that I send you a fixed package before testing.
>>>
>>> @Patrick: I will let you know the outcome so that you know if upgrading is safe. 
>>
>> Bad news:
>> 1. I do not have the message from udevd anymore but...
>> 2. Philippe's Braille display does not work at all after upgrading.
>>   Unfortunately he can't investigate further right now (no sighted
>>   person around to help so he can't access his computer).
>>
>> So I would suggest that another user without systemd tests 5.4 if
>> possible. Just wondering: maybe these recent changes:
>>> Systemd/Udev changes:
>>>   Multiple USB braille devices are handled - each starts its own brltty.
>>>   Disconnecting a USB braille device stops the associated brltty.
>> are causing the issue? But that's just an assumption, of course.
>>
>> Didier
>> _______________________________________________
>> This message was sent via the BRLTTY mailing list.
>> To post a message, send an e-mail to: BRLTTY at mielke.cc
>> For general information, go to: http://mielke.cc/mailman/listinfo/brltty
> 
> I'm not sure what you meant aout #450 for the quote. Just in case, my edited 
> udev.rules is attached to this message. Note that it contains more changes than 
> just fixing the quote problem.

Sorry I couldn't answer to this message sooner although I saw it yesterday.
By the way a message that I sent to your email address was also rejected
(and sorry to have bothered you with this issue in two voicemail messages).

To clarify, in the source tarball I sent to Philippe, only the commit
#cbdd024 was somehow applied (I was not aware of the further ones then).

But it seemed to work with just this one, so should we make more tests
with all commits applied and if the answer is yes, what use cases should
be tested?

Didier