Bug 280401 - www/freenginx-devel: Invalid root path
Summary: www/freenginx-devel: Invalid root path
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jochen Neumeister
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-07-22 08:23 UTC by Vedran Miletic
Modified: 2024-10-14 09:39 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (joneum)


Attachments
Patch (3.09 KB, patch)
2024-09-03 12:04 UTC, Herbert J. Skuhra
no flags Details | Diff
Previous patch plus update to version 1.27.4 (3.90 KB, patch)
2024-09-03 12:44 UTC, Herbert J. Skuhra
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vedran Miletic 2024-07-22 08:23:03 UTC
After installation of www/freenginx-devel, the file /usr/local/etc/freenginx/nginx.conf has two root directives

root   /usr/local/www/nginx
root   /usr/local/www/nginx-dist

Directory /usr/local/www contains symlink named freenginx that links to /usr/local/www/freenginx-dist, but only nginx-dist exists in the same directory.

It would be great if nginx-dist can be renamed to freenginx-dist and the default config file paths adjusted accordingly (freenginx instead of nginx and freenginx-dist instead of nginx-dist).
Comment 1 Vedran Miletic 2024-07-25 05:08:17 UTC
FWIW, www/freenginx is not affected by this; the mentioned files are named nginx in it.
Comment 2 Herbert J. Skuhra 2024-09-03 12:04:20 UTC
Created attachment 253294 [details]
Patch

Can you try the attached patch?
Comment 3 Herbert J. Skuhra 2024-09-03 12:44:42 UTC
Created attachment 253295 [details]
Previous patch plus update to version 1.27.4

The 2nd patch additionally updates www/freenginx-devel to version 1.27.4.

https://freenginx.org/en/CHANGES
Comment 4 Herbert J. Skuhra 2024-10-11 07:10:06 UTC
Ping.
Comment 5 Vedran Miletic 2024-10-14 09:39:27 UTC
(In reply to Herbert J. Skuhra from comment #4)
Apologies for the delay, I missed the previous notification. I will try it this week.