Bug 212591 - [patch] www/awstats: fix path in awstats_updateall.pl
Summary: [patch] www/awstats: fix path in awstats_updateall.pl
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Adam Weinberger
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-09-11 21:31 UTC by Vidar Karlsen
Modified: 2016-09-12 18:23 UTC (History)
1 user (show)

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


Attachments
Small fix (321 bytes, text/x-perl-script)
2016-09-11 21:31 UTC, Vidar Karlsen
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vidar Karlsen 2016-09-11 21:31:38 UTC
Created attachment 174667 [details]
Small fix

The awstats_updateall.pl script looks for the awstats config files in /etc.
I have attached a patch to make it look in /usr/local/etc instead.
Comment 1 Adam Weinberger freebsd_committer 2016-09-11 22:55:07 UTC
Hardcoding /usr/local is wrong. It needs to be a REINPLACE_CMD (sed) to ${LOCALBASE}.
Comment 2 commit-hook freebsd_committer 2016-09-12 18:22:07 UTC
A commit references this bug:

Author: adamw
Date: Mon Sep 12 18:21:47 UTC 2016
New revision: 421948
URL: https://svnweb.freebsd.org/changeset/ports/421948

Log:
  Fix path to config file location in an ancillary script,
  bump PORTREVISION.

  PR:		212591
  Submitted by:	Vidar Karlsen

Changes:
  head/www/awstats/Makefile
Comment 3 Adam Weinberger freebsd_committer 2016-09-12 18:23:50 UTC
Committed, thanks Vidar. I only recently took over this port, so if you find any other places where I missed the correct path please file another PR!