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

docs: update uninstall.md to include removal of bash.bashrc.backup-before-nix file #10304

Closed
wants to merge 1 commit into from

Conversation

BBBmau
Copy link

@BBBmau BBBmau commented Mar 23, 2024

Motivation

Was following the uninstall docs and still couldn't get the install to work until i realized i needed to remove the actual bash.bashrc.backup-before-nix file completely despite it being empty. I've added this in the final note of the docs in the event that others run into this issue also.

Context

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@BBBmau BBBmau requested a review from edolstra as a code owner March 23, 2024 20:58
@BBBmau BBBmau changed the title Update uninstall.md to include removal of bash.bashrc.backup-before-nix file docs: update uninstall.md to include removal of bash.bashrc.backup-before-nix file Mar 23, 2024
@abathur
Copy link
Member

abathur commented Mar 24, 2024

Moving this file out of the way is already documented in step 1:

sudo mv /etc/bash.bashrc.backup-before-nix /etc/bash.bashrc

@BBBmau
Copy link
Author

BBBmau commented Mar 28, 2024

@abathur this only moves the contents of /etc/bash.bashrc.backup-before-nix into /etc/bash.bashrc this does not delete the file once moved which is where my issue comes from since I had to explicitly do a deletion of /etc/bash.bashrc.backup-before-nix even after doing all the steps. I still propose that we include this.

@fricklerhandwerk
Copy link
Contributor

That doesn't sound right. mv literally moves the file, the original file thus won't exist afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants