On Fri, Jul 08, 2011 at 11:23:43AM +0800, Zheng, Wu wrote:
> Hi Ven:
>
> For it, ${BOARDNAME_REVISION}_bluetooth or ${BOARDNAME}_bluetooth is not
> BINARIES, it only is a shell script which is for config the file of
> /etc/Bluetooth/audio.conf for different devices.
>
In that case, why not just make one script (/usr/bin/bluez-preconfig.sh
or something) that does the "Right Thing" (tm) internally, includin
parsing boardname? It would be much more sane than embedding it in
rc.sysinit.
> Bluez only use the same file for different devices, bluez can not select
> config file and it only use the same file like /etc/Bluetooth/audio.conf
> for config.
Then why not create and submit a patch to the MeeGo package for bluez to
fix this properly? Then, submit it upstream to see about getting it (or
any form of smart start up config file detection) merged. I see no reason
why bluez can't have/utilize a configs dir model similar to rc.d or
xorg.conf.d (for example: /etc/Bluetooth/conf.d/*.conf)
> And for different devices, the config files of bluez maybe are different.
This is not a new problem... just remove it from rc.sysinit to be clean
and self contained.
Shane...
>
> > -----Original Message-----
> > From: Arjan van de Ven [mailto:[email protected]]
> > Sent: Thursday, July 07, 2011 9:47 PM
> >
> > On 7/7/2011 12:43 AM, zhengwu wrote:
> > > ++++++ check-bluetoothd-config.patch (new)
> > > --- check-bluetoothd-config.patch
> > > +++ check-bluetoothd-config.patch
> > > +--- old/rc.d/rc.sysinit 2011-07-05 11:02:17.472666472 +0800
> > > ++++ new/rc.d/rc.sysinit 2011-07-05 13:28:36.905728247 +0800
> > > +@@ -327,6 +327,15 @@
> > > + fi
> > > +
> > > + if [ -x "/usr/sbin/bluetoothd" ] ; then
> > > ++ if [ -e /etc/boardname ]; then
> > > ++ BOARDNAME_REVISION=`cat /etc/boardname`
> > > ++ BOARDNAME=`echo $BOARDNAME_REVISION | sed
> > > 's!_[a-zA-Z0-
> > 9]*$!!g'`
> > > ++ if [ -x /usr/bin/$BOARDNAME_REVISION_bluetooth ];
> > > then
> > > ++ /usr/bin/${BOARDNAME_REVISION}_bluetooth
> > > ++ elif [ -x /usr/bin/$BOARDNAME_bluetooth ]; then
> > > ++ /usr/bin/${BOARDNAME}_bluetooth
> > > ++ fi
> > > ++ fi
> > > + /usr/sbin/bluetoothd
> > > + fi
> > >
> >
> >
> > ewwwww?
> >
> > we have different bluetooth BINARIES for different boards?
> > seriously?
_______________________________________________
MeeGo-packaging mailing list
[email protected]
http://lists.meego.com/listinfo/meego-packaging
