On 09/27/2016 03:52 PM, Aline Manera wrote:

On 09/26/2016 01:17 PM, Ramon Medeiros wrote:

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

And we don't want it, right?

Well, at Fedora 24, removing the package, the conf.d directory persists:

ramonn@jarvis:~$ sudo rpm -e nginx
[sudo] password for ramonn:
ramonn@jarvis:~$ ls -l /etc/nginx/
conf.d/ default.d/ mime.types mime.types.rpmnew
ramonn@jarvis:~$ ls -l /etc/nginx/default.d/
total 0
ramonn@jarvis:~$ ls -l /etc/nginx/conf.d/
total 0
lrwxrwxrwx. 1 root root 44 Sep 28 10:37 wok.conf -> /home/ramonn/GitTrees/WOK/src/nginx/wok.conf

Signed-off-by: Ramon Medeiros <ramonn@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@br.ibm.com