Re: [PATCH] Add udev rules to start bluetooth on-demand

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

 



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


--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux