<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 18, 2018 at 3:40 PM, Willy Sudiarto Raharjo <span dir="ltr"><<a href="mailto:willysr@slackbuilds.org" target="_blank">willysr@slackbuilds.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">>     See my changes here<br>
>     <a href="https://git.slackbuilds.org/slackbuilds/commit/?h=letsencrypt-review&id=9abf7d8542cc0237903252bbc34b8f0ee8015e0a" rel="noreferrer" target="_blank">https://git.slackbuilds.org/<wbr>slackbuilds/commit/?h=<wbr>letsencrypt-review&id=<wbr>9abf7d8542cc0237903252bbc34b8f<wbr>0ee8015e0a</a><br>
><br>
> ​I already saw those.  But what I mean is that the patch is not<br>
> necessary.  Since the constants.py file no longer contains apache2ctl<br>
> references, you don't need the new patch file and can just change the<br>
> sed line to point to configurator.py instead of constants.py, right?​<br>
<br>
</span>There are other paths needs to be changed as well because the default<br>
location refers to Debian/Ubuntu.<br>
<br>
I tested new cert request and it's still working fine here</blockquote><div><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">I see.  The /apache2 ​/httpd modification is already done in the slackbuild as well in the for loop containing the sed command for that.  The only changes to configurator.py not handled by changing the sed target are:</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><div class="gmail_default">vhost_files="httpd-vhosts.conf",</div><div class="gmail_default"><br></div><div class="gmail_default">le_vhost_ext="-le-ssl.conf",</div><div><br></div><div>The le_vhost_ext is of no consequence because I can specify that on the command line.</div><div><br></div><div>But the "vhost_files" one breaks things for me.  I've been maintaining web servers since the '90s and like almost every other web admin I've ever met, i maintain vhosts configs in separate files per vhost.  Since certbot doesn't provide a method for changing this option via the command line, this forces everyone to use the example vhost file as the only place to put vhosted domains they wish to be renewed by certbot in that one file.</div><div><br></div></div><div class="gmail_default" style="font-family:tahoma,sans-serif">Honestly, I think just changing the one sed line to configurator.py and removing the patch would be the best way to go.  Not only would it not break things for anybody, but it would also prevent you from having to maintain a patch file that will likely change more often than the filename where the apache2ctl string needs to be changed.  Even then, if you really wanted to and since this build is only for slackware, you could encapsulate the apache2ctl/apachectl sed line in its own for loop like the apache2/httpd loop and then you never have to worry about it.​</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><div class="gmail_default">for i in $(grep -ri apache2ctl * | cut -d: -f1 | sort -u)</div><div class="gmail_default">do</div><div class="gmail_default">  sed -i 's/apache2ctl/apachectl/' $i</div><div class="gmail_default">done</div></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div></div></div></div>