Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition of config.php.dist to weathermap #160

Open
gbilson opened this issue Nov 18, 2024 · 1 comment
Open

Addition of config.php.dist to weathermap #160

gbilson opened this issue Nov 18, 2024 · 1 comment
Labels
bug Something isn't working resolved A resolved issue

Comments

@gbilson
Copy link

gbilson commented Nov 18, 2024

Renamed config.php.dist to config.php.

Then I had to edit config.php and add the single quotes around both !defined statement variables WM_CONFIGDIR and WM_OUTPUTDIR they were missing, which wouldn't allow weathermap to install.

/* we will define them here */
if (!defined('WM_CONFIGDIR')) {
define('WM_CONFIGDIR', $configdir);
}

if (!defined('WM_OUTPUTDIR')) {
define('WM_OUTPUTDIR', $outputdir);

I also followed the directions in config.php and copied the content of configs and output to the new location and weathermap installed.

/**

  • These two settings need to be defined per your Cacti install. In legacy
  • versions of weathermap, these two files were located in the plugin
  • directory. However, in Cacti 1.3, the archive function will interfere
  • in that locations setting. So, pick a location that is owned by
  • the apache user, but is not in the Cacti base path. Ensure that
  • the path's end with a '/'.
    */
@TheWitness
Copy link
Member

Okay, I fixed the single quote issue. The trailing slash we will handle programmatically.

@TheWitness TheWitness added bug Something isn't working resolved A resolved issue labels Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working resolved A resolved issue
Projects
None yet
Development

No branches or pull requests

2 participants