[Kimchi-devel] [PATCH][Wok] Issue #166: wok is pointing to /etc/nginx/conf.d which does not exists on OpenSuse 42.1

[Ramon Medeiros]

On 09/22/2016 09:00 AM, Aline Manera wrote:
> Hi Ramon,
>
> On 09/21/2016 02:58 PM, Ramon Medeiros wrote:
>> Create dir if not exist
>
> Is it the recommendation from nginx?
> I mean, this directory should be installed by nginx and if it is not 
> maybe there is a better directory (already installed by nginx) to 
> place configuration files.
>
> Also, doing that, the system will keep the directory on package removal.
Nop,

just remove the rpm and the file is still there
>
>>
>> Signed-off-by: Ramon Medeiros <ramonn at linux.vnet.ibm.com>
>> ---
>>   src/wok/proxy.py | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/src/wok/proxy.py b/src/wok/proxy.py
>> index b68b86e..4b06fc2 100644
>> --- a/src/wok/proxy.py
>> +++ b/src/wok/proxy.py
>> @@ -116,6 +116,11 @@ def _create_proxy_config(options):
>>       # SSL certifications.
>>       if not paths.installed:
>>           dst = os.path.join(paths.sys_nginx_conf_dir, "wok.conf")
>> +
>> +        # directoy does not exist: create it
>> +        if not os.path.exists(paths.sys_nginx_conf_dir):
>> +            os.makedirs(paths.sys_nginx_conf_dir)
>> +
>>           if os.path.isfile(dst) or os.path.islink(dst):
>>               os.remove(dst)
>>           os.symlink(os.path.join(nginx_config_dir, "wok.conf"), dst)
>

-- 

Ramon Nunes Medeiros
Kimchi Developer
Linux Technology Center Brazil
IBM Systems & Technology Group
Phone : +55 19 2132 7878
ramonn at br.ibm.com