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

Fix get email for non-system-wide Perl installations #249

Merged
merged 1 commit into from
Mar 28, 2014
Merged

Conversation

hollie
Copy link
Owner

@hollie hollie commented Aug 18, 2013

When a user installs a local Perl version to run MisterHouse under instead of using the system-provided Perl, then the hardcoded Perl path at the shebang in get_email causes the module search path to be incorrect.

#!/usr/bin/env perl

properly solves this issue.

Related issues are #141 and #247.

When a user installs a local Perl version to run MisterHouse
under instead of using the system-provided Perl, then the
hardcoded Perl path at the shebang in get_email causes the
module search path to be incorrect. /usr/bin/env perl
properly solves this issue.
@hollie
Copy link
Owner Author

hollie commented Jan 5, 2014

Note: this needs to be extended to the other Perl scripts that are called by MisterHouse too. This is an initial pull request to enable users from other platforms to test it out.

krkeegan added a commit that referenced this pull request Mar 28, 2014
Fix get email for non-system-wide Perl installations
@krkeegan krkeegan merged commit 1a51721 into master Mar 28, 2014
@hollie hollie deleted the fix_get_email branch January 8, 2017 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants