[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#796591: debian-installer: beep on x86 UEFI for accessibility



Cyril Brulebois <kibi@debian.org> (2015-09-14):
> Hi Samuel + Didier,
> 
> Samuel Thibault <sthibault@debian.org> (2015-09-14):
> > Cyril Brulebois, le Mon 14 Sep 2015 04:20:27 +0200, a écrit :
> > > Samuel Thibault <sthibault@debian.org> (2015-09-14):
> > > > Just to confirm: I could test on my UEFI laptop, beep works fine.
> > > 
> > > Great, thanks; do you want me to backport that to stable?
> > 
> > That can be useful, yes.
> 
> ACK, I have that locally.
> 
> 
> I also looked at the following commit (1af5cf90995c12bdb70083181d9a97a504e032aa),
> adding 's' shortcut, but cherry-picking it seems less obvious since the multiarch
> menus weren't reworked in jessie (comments added on the right).
>                 $one->("... Install with speech synthesis", Speach => 1);
>                 $one->("... Install with speech synthesis", Graphical => 1, Speach => 1);               # 32
>                 $one->("... 64 bit speech install", SixtyFour => 1, Graphical => 1, Speach => 1)        # 64
>                     if $sixtyfour;
> 
> instead of:
>                 $one->("... Install with speech synthesis", Graphical => 1, Speach => 1);               # hotkey gets added here only
>                 $one->("... 32 bit speech install", ThirtyTwo => 1, Graphical => 1, Speach => 1)
>                     if $thirtytwo;
> 
> I suspect we could add the shortcut on the 32 line and not touch the 64 line,
> given 32-bit is the default with that image in jessie (as opposed to 64-bit in
> stretch), but I'd appreciate a confirmation for both points:
>  - this change is welcome on the accessibility side (→ Samuel);
>  - my 32/64 logic is not off (→ Didier).

Based on Samuel's answer, I've pushed two commits to the pu/jessie
branch.

Double-checking/testing is more than welcome.

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature


Reply to: