updating the env

[tcurdt]

I have a dep that creates a file in "/etc/profile.d" which updates the PATH but since the file does not get sourced the "in_path?" check fails.

Not really sure how to update the environment.

Any thoughts how to do this?

[Ben Hoskings]

This message slipped through the cracks of my inbox, sorry :)

Is this still a problem?

- Ben

> --
> To post, email babush...@googlegroups.com
> To unsubscribe, email babushka_app...@googlegroups.com
> ~
> http://babushka.me
> http://github.com/benhoskings/babushka
> http://groups.google.com/group/babushka_app

[Torsten Curdt]

Unfortunately that's still a big problem. You never run into this before?

cheers,

Torsten

[Paul Annesley]

So you have something along these lines? (pseudocode-ish)

met? { ENV["PATH"].contains?("/some/dir") }
meet { write_file_which_sets_path("/etc/profile.d/something") }

I would never expect writing a file to disk to change the ENV of a running process.

Can your `met?` block start a sub-shell which would load /etc/profile.d files and check for the correct $PATH?
It might need to be a login shell…?

— Paul

[Ben Hoskings]

Yep, I'd start a subshell. You should be able to do this with #login_shell.

I have a dep that achieves a similar purpose here:

https://github.com/conversation/babushka-deps/blob/master/env.rb

- Ben

[Torsten Curdt]

It's my rbenv dep here

https://github.com/tcurdt/babushka-deps/blob/master/rbenv.rb

So maybe I just need to replace "shell" with "login_shell" then?

Will give that a try.

[Ben Hoskings]

Hi Torsten, how'd you go with this?

- Ben

[Torsten Curdt]

Thanks for checking - using "login_shell" did the trick. Still a few things to improve though. Is there a 

raw_login_shell as well? (to handle stdout etc)

Right now I am also using login_shell to check for the path - not ideal.

[Ben Hoskings]

There isn't. There are a few combinations of shell methods missing -- for example, there are #shell! and #log_shell, but no #log_shell!.

Perhaps I should refactor some of those modified shell methods to be option-driven instead.

- Ben

[Torsten Curdt]

+1

find it quite confusing what is available and what is not

[Ben Hoskings]

Ticketed this: https://github.com/benhoskings/babushka/issues/248

Cheers

Ben