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

Norc for pure env #605

Closed
wants to merge 2 commits into from
Closed

Norc for pure env #605

wants to merge 2 commits into from

Conversation

zimbatm
Copy link
Member

@zimbatm zimbatm commented Aug 16, 2015

What would you think of these changes ?

Just looking for feedback at this stage as I haven't figured-out how to test these changes yet.

@danbst
Copy link
Contributor

danbst commented May 17, 2016

+1. My current workaround is

# .bashrc
if [ -f /etc/bashrc ]; then
    . /etc/bashrc
fi
if [[ -z "$IN_NIX_SHELL" ]]; then
    cd /usr/local/ec2
    ... other logic
fi

@zimbatm
Copy link
Member Author

zimbatm commented May 17, 2016

My workaround is the reverse, my .bashrc looks like:

# If not running interactively, don't do anything
if [[ -z "$PS1" ]]; then return; fi
if [[ -n $IN_NIX_SHELL ]]; then return; fi

# usual code here
#...

@deepfire
Copy link

deepfire commented Jun 6, 2016

+1

@zimbatm zimbatm closed this Mar 4, 2017
@zimbatm zimbatm deleted the norc-for-pure-env branch March 4, 2017 16:06
@zimbatm zimbatm restored the norc-for-pure-env branch March 4, 2017 16:06
@deepfire
Copy link

deepfire commented Mar 5, 2017

@zimbatm, should the list be pinged about this? AIUI, the issue is as live as ever..

@zimbatm
Copy link
Member Author

zimbatm commented Mar 5, 2017

Oops, I removed the branch by mistake and it auto-closed the associated PR.

@zimbatm zimbatm reopened this Mar 5, 2017
@zimbatm
Copy link
Member Author

zimbatm commented Mar 5, 2017

Yes I would still like this to be implemented but don't know what the next steps are.

@deepfire
Copy link

deepfire commented Mar 5, 2017

@zimbatm, discussion, discussion, discussion! Ping the list, if the set of people participating here is insufficient -- and it seems that it isn't.

@zimbatm
Copy link
Member Author

zimbatm commented Mar 5, 2017

Feel free to take that over. I'm already juggling with too many things at the moment.

@zimbatm
Copy link
Member Author

zimbatm commented Mar 5, 2017

Once #1152 goes through it will also make it easier to validate PR functionalities

@danbst
Copy link
Contributor

danbst commented Mar 5, 2017

Actually, what is the state of issue in Nix 1.12? The current Perl script won't be used in future I guess

@zimbatm
Copy link
Member Author

zimbatm commented Aug 13, 2019

Closing as the PR is out of date

@zimbatm zimbatm closed this Aug 13, 2019
@zimbatm zimbatm deleted the norc-for-pure-env branch August 13, 2019 07:35
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

5 participants