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

Allow passing in APP_INCLUDE to php-resque #127

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kierate
Copy link

@kierate kierate commented May 18, 2015

Currently APP_INLCUDE is hardcoded and will not work if bootstrap.php.cache is outside of the app/ directory (like in any project using the Symfony 3 directory structure).

This change allows the APP_INCLUDE to be set via the configuration, and also via the environment variable (which overrides the value from the config).

By default it works as previously i.e. if you do not pass in the new config value of the APP_INLUCDE environment variable then it takes %kernel.root_dir%/bootstrap.php.cache.

Example config for a project with Symfony 3 directory structure would be:

bcc_resque:
    app_include: %kernel.root_dir%/../var/bootstrap.php.cache

@mpclarkson
Copy link

This really needs to be merged.

@tomtomau
Copy link

+1 - this is really important

@gun1000000
Copy link

+1

1 similar comment
@mathielen
Copy link

+1

@mpclarkson
Copy link

As it is clear that this bundle is no longer being actively maintained, I have taken it upon myself to maintain a fork here https://github.com/mpclarkson/resque-bundle. I welcome any contributions. @tomtomau ... small world ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants