On Wed, Aug 31, 2011 at 02:12:19PM +0200, Stefan Lippers-Hollmann wrote:
> On Wednesday 31 August 2011, Hermann Lauer wrote:
> [...]
> > auto device detection is not working any more in squeeze, code looks bad.
with auto device detection I mean DEVICE="", as you can see from my
/etc/lirc/hardware.conf contained in the bugreport.
> I assume you use something like /dev/ttyS1 instead, which would be
> wrong.
no, see report.
> > The appended patch (to an older version, probably lenny) restores
> > that functionality.
> [...]
if [ -z "$DEVICE" ]; then
- if [ -c $dev ]; then
- DEVICE="$dev"
- break
- fi
+ for dev in /dev/lirc0 /dev/lirc; do
+ if [ -c $dev ]; then
+ DEVICE="$dev"
+ break
+ fi
+ done
fi
As you can see from the patch (relevant code shown above),
the whole part is only for the DEVICE="" case.
The lines in squeeze (marked with -) are not setting
the "dev" variable in the shell code, so it's broken code.
The "+" lines shows the original working code (which sets the "dev"
variable, so the test makes sense) - is known, for what reason
(and when) that was changed ?
>
> The initscript itself is the next topic on the todo list for lirc, as
> especially the new in-kernel RC_CORE subsystem needs further changes.
> However these will have to go, and get tested in-, unstable first,
> right now I'm not sure how much of that can be backported to squeeze
> though.
People which set DEVICE in their hardware.conf should not see any effect from
this patch.
Thanks for caring,
greetings
Hermann
--
Netzwerkadministration/Zentrale Dienste, Interdiziplinaeres
Zentrum fuer wissenschaftliches Rechnen der Universitaet Heidelberg
IWR; INF 368; 69120 Heidelberg; Tel: (06221)54-8236 Fax: -5224
Email: hermann.la...@iwr.uni-heidelberg.de
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
