Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-19 Thread Marcel Holtmann
Hi Dan, Plus, as I stated earlier, libxml will be required to handle Wispr responses (and hotspot 2.0), which are legitimate XML trees. So, why not just add it as a dependency right now? that is a funny comment since within ConnMan, we are using GMarkup to handle the WISPr login.

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-19 Thread Jonh Wendell
One note: The original patch was supposed to handle 511 http status code, not wispr. see http://tools.ietf.org/html/rfc6585 2013/2/19 Marcel Holtmann mar...@holtmann.org Hi Dan, Plus, as I stated earlier, libxml will be required to handle Wispr responses (and hotspot 2.0), which are

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-19 Thread Marcel Holtmann
Hi Jonh, One note: The original patch was supposed to handle 511 http status code, not wispr. see http://tools.ietf.org/html/rfc6585 I do not remember any portal using 511 status. At least the ones we tested have not used it. However we have seen hotspots that are using 200 success status code

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-18 Thread Marcel Holtmann
Hi Jonh, Plus, as I stated earlier, libxml will be required to handle Wispr responses (and hotspot 2.0), which are legitimate XML trees. So, why not just add it as a dependency right now? that is a funny comment since within ConnMan, we are using GMarkup to handle the WISPr login. And

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-18 Thread Dan Williams
On Tue, 2013-02-19 at 01:45 +0100, Marcel Holtmann wrote: Hi Jonh, Plus, as I stated earlier, libxml will be required to handle Wispr responses (and hotspot 2.0), which are legitimate XML trees. So, why not just add it as a dependency right now? that is a funny comment since within

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-13 Thread Jonh Wendell
Hi Dan. 2013/2/11 Dan Williams d...@redhat.com On Mon, 2013-02-11 at 15:30 -0200, Jonh Wendell wrote: In these patches I want to fix the 511-http-status. As it's something new, of course most hotspots don't use that (including my employer). Almost all of them rely on 30X Moved with help

[PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Jonh Wendell
From: Jonh Wendell jonh.wend...@oiwifi.com.br libsoup already depends on libxml2 but we need to explicitly link to it. Signed-off-by: Jonh Wendell jonh.wend...@oiwifi.com.br --- configure.ac| 14 +++--- src/Makefile.am | 4 ++-- 2 files changed, 13 insertions(+), 5 deletions(-)

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Dan Williams
On Mon, 2013-02-11 at 12:09 -0200, Jonh Wendell wrote: From: Jonh Wendell jonh.wend...@oiwifi.com.br libsoup already depends on libxml2 but we need to explicitly link to it. At least we already theoretically required it; though is it possible to use GMarkup here instead of libxml2? GMarkup

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Dan Williams
On Mon, 2013-02-11 at 17:10 +0100, Bastien Nocera wrote: On Mon, 2013-02-11 at 10:06 -0600, Dan Williams wrote: On Mon, 2013-02-11 at 12:09 -0200, Jonh Wendell wrote: From: Jonh Wendell jonh.wend...@oiwifi.com.br libsoup already depends on libxml2 but we need to explicitly link to

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Bastien Nocera
On Mon, 2013-02-11 at 10:06 -0600, Dan Williams wrote: On Mon, 2013-02-11 at 12:09 -0200, Jonh Wendell wrote: From: Jonh Wendell jonh.wend...@oiwifi.com.br libsoup already depends on libxml2 but we need to explicitly link to it. At least we already theoretically required it; though is

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Jonh Wendell
In these patches I want to fix the 511-http-status. As it's something new, of course most hotspots don't use that (including my employer). Almost all of them rely on 30X Moved with help of the Wispr 'pseudo-protocol'. It's on my TODO list to work on those scenarios. That would touch only code in

Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency

2013-02-11 Thread Dan Williams
On Mon, 2013-02-11 at 15:30 -0200, Jonh Wendell wrote: In these patches I want to fix the 511-http-status. As it's something new, of course most hotspots don't use that (including my employer). Almost all of them rely on 30X Moved with help of the Wispr 'pseudo-protocol'. It's on my TODO