On 11/20/2010 02:16 PM, David Woodhouse wrote:
This one probably isn't ready to apply as-is, but it's working for me right now. Problems include:
- it should have proper CMake magic to find and build against (and optionally *not* build against) libproxy.
- it probably shouldn't unconditionally subvert the old code which looks in /etc/sysconfig/proxy for per-url-scheme information.
- There's a memory leak when we return a proxy specified by libproxy.
David, thanks a lot for the patches I have applied the first one locally as is. It only changes the API to do it more powerful, but the behavior is basically unchanged. (and thanks a lot for separating the patches in that logical way) For the second one, I added a real check for libproxy, and compiled the files conditionally. If libproxy is found, then it is used, otherwise the default Sysconfig one is used. May be we should remove the Sysconfig one completely then. However once I build, the testcases fail. The following tests FAILED: 12 - Fetcher_test (Failed) 17 - MediaSetAccess_test (Failed) 24 - RepoInfo_test (Failed) 48 - MirrorList_test (Failed) Errors while running CTest -- Duncan Mac-Vicar P. - Novell® Making IT Work As One™ SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg)