<div dir="ltr">i haven't check the 40x and 50x pages but are you saying the default pages w/o changing the included nginx.conf file? i would agree that this is a somewhat trivial change that doesn't affect the binary or the original source. another option is to correct the shipped nginx.conf to not reference the nonexistent files.<div>
<br></div><div>it would make sense that a clean build/install of the package should be fully functional. for example, chown'ing log files so the pre-configured runas user can actually write to them instead of leaving it as an undocumented step.</div>
<div><br></div><div>-m</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Apr 12, 2014 at 3:40 PM, Doogster <span dir="ltr"><<a href="mailto:thedoogster@gmail.com" target="_blank">thedoogster@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Removing the 40x and 50x error pages is a problem with the SlackBuild,<br>
and it should be fixed.<br>
<br>
The rest of what my patch does is a matter of taste.<br>
<div class="HOEnZb"><div class="h5"><br>
On Sat, Apr 12, 2014 at 2:53 PM, Klaatu <<a href="mailto:klaatu@straightedgelinux.com">klaatu@straightedgelinux.com</a>> wrote:<br>
> I hear what you're saying, Doogster, but is that really an issue with<br>
> the Slackbuild? It looks like that is the way Nginx distributes itself,<br>
> and Slack{ware,Builds.org} policy, I believe, is to not change the<br>
> upstream package unless not doing so will result in the thing literally<br>
> not launching or being functional on Slackware.<br>
><br>
> I realise that you may be making the argument that it is not functional<br>
> without being changed, and I can see what you mean. However, I use Nginx<br>
> on a few different servers, and on one I do change it to match the<br>
> htdocs webroot, but on others I change it to something else entirely.<br>
><br>
> Bottom line is, if the SBo maintainer patches the config file for me,<br>
> I'm just going to have to unpatch it later to suit however I have it set<br>
> up on any given box.<br>
><br>
> This is one of the things that drives me nuts about Debian and Red Hat;<br>
> they change the defaults from something I'm going to have to change, to<br>
> something else I'm going to have to change...<br>
><br>
><br>
> tl;dr<br>
> Not flaming you or disrespecting your opinion, just wondering if it's<br>
> really correct to patch the config from the point of install, or whether<br>
> a simple note in the README to remind the user "hey! you need to<br>
> configure this!") and maybe even a link to your blogpost so that new sys<br>
> admins might have some sensible example of what a good config would look<br>
> like.<br>
><br>
> -klaatu<br>
><br>
><br>
><br>
> On 04/13/2014 05:28 AM, Doogster wrote:<br>
>> Hey Larry. There's a pretty serious problem with the current SBo nginx<br>
>> SlackBuild: it removes the 40x and 50x error pages that are supposed<br>
>> to be included. I hope that as the new maintainer, you can address<br>
>> that.<br>
>><br>
>> My LEMP Howto (<a href="http://duganchen.ca/lemp-on-slackware/" target="_blank">http://duganchen.ca/lemp-on-slackware/</a>) has a patch<br>
>> that might give you some ideas.<br>
>><br>
>> On Sat, Apr 12, 2014 at 10:07 AM, Willy Sudiarto Raharjo<br>
>> <<a href="mailto:willysr@slackbuilds.org">willysr@slackbuilds.org</a>> wrote:<br>
>>>>> I'm interested.... Maintainer is sometimes slow to respond... Might want to<br>
>>>>> wait a few days though.<br>
>><br>
>> It's been more than a week :)<br>
>> Anyway, Larry Hajali was the first to respond to become a new maintainer<br>
>><br>
>><br>
>>><br>
>>> _______________________________________________<br>
>>> SlackBuilds-users mailing list<br>
>>> <a href="mailto:SlackBuilds-users@slackbuilds.org">SlackBuilds-users@slackbuilds.org</a><br>
>>> <a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
>>> Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
>>> FAQ - <a href="http://slackbuilds.org/faq/" target="_blank">http://slackbuilds.org/faq/</a><br>
>>><br>
>> _______________________________________________<br>
>> SlackBuilds-users mailing list<br>
>> <a href="mailto:SlackBuilds-users@slackbuilds.org">SlackBuilds-users@slackbuilds.org</a><br>
>> <a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
>> Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
>> FAQ - <a href="http://slackbuilds.org/faq/" target="_blank">http://slackbuilds.org/faq/</a><br>
>><br>
>><br>
>><br>
><br>
> _______________________________________________<br>
> SlackBuilds-users mailing list<br>
> <a href="mailto:SlackBuilds-users@slackbuilds.org">SlackBuilds-users@slackbuilds.org</a><br>
> <a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
> Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
> FAQ - <a href="http://slackbuilds.org/faq/" target="_blank">http://slackbuilds.org/faq/</a><br>
><br>
_______________________________________________<br>
SlackBuilds-users mailing list<br>
<a href="mailto:SlackBuilds-users@slackbuilds.org">SlackBuilds-users@slackbuilds.org</a><br>
<a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
FAQ - <a href="http://slackbuilds.org/faq/" target="_blank">http://slackbuilds.org/faq/</a><br>
<br>
</div></div></blockquote></div><br></div>