Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

require job files once #31

Merged
merged 1 commit into from

2 participants

@qzio

I have a stupid setup, where my jobs can't be autoloaded through an autoloader, So i need to require them through a bootstrap that's included in all jobs.

since the validate_job_workers include the file again, with @ and without _once gearman-manager blows up with an

Child exited with non-zero exit code 65280

error.

this commit uses require_once instead of @include which seems to fix the problem for me.

why is @include used everywhere instead of require_once?
I don't know gearman-manager enough to know the argument so please enlighten me :)

@brianlmoon brianlmoon merged commit 7d09c45 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 8, 2011
  1. @qzio

    require job files once

    qzio authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  pecl-manager.php
View
2  pecl-manager.php
@@ -202,7 +202,7 @@ public function do_job($job) {
protected function validate_lib_workers() {
foreach($this->functions as $func => $props){
- @include $props["path"];
+ require_once $props["path"];
$real_func = $this->prefix.$func;
if(!function_exists($real_func) &&
(!class_exists($real_func) || !method_exists($real_func, "run"))){
Something went wrong with that request. Please try again.