-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Option to skip _patch_bashrc_and_profile #705 #706
Comments
I think the default behavior should be like this:
Also do we have to patch the bashrc? I'm a zsh user, on zsh you only have to add |
Also worth mentioning that adding So yeah I think we should just give a warning and ask the user if they want to patch their EDIT: I changed it instead to just patch if we detect that |
@Samueru-sama I think you are absolutely right. I would like to stop patching configuration files, and I would like to delegate everything to the README. It's not "AppMan" who should handle them. I want to disable the function, or at least I would like an option that can solve the problems. |
In that case we simply issue a warning and ask if the user would like that appman patches their edit: adding the question part right now |
@Samueru-sama we must go talk at #707 |
@rvenutolo let us know if this works for you. |
This should work for me. Thanks very much. |
I would love an option to have the
APP-MANAGER
script skip the function_patch_bashrc_and_profile
where the script writes the following to~/.bashrc
(and other dotfiles):Personally, I don't like anything touching my bash dotfiles and prefer to manage my
PATH
manually via my own dotfiles.Any option to disable that would be great. I don't know if you'd prefer an environment variable to control it like
[[ -z "${APPMAN_SKIP_PATCH_BASHRC_AND_PROFILE:-} ]] && _patch_bashrc_and_profile
or a value in theappman-config
config file.Unrelated to this, I just want to say I love
appman
. I had ignored AppImages due to the lack of a package management command line tool. Thank you so much for this. This plusAppImaGen
andArchImage
are great. I very much appreciate it.Originally posted by @rvenutolo in #705
The text was updated successfully, but these errors were encountered: