-
Notifications
You must be signed in to change notification settings - Fork 1
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
Create uninstallation scripts #14
Comments
This is just a thought, but could we add some more environment variables during the installation process that would make the uninstall easier. E.g the parent shell, the path to folder, the presentation year etc? Or could we write some of this information to a file (maybe a hidden file) that lives in the main m269 folder? Of course a file like this could be accidently deleted or changed. |
Good ideas! The uninstallation script could be written assuming the existence of variables like |
I have a working uninstallation script for Unix (at least on my machine) and will submit the PR. This can be treated like a first draft, as some parts might need a few tweaks and changes etc. A couple of points:
|
Thanks. This will require some iterations (also for Windows) until completed, so could you please withdraw the PR and instead create an and commit a branch? On this issue's page, on the right sidebar, there's a link to create a branch associated to this issue under the 'Development' heading. |
I have withdrawn the PR, but I don't think I have permissions to create a new branch in this repo. Maybe if you created the new branch I could submit PR to the new branch? |
Had a quick look at your PR. Some suggestions:
|
Thanks, I have incorporated those suggestions. A couple of questions:
|
I think the Unix uninstallation script is ready for more critique, testing and suggestions. Some of my thoughts were:
|
I prefer the env vars to be written to On further reflection, there's no point in keeping the aliases if the venv and allowed.py are removed. And the whole point of the script is to remove the 1Gb venv. So, to simplify, I suggest the script simply warns the user at the beginning that it will remove the venv in ~/venvs/m269-YYj and the aliases nb, allowed and m269-YYj and that therefore the notebooks will be no longer readable or executable unless another Jupyter environment exists. Then asks the user to confirm to proceed. The only other thing to ask the user is if they want to keep the M269 style of notebooks, otherwise it will be removed. |
It would be nice to have uninstallation scripts
uninstall.sh
anduninstall.ps1
. The scripts would do the following:[Mm]269-\d\d[Jj]
and extract the year from itallowed.py
andm269.json
files in the current foldernb
andallowed
, the 'content' of the alias must be checked; maybe the easiest is to check if the alias mentions the current folder~/.jupyter/custom/custom.css
Removing lines from the CSS and shell startup files can be done with
sed
on Unix, but no clue how to do it in Windows.Once the scripts are done and tested:
FILES
in the corresponding installation scriptThe text was updated successfully, but these errors were encountered: