FreeBSD Bugzilla – Attachment 181693 Details for
Bug 218351
net-mgmt/librenms: validate.php fails in fresh FreeBSD 11 Jail
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Patch V4
patch.diff (text/plain), 10.66 KB, created by
Jonathan Price
on 2017-04-11 18:58:22 UTC
(
hide
)
Description:
Patch V4
Filename:
MIME Type:
Creator:
Jonathan Price
Created:
2017-04-11 18:58:22 UTC
Size:
10.66 KB
patch
obsolete
>Index: Makefile >=================================================================== >--- Makefile (revision 437919) >+++ Makefile (working copy) >@@ -2,7 +2,7 @@ > # $FreeBSD$ > > PORTNAME= librenms >-PORTVERSION= 1.25 >+PORTVERSION= 1.26 > PORTEPOCH= 1 > CATEGORIES= net-mgmt > >@@ -15,16 +15,16 @@ > GH_ACCOUNT= librenms > > USES= php:cli python shebangfix >-SHEBANG_FILES= mibs/process.pl scripts/deploy-docs.sh scripts/ifAlias \ >+SHEBANG_FILES= mibs/process.pl scripts/deploy-docs.sh \ > scripts/Migration/Standard_Conversion/convert_no_xml.sh \ > scripts/Migration/Standard_Conversion/destwork_no_xml.sh \ > scripts/Migration/Standard_Conversion/mkdir.sh \ > scripts/Migration/XML_Conversion/convert.sh \ > scripts/Migration/XML_Conversion/destwork.sh \ >- scripts/Migration/XML_Conversion/mkdir.sh validate.php \ >+ scripts/Migration/XML_Conversion/mkdir.sh \ > scripts/removespikes.php scripts/agent-local/nfs-stats.sh \ > scripts/removespikes.pl scripts/watchmaillog/watchmaillog.pl >-USE_PHP= ctype filter gd json mcrypt mysqli session snmp tokenizer xml >+USE_PHP= ctype curl filter gd hash json mcrypt mysqli openssl session simplexml snmp tokenizer xml > > NO_BUILD= yes > >@@ -33,7 +33,8 @@ > ${LOCALBASE}/share/pear/Net/IPv4.php:net/pear-Net_IPv4 \ > ${LOCALBASE}/share/pear/Net/IPv6.php:net/pear-Net_IPv6 \ > snmpget:net-mgmt/net-snmp \ >- bash:shells/bash >+ bash:shells/bash \ >+ sudo:security/sudo > > OPTIONS_DEFINE= APACHEMOD FPING IPMITOOL LIBVIRT MYSQLD NAGPLUGINS NMAP PYPOLLER WMIC DOCS EXAMPLES X11 > OPTIONS_DEFAULT= APACHEMOD FPING IPMITOOL LIBVIRT NAGPLUGINS NMAP PYPOLLER WMIC >@@ -64,12 +65,13 @@ > > ROOT_FILES= addhost.php adduser.php alerts.php build-base.php build.sql \ > billing-calculate.php \ >- check-services.php config_to_json.php daily.php daily.sh \ >- delhost.php discovery.php dist-pollers.php irc.php \ >- poll-billing.php poller.php poller-service.py poller-wrapper.py renamehost.php \ >+ check-services.php composer.json config_to_json.php cronic daily.php daily.sh \ >+ delhost.php discovery-wrapper.py discovery.php dist-pollers.php irc.php \ >+ mkdocs.yml pbin.sh phpunit.xml poll-billing.php poller.php poller-service.py \ >+ poller-wrapper.py readmegen.yml renamehost.php services-wrapper.py \ > snmp-scan.php snmptrap.php syslog.php validate.php > >-ROOT_DIRS= LibreNMS contrib html includes lib mibs scripts sql-schema >+ROOT_DIRS= LibreNMS contrib doc html includes lib mibs misc scripts sql-schema tests vendor > > DOCS= AUTHORS.md CHANGELOG CONTRIBUTING.md LICENSE.txt README.md > PORTDOCS= * >@@ -102,6 +104,10 @@ > # Create empty rrd dir writable by www > @${MKDIR} ${STAGEDIR}/${WWWDIR}/rrd > @${CHMOD} 0775 ${STAGEDIR}/${WWWDIR}/rrd >+# Create empty logs dir >+ @${MKDIR} ${STAGEDIR}/${WWWDIR}/logs >+# Make daily.sh executable >+ @${CHMOD} 0755 ${STAGEDIR}/${WWWDIR}/daily.sh > # Files in scripts dir should be executable > @${FIND} ${STAGEDIR}/${WWWDIR}/scripts -type f -exec ${CHMOD} +x {} \; > # Ensure PHP and Python scripts are executable >@@ -130,6 +136,7 @@ > ${STAGEDIR}/${WWWDIR}/config.php.sample > @${ECHO} @sample ${WWWDIR}/config.php.sample >> ${TMPPLIST} > @${ECHO} @dir ${WWWDIR}/rrd >> ${TMPPLIST} >+ @${ECHO} @dir ${WWWDIR}/logs >> ${TMPPLIST} > @${ECHO} @dir ${WWWDIR}/lib/influxdb-php/vendor/guzzlehttp/guzzle/build >> ${TMPPLIST} > @${ECHO} @dir ${WWWDIR}/lib/influxdb-php/vendor/guzzlehttp/guzzle/docs >> ${TMPPLIST} > @${ECHO} @dir ${WWWDIR}/lib/influxdb-php/vendor/guzzlehttp/guzzle/tests >> ${TMPPLIST} >Index: distinfo >=================================================================== >--- distinfo (revision 437919) >+++ distinfo (working copy) >@@ -1,3 +1,3 @@ >-TIMESTAMP = 1489149658 >-SHA256 (librenms-librenms-1.25_GH0.tar.gz) = 7b966c668dd215dd35add4d96f5cb04b63e83956e9db32a962dfd9da2717827d >-SIZE (librenms-librenms-1.25_GH0.tar.gz) = 55302189 >+TIMESTAMP = 1491554959 >+SHA256 (librenms-librenms-1.26_GH0.tar.gz) = cc76a343296c48a6a2423efc9138e2a52d2827178ba733c0714dc6ef38d264e8 >+SIZE (librenms-librenms-1.26_GH0.tar.gz) = 57165188 >Index: files/patch-config.php.default >=================================================================== >--- files/patch-config.php.default (nonexistent) >+++ files/patch-config.php.default (working copy) >@@ -0,0 +1,11 @@ >+--- config.php.default.orig 2017-03-26 12:06:00 UTC >++++ config.php.default >+@@ -10,7 +10,7 @@ $config['db_name'] = 'librenms'; >+ >+ // This is the user LibreNMS will run as >+ //Please ensure this user is created and has the correct permissions to your install >+-$config['user'] = 'librenms'; >++$config['user'] = 'www'; >+ >+ ### This should *only* be set if you want to *force* a particular hostname/port >+ ### It will prevent the web interface being usable form any other hostname > >Property changes on: files/patch-config.php.default >___________________________________________________________________ >Added: fbsd:nokeywords >## -0,0 +1 ## >+yes >\ No newline at end of property >Added: svn:eol-style >## -0,0 +1 ## >+native >\ No newline at end of property >Added: svn:mime-type >## -0,0 +1 ## >+text/plain >\ No newline at end of property >Index: files/patch-daily.sh >=================================================================== >--- files/patch-daily.sh (nonexistent) >+++ files/patch-daily.sh (working copy) >@@ -0,0 +1,47 @@ >+--- daily.sh.orig 2017-03-26 12:06:00 UTC >++++ daily.sh >+@@ -113,7 +113,7 @@ main () { >+ # only try to su if we are root (or sudo) >+ if [[ "$EUID" -eq 0 ]]; then >+ echo "Re-running ${DAILY_SCRIPT} as ${LIBRENMS_USER} user" >+- su -l "$LIBRENMS_USER" -c "$DAILY_SCRIPT $@" >++ sudo -u "$LIBRENMS_USER" "$DAILY_SCRIPT" >+ exit; >+ fi >+ >+@@ -123,27 +123,6 @@ main () { >+ fi >+ >+ if [[ -z "$arg" ]]; then >+- up=$(php daily.php -f update >&2; echo $?) >+- if [[ "$up" == "0" ]]; then >+- $DAILY_SCRIPT no-code-update >+- exit >+- elif [[ "$up" == "1" ]]; then >+- # Update to Master-Branch >+- old_ver=$(git show --pretty="%H" -s HEAD) >+- status_run 'Updating to latest codebase' 'git pull --quiet' 'update' >+- new_ver=$(git show --pretty="%H" -s HEAD) >+- if [ "$old_ver" != "$new_ver" ]; then >+- status_run "Updated from $old_ver to $new_ver" '' >+- fi >+- elif [[ "$up" == "3" ]]; then >+- # Update to last Tag >+- old_ver=$(git describe --exact-match --tags $(git log -n1 --pretty='%h')) >+- status_run 'Updating to latest release' 'git fetch --tags && git checkout $(git describe --tags $(git rev-list --tags --max-count=1))' 'update' >+- new_ver=$(git describe --exact-match --tags $(git log -n1 --pretty='%h')) >+- if [[ "$old_ver" != "$new_ver" ]]; then >+- status_run "Updated from $old_ver to $new_ver" '' >+- fi >+- fi >+ >+ cnf=$(echo $(grep '\[.distributed_poller.\]' config.php | egrep -v -e '^//' -e '^#' | cut -d = -f 2 | sed 's/;//g')) >+ if ((${BASH_VERSINFO[0]} < 4)); then >+@@ -167,7 +146,6 @@ main () { >+ post-pull) >+ # List all tasks to do after pull in the order of execution >+ status_run 'Updating SQL-Schema' 'php includes/sql-schema/update.php' >+- status_run 'Updating submodules' "$DAILY_SCRIPT submodules" >+ status_run 'Cleaning up DB' "$DAILY_SCRIPT cleanup" >+ status_run 'Fetching notifications' "$DAILY_SCRIPT notifications" >+ status_run 'Caching PeeringDB data' "$DAILY_SCRIPT peeringdb" > >Property changes on: files/patch-daily.sh >___________________________________________________________________ >Added: fbsd:nokeywords >## -0,0 +1 ## >+yes >\ No newline at end of property >Added: svn:eol-style >## -0,0 +1 ## >+native >\ No newline at end of property >Added: svn:mime-type >## -0,0 +1 ## >+text/plain >\ No newline at end of property >Index: files/patch-validate.php >=================================================================== >--- files/patch-validate.php (nonexistent) >+++ files/patch-validate.php (working copy) >@@ -0,0 +1,39 @@ >+--- validate.php.orig 2017-04-11 18:45:34 UTC >++++ validate.php >+@@ -108,13 +108,6 @@ if (!($username === 'root' || (isset($co >+ print_fail('You need to run this script as root' . (isset($config['user']) ? ' or '.$config['user'] : '')); >+ } >+ >+-if ($git_found === true) { >+- if ($config['update_channel'] == 'master' && $cur_sha != $versions['github']['sha']) { >+- $commit_date = new DateTime('@'.$versions['local_date'], new DateTimeZone(date_default_timezone_get())); >+- print_warn("Your install is out of date, last update: " . $commit_date->format('r')); >+- } >+-} >+- >+ // Check php modules we use to make sure they are loaded >+ $extensions = array('pcre','curl','session','snmp','mcrypt'); >+ foreach ($extensions as $extension) { >+@@ -302,22 +295,6 @@ if (dbFetchCell('SELECT COUNT(*) FROM `d >+ } >+ >+ >+-if ($git_found === true) { >+- if ($versions['local_branch'] != 'master') { >+- print_warn("Your local git branch is not master, this will prevent automatic updates."); >+- } >+- >+- // check for modified files >+- $modifiedcmd = 'git diff --name-only --exit-code'; >+- if ($username === 'root') { >+- $modifiedcmd = 'su '.$config['user'].' -c "'.$modifiedcmd.'"'; >+- } >+- exec($modifiedcmd, $cmdoutput, $code); >+- if ($code !== 0 && !empty($cmdoutput)) { >+- print_warn("Your local git contains modified files, this could prevent automatic updates.\nModified files:"); >+- print_list($cmdoutput, "\t %s\n"); >+- } >+-} >+ // Modules test >+ $modules = explode(',', $options['m']); >+ foreach ($modules as $module) { > >Property changes on: files/patch-validate.php >___________________________________________________________________ >Added: fbsd:nokeywords >## -0,0 +1 ## >+yes >\ No newline at end of property >Added: svn:eol-style >## -0,0 +1 ## >+native >\ No newline at end of property >Added: svn:mime-type >## -0,0 +1 ## >+text/plain >\ No newline at end of property >Index: files/pkg-message.in >=================================================================== >--- files/pkg-message.in (revision 437919) >+++ files/pkg-message.in (working copy) >@@ -19,5 +19,9 @@ > 33 */6 * * * www cd %%WWWDIR%% && %%PHP%% discovery.php -h all >> /dev/null 2>&1 > */5 * * * * www cd %%WWWDIR%% && %%PHP%% discovery.php -h new >> /dev/null 2>&1 > */5 * * * * www cd %%WWWDIR%% && %%PYTHON%% poller-wrapper.py 16 >> /dev/null 2>&1 >-15 0 * * * www sh %%WWWDIR%%/daily.sh >> /dev/null 2>&1 >+15 0 * * * www %%LOCALBASE%%/bin/bash %%WWWDIR%%/daily.sh >> /dev/null 2>&1 > * * * * * www %%PHP%% %%WWWDIR%%/alerts.php >> /dev/null 2>&1 >+*/5 * * * * www %%PHP%% %%WWWDIR%%/poll-billing.php >> /dev/null 2>&1 >+01 * * * * www %%PHP%% %%WWWDIR%%/billing-calculate.php >> /dev/null 2>&1 >+*/5 * * * * www %%PHP%% %%WWWDIR%%/check-services.php >> /dev/null 2>&1 >+
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Flags:
freebsd
:
maintainer-approval?
Actions:
View
|
Diff
Attachments on
bug 218351
:
181565
|
181682
|
181691
| 181693