As discussed in previous mails.
Cheers
Hi Bastien,
> > > > > > > > > > > As discussed in previous mails.
> > > > > > > > > >
> > > > > > > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > > > > > > not bluezd.
> > > > > > > > > >
> > > > > > > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > > > > > > added to the distribution automatically. For all this changes, you have
> > > > > > > > > > to check with make distcheck.
> > > > > > > > >
> > > > > > > > > Fixed version attached
> > > > > > > >
> > > > > > > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > > > > > > sense to install the init script and the udev rule at the same time.
> > > > > > >
> > > > > > > Revised patch attached
> > > > > >
> > > > > > +
> > > > > > +if INISCRIPTS
> > > > > > +else
> > > > > > +rules_DATA += bluetooth.rules
> > > > > > endif
> > > > >
> > > > > ifndef isn't an automake construct.
> > > > >
> > > >
> > > > then if !INITSCRIPTS should do it. However I was more asking about the
> > > > spelling mistake. You forgot a T ;)
> > >
> > > Well, that's embarrassing...
> >
> > are you sending a new patch :)
>
> Thought you'd fix the typo, and commit.
not if you wanna keep your author name on the patch.
Also does if !INITSCRIPTS not work?
And why does your patch now has two statements for rules_DATA?
Regards
Marcel
On Sat, 2009-06-27 at 12:27 +0200, Marcel Holtmann wrote:
> Hi Bastien,
>
> > > > > > > > > > As discussed in previous mails.
> > > > > > > > >
> > > > > > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > > > > > not bluezd.
> > > > > > > > >
> > > > > > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > > > > > added to the distribution automatically. For all this changes, you have
> > > > > > > > > to check with make distcheck.
> > > > > > > >
> > > > > > > > Fixed version attached
> > > > > > >
> > > > > > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > > > > > sense to install the init script and the udev rule at the same time.
> > > > > >
> > > > > > Revised patch attached
> > > > >
> > > > > +
> > > > > +if INISCRIPTS
> > > > > +else
> > > > > +rules_DATA += bluetooth.rules
> > > > > endif
> > > >
> > > > ifndef isn't an automake construct.
> > > >
> > >
> > > then if !INITSCRIPTS should do it. However I was more asking about the
> > > spelling mistake. You forgot a T ;)
> >
> > Well, that's embarrassing...
>
> are you sending a new patch :)
Thought you'd fix the typo, and commit.
Patch attached.
Hi Bastien,
> > > > > > > > > As discussed in previous mails.
> > > > > > > >
> > > > > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > > > > not bluezd.
> > > > > > > >
> > > > > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > > > > added to the distribution automatically. For all this changes, you have
> > > > > > > > to check with make distcheck.
> > > > > > >
> > > > > > > Fixed version attached
> > > > > >
> > > > > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > > > > sense to install the init script and the udev rule at the same time.
> > > > >
> > > > > Revised patch attached
> > > >
> > > > +
> > > > +if INISCRIPTS
> > > > +else
> > > > +rules_DATA += bluetooth.rules
> > > > endif
> > >
> > > ifndef isn't an automake construct.
> > >
> >
> > then if !INITSCRIPTS should do it. However I was more asking about the
> > spelling mistake. You forgot a T ;)
>
> Well, that's embarrassing...
are you sending a new patch :)
Regards
Marcel
On Wed, 2009-06-24 at 18:46 +0200, Marcel Holtmann wrote:
> Hi Bastien,
>
> > > > > > > > As discussed in previous mails.
> > > > > > >
> > > > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > > > not bluezd.
> > > > > > >
> > > > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > > > added to the distribution automatically. For all this changes, you have
> > > > > > > to check with make distcheck.
> > > > > >
> > > > > > Fixed version attached
> > > > >
> > > > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > > > sense to install the init script and the udev rule at the same time.
> > > >
> > > > Revised patch attached
> > >
> > > +
> > > +if INISCRIPTS
> > > +else
> > > +rules_DATA += bluetooth.rules
> > > endif
> >
> > ifndef isn't an automake construct.
> >
>
> then if !INITSCRIPTS should do it. However I was more asking about the
> spelling mistake. You forgot a T ;)
Well, that's embarrassing...
Hi Bastien,
> > > > > > > As discussed in previous mails.
> > > > > >
> > > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > > not bluezd.
> > > > > >
> > > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > > added to the distribution automatically. For all this changes, you have
> > > > > > to check with make distcheck.
> > > > >
> > > > > Fixed version attached
> > > >
> > > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > > sense to install the init script and the udev rule at the same time.
> > >
> > > Revised patch attached
> >
> > +
> > +if INISCRIPTS
> > +else
> > +rules_DATA += bluetooth.rules
> > endif
>
> ifndef isn't an automake construct.
>
then if !INITSCRIPTS should do it. However I was more asking about the
spelling mistake. You forgot a T ;)
Regards
Marcel
On Wed, 2009-06-24 at 18:43 +0200, Marcel Holtmann wrote:
> On Wed, 2009-06-24 at 12:35 +0100, Bastien Nocera wrote:
> > On Sun, 2009-06-21 at 16:10 +0200, Marcel Holtmann wrote:
> > > Hi Bastien,
> > >
> > > > > > As discussed in previous mails.
> > > > >
> > > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > > not bluezd.
> > > > >
> > > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > > added to the distribution automatically. For all this changes, you have
> > > > > to check with make distcheck.
> > > >
> > > > Fixed version attached
> > >
> > > we might need to put this into ifndef INITSCRIPTS since it makes no
> > > sense to install the init script and the udev rule at the same time.
> >
> > Revised patch attached
>
> +
> +if INISCRIPTS
> +else
> +rules_DATA += bluetooth.rules
> endif
ifndef isn't an automake construct.
On Wed, 2009-06-24 at 12:35 +0100, Bastien Nocera wrote:
> On Sun, 2009-06-21 at 16:10 +0200, Marcel Holtmann wrote:
> > Hi Bastien,
> >
> > > > > As discussed in previous mails.
> > > >
> > > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > > not bluezd.
> > > >
> > > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > > added to the distribution automatically. For all this changes, you have
> > > > to check with make distcheck.
> > >
> > > Fixed version attached
> >
> > we might need to put this into ifndef INITSCRIPTS since it makes no
> > sense to install the init script and the udev rule at the same time.
>
> Revised patch attached
+
+if INISCRIPTS
+else
+rules_DATA += bluetooth.rules
endif
???
Regards
Marcel
On Sun, 2009-06-21 at 16:10 +0200, Marcel Holtmann wrote:
> Hi Bastien,
>
> > > > As discussed in previous mails.
> > >
> > > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > > not bluezd.
> > >
> > > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > > added to the distribution automatically. For all this changes, you have
> > > to check with make distcheck.
> >
> > Fixed version attached
>
> we might need to put this into ifndef INITSCRIPTS since it makes no
> sense to install the init script and the udev rule at the same time.
Revised patch attached
Hi Bastien,
> > > As discussed in previous mails.
> >
> > please name it bluetooth.rules. The daemon is also called bluetoothd and
> > not bluezd.
> >
> > Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> > added to the distribution automatically. For all this changes, you have
> > to check with make distcheck.
>
> Fixed version attached
we might need to put this into ifndef INITSCRIPTS since it makes no
sense to install the init script and the udev rule at the same time.
Regards
Marcel
On Sun, 2009-06-21 at 15:50 +0200, Marcel Holtmann wrote:
> Hi Bastien,
>
> > As discussed in previous mails.
>
> please name it bluetooth.rules. The daemon is also called bluetoothd and
> not bluezd.
>
> Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
> added to the distribution automatically. For all this changes, you have
> to check with make distcheck.
Fixed version attached
Hi Bastien,
> As discussed in previous mails.
please name it bluetooth.rules. The daemon is also called bluetoothd and
not bluezd.
Also you need to have the EXTRA_DIST part in it since _DATA doesn't get
added to the distribution automatically. For all this changes, you have
to check with make distcheck.
Regards
Marcel