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

'perlbrew use' should not break the user's shell #435

Merged
merged 1 commit into from
Jan 20, 2015

Conversation

calid
Copy link
Contributor

@calid calid commented Dec 24, 2014

Starting bash subshells with --norc breaks the users shell setup and
seems unecessary. The --profile option is also unecessary since a
non-login shell is being started anyways.

Starting bash subshells with --norc breaks the users shell setup and
seems unecessary. The --profile option is also unecessary since a
non-login shell is being started anyways.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 3663805 on calid:bash-subshell-fix into 7e9a68d on gugod:develop.

@calid
Copy link
Contributor Author

calid commented Dec 24, 2014

Here's a screen grab showing what I'm referring to. Notice that my prompt (among other things) is completely busted in the perlbrew subshell:

selection_019

gugod added a commit that referenced this pull request Jan 20, 2015
'perlbrew use' should not break the user's shell
@gugod gugod merged commit de8a058 into gugod:develop Jan 20, 2015
@gugod
Copy link
Owner

gugod commented Jan 20, 2015

Hi @calid thanks for the work. However it is done in the generated file. I re-applied to perlbrew.pm in 11ad57a. :)

jsonn pushed a commit to jsonn/pkgsrc that referenced this pull request Feb 4, 2015
--------------
0.73: # 2015-02-01T09:59:45+0100
- fixes and improvements from Manuel Streuhofer, Aristotle Pagaltzis, Sebastian
  Willert, David Hand, Eric Enns,
  Peter Roberts, Thomas Sibley, Ivan Baidakou, thanks!
- fix 'perlbrew user' for fish users gugod/App-perlbrew#429
- fix sub-shell for bash users gugod/App-perlbrew#435
- clarify the 'perlbrew use' output: gugod/App-perlbrew#439
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.

None yet

3 participants