Skip to content

Revert "Use require_relative to load cookbooks"#4808

Merged
philbogle merged 1 commit into
stagingfrom
revert-4802-require_relative
Oct 26, 2015
Merged

Revert "Use require_relative to load cookbooks"#4808
philbogle merged 1 commit into
stagingfrom
revert-4802-require_relative

Conversation

@philbogle

Copy link
Copy Markdown
Contributor

Reverts #4802

@philbogle

Copy link
Copy Markdown
Contributor Author

@wjordan FYI I am rolling this back; we'll need a different approach because require_relative doesn't work reliably in Pegasus (probably due to the way it eval files).

Maybe we could do that trick of:
require File.expand_path('../../config/environment', FILE)

Please let me know if you'd like to make that change or if it would be helpful for me to.

philbogle added a commit that referenced this pull request Oct 26, 2015
Revert "Use require_relative to load cookbooks"
@philbogle philbogle merged commit c577617 into staging Oct 26, 2015
@philbogle

Copy link
Copy Markdown
Contributor Author

I checked in an updated version which does the expand_path trick.

@davidsbailey davidsbailey deleted the revert-4802-require_relative branch January 7, 2016 19:20
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.

1 participant