prdatur / phpminer

PHPMiner is a web interface for cgminer on linux machines
Other
34 stars 15 forks source link

cronjob cron.php references now unused /config/config.json #33

Closed neurocis closed 10 years ago

neurocis commented 10 years ago

Just upgraded to latest and purged my unused config files but it appears cron.php still references config.json as opposed to mysql db for configuration resulting in notification and auto task failure:

cron.php

// Get the system config.
$config = new Config(SITEPATH . '/config/config.json');

// Can't do anything if nothing is configurated.'
if ($config->is_empty()) {
    unlock();
    exit;
}
prdatur commented 10 years ago

thx

prdatur commented 10 years ago

Sorry closed it to fast, the string it self was never used with the newest version. It just was provided but there is no constructor in config which takes this path. What exactly don't work with notifications?

neurocis commented 10 years ago

Hmm, I just pulled the latest commit, but get the dreaded (same as before), thought it was exiting as config.json no longer existed (only had this issue after clearing out the config files):

Manually firing cron.php gives the usual 'PHP Warning: Module 'PDO' already loaded in Unknown on line 0' error, and I have verified /etc/cron.d/phpminer exists and is correct.

As I said, this only started happening since clearing out the unused config files.

neurocis commented 10 years ago

Grrr, fixed, orphan /tmp/phpminer_cron.pid, and multiple defunct cron.php processes.

removal fixed.

prdatur commented 10 years ago

That is another issue which I will now open, the cron job is sometimes defunc and i don't know why.