Re: [PATCH v2 11/11] usbip: USB over WebSocket

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

 



If you are looking for a good C websockets library, gpl etc try:

https://libwebsockets.org/trac/libwebsockets


I used it in a project and was very pleased with how it worked and the
support where it is hosted.


Regards, Steve

On Wed, Apr 22, 2015 at 11:52 PM, fx IWATA NOBUO
<Nobuo.Iwata@xxxxxxxxxxxxxxx> wrote:
> Hi Bjørn
>
>> This cannot possibly be necessary.  Pointing to the toplevel "COPYING"
>> file should be more than enough, if anything at all.
>>
>> And BTW, the same goes for the tools/usb/usbip/COPYING that seems to
>> have snuck in somehow.  I also wonder a bit about the AUTHORS file
> both
>> places.  We have git and MAINTAINERS for such things, don't we?
>
> I will remove from v3.
>
>> > --- /dev/null
>> > +++ b/tools/usb/usbip/websocket/configure.ac
>> > @@ -0,0 +1,61 @@
>> > +dnl Process this file with autoconf to produce a configure script.
>> > +
>> > +AC_PREREQ(2.59)
>> > +AC_INIT([usbip-utils], [1.1.1], [linux-usb@xxxxxxxxxxxxxxx])
>> > +AC_DEFINE([USBIP_VERSION], [0x00000111], [binary-coded decimal
> version
>> number])
>>
>> This should either use the upper level definitions or have its own
>> namespace.  Defining the same symbols at different levels is a recipe
>> for confusion.  In fact, you are already out of sync with the
>> definitions in tools/usb/usbip/configure.ac
>
> Sorry for my carelessness.
> I will use own AC_INIT() in v3 and remove AC_DEFINE([USBIP_VERSION])
> from v3.
>
>> > Implementation of this patch depends on Poco C++
>> > (http://pocoproject.org/).
>>
>> Really?  Is that OK?
>>
>> Yuck. Any reason this cannot be written in C?
>
> I couldn't find good C library which supports both server and client.
> I intended other implementation may be stored under 'websocket'
> directory next to 'poco'.
>
> Thank you for your comments,
>
> n.iwata
> //
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux