Addin a symlink for CentOS #6
Conversation
As defined in an issue https://github.com/Islandora-Devops/ansible-role-crayfish/issues/5 to install in a CentOS defined additional config dir for Apache
There was a problem hiding this comment.
This works, but I think we can make it cleaner. Instead of creating a separate task for CentOS, we can change the existing symlink task to use a variable (e.g. httpd_conf_directory_enabled) and set it in their respective vars configuration files (Debian.yml and RedHat.yml).
E.g. RedHat.yml would have the line:
httpd_conf_directory_enabled: "{{httpd_conf_directory}}/conf.d"
and the existing symlink task's dest line would read:
dest: "{{httpd_conf_directory_enabled}}/{{item}}.conf"
|
@seth-shaw-unlv if there is a cleaner solution it would be good. I mention it in #3 (comment) . Can it be done in this PR or to create a new? |
|
I'll leave it up to you @DigitLib, you can either update your PR, or we can close it and I can issue a new PR based on my fork testing the new solution. |
|
@seth-shaw-unlv great fork! :) I will close my PR |
|
Closed because @seth-shaw-unlv made a cleaner solution. |
* centos fixes install tesseract and imagemagick Use variables for apache config dirs * removing debugging task * Replace leptonica source tgz with git clone The previous source .tgz disappeared when a new version appeared, but the github version sticks around. Replacing the gzipped source sith github clone for stability. * removed tesseract-ocr install * symlink httpd conf for centos See review of /pull/6
As defined in an issue https://github.com/Islandora-Devops/ansible-role-crayfish/issues/5 to install in a CentOS defined additional config dir for Apache.
@dannylamb PR created