Re: [PATCH v2 1/5] A new object 'NMProxyConfig' with fields for proxy.

On Tue, Jul 12, 2016 at 05:57:16PM +0530, Atul Anand wrote:
A new object NMProxyConfig has been implemented which contain fields
for proxy related stuff.

Could you please prefix each commit subject with the component or area
affected by the change as:

  core: ...


  proxy: ...

Would it be possible to change the order of commits so that the
libnm-core patch comes before (as this patch depends on it)?

+nm_proxy_config_merge_setting (NMProxyConfig *config, NMSettingProxy *setting)
+     priv->proxies = NULL;
+     priv->excludes = NULL;
+     priv->pac_url = NULL;
+     priv->pac_script = NULL;

It seems this clears all existing parameters, including the pac_url
previously set (for example from DHCP), is it correct?


Attachment: signature.asc
Description: PGP signature

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