public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
From: Till Kamppeter <till.kamppeter@gmail.com>
To: Bastien Nocera <hadess@hadess.net>
Cc: linux-bluetooth@vger.kernel.org, Tim Waugh <twaugh@redhat.com>
Subject: Re: PATCHES: Device discovery of the "bluetooth" CUPS backend does not work
Date: Tue, 11 Aug 2009 14:24:32 +0200	[thread overview]
Message-ID: <4A816300.7050906@gmail.com> (raw)
In-Reply-To: <1249990438.2022.23.camel@localhost.localdomain>

Bastien Nocera wrote:
> You're modifying a file in the Ubuntu package (it's in the debian/
> sub-directory), not the one from bluez (src/bluetooth.conf in the
> sources).
>

For the Ubuntu packaging I did the change in the Ubuntu-provided version 
of the file as the Ubuntu (Debian?) package maintainer did other changes 
there. Putting my change in the original file then would have resulted 
in them being overwritten. In the upstream fix I naturally expect the 
change to be applied to src/bluetooth.conf.

>> A Bluetooth printer has to get paired with a computer before the 
>> Bluetooth CUPS backend can find it.
> 
> I know that, seeing as I wrote the discovery code in the bluetooth
> backend :)
>

OK, I mentioned that only to tell why the Bluetooth printers should be 
considered local.

>>  This makes it getting assigned to 
>> one single computer, whereas a network printer (Ethernet or WLAN) gets 
>> discovered by all computers in the network and all computers can have a 
>> working queue accessing to the same network printer. Therefore a 
>> Bluetooth printer has more the characteristics of a local printer.
> 
> Seems fair.
> 

OK.

> 
> You're right, read the patch in reverse...
>

OK.

>>> In the future, please attach the patches to your mail, and make sure
>>> they apply to the upstream sources.
>> Sorry, was my first patch submission to this project.
> 
> Could you rebase your fixes into patches ready for inclusion in the git
> tree? If you don't know git, I'll make those changes myself.
> 

Could you put them in? Then I do not need to download the git only to 
format two simple one-point-only-changed patches. Thanks.

>> Do you have a bug/feature request tracking system?
> 
> There isn't one.

As vluez seems to be a part of the kernel project, does the kernel even 
not have a bug tracking system?

    Till

  reply	other threads:[~2009-08-11 12:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-10 20:19 PATCHES: Device discovery of the "bluetooth" CUPS backend does not work Till Kamppeter
2009-08-11 10:33 ` Bastien Nocera
2009-08-11 11:19   ` Till Kamppeter
2009-08-11 11:33     ` Bastien Nocera
2009-08-11 12:24       ` Till Kamppeter [this message]
2009-08-11 13:12         ` Bastien Nocera
2009-08-11 22:09           ` Marcel Holtmann
2009-08-11 22:14           ` Marcel Holtmann
  -- strict thread matches above, loose matches on Subject: below --
2009-08-26 17:59 Till Kamppeter
2009-08-26 18:14 ` Marcel Holtmann
2009-08-26 19:12   ` Till Kamppeter
2009-08-27  2:35     ` Marcel Holtmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A816300.7050906@gmail.com \
    --to=till.kamppeter@gmail.com \
    --cc=hadess@hadess.net \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=twaugh@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox