GNOME Bugzilla – Bug 625076
neonhttpsrc: add connect-timeout and read-timeout properties
Last modified: 2010-08-30 13:21:54 UTC
Created attachment 166418 [details] [review] Adds connect and read timeout properties The attached patch adds connect and read timeout properties to neonhttpsrc.
> --- gst/neon/gstneonhttpsrc.c (revision 766) > +++ gst/neon/gstneonhttpsrc.c (working copy) Seriously? I have to adjust paths as well now? David, is it really so hard to set up git in windows that you can't submit patches in proper git format-patch format like everyone else? If you use SVN or CVS, that's up to you of course, but you can easily copy the patches over and re-committed them locally in a git branch and then use git-format... commit f724428e073df3c3fc7c50407efe365233cdc030 Author: David Hoyt <dhoyt@llnl.gov> Date: Fri Aug 6 11:04:57 2010 +0100 neonhttpsrc: add connect-timeout and read-timeout properties https://bugzilla.gnome.org/show_bug.cgi?id=625076
Comment on attachment 166418 [details] [review] Adds connect and read timeout properties Committed, grudgingly.
Sorry about the paths - that was unintentional. As for using git -- I haven't had the chance to learn it yet with all the other things on my to-do list. Once I learn it and can parse what you said, I'd be happy to do as requested. Again, I apologize for the inconvenience. If you prefer, you can leave the patches be until I get the opportunity to learn git and put them in the right format. I just don't want to forget about them since they may still be beneficial to the community in some small sense.
This change causes: (gst-plugin-scanner:4906): GStreamer-WARNING **: Failed to load plugin '/usr/local/lib/gstreamer-0.10/libgstneonhttpsrc.so': /usr/local/lib/gstreamer-0.10/libgstneonhttpsrc.so: undefined symbol: ne_set_connect_timeout Either this needs to be reversed or we need to bump the required neon version. I appear to have 0.26.4-4 (ubuntu).
ne_set_connect_timeout() was added in libneon 0.27, it seems, which is available in Lucid as neon27. I propose we just bump the requirement, since that was released in 2007.
Created attachment 168859 [details] [review] configure: Bump neon requirement to 0.27.0 Needed in order to have ne_set_connect_timeout Fixes bug #625076
commit dab76bc68ef49d0a4c77809f049d90def8aee083 Author: Jan Schmidt <thaytan@noraisin.net> Date: Fri Aug 27 15:49:39 2010 +1000 configure: Bump neon requirement to 0.27.0 Needed in order to have ne_set_connect_timeout Fixes bug #625076