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

To add a uninstall script for packages #457

Open
sansyrox opened this issue Jan 29, 2019 · 4 comments
Open

To add a uninstall script for packages #457

sansyrox opened this issue Jan 29, 2019 · 4 comments

Comments

@sansyrox
Copy link
Member

Is your feature request related to a problem? Please describe.

To add an uninstall script for the packages. it would help us ease the process of implementing the soft-reset feature.

Describe the solution you'd like

To create an uninstall script.

Describe alternatives you've considered

We can either delete all packages from the package from a soft-reset daemon itself or create a separate script. I would recommend deleting it from the same file.

Additional context

Add any other context or screenshots about the feature request here.

@sansyrox
Copy link
Member Author

@mariobehling @Orbiter @hongquan @cweitat , your suggestions?

@hongquan
Copy link
Member

hongquan commented Feb 3, 2019

Actually, I want the "reset" to be implemented by packaging the susi_linux as deb or snap package. We will be able to solve many issues with this: factory reset, soft reset, automatic update, uninstall.

So, every implementation which we did until now: deliver a copy of susi_linux in tarball and use it for factory reset, use git to fetch the source code... all are dirty quirks. We should not continue that path.

Don't write the uninstall script now, it is still a quirk as other ones. If you have time, please invest in the deb/snap direction.

@sansyrox
Copy link
Member Author

sansyrox commented Feb 3, 2019

Okay @hongquan , I will try to look into packaging. Do you know of any good place to start reading?

@hongquan
Copy link
Member

hongquan commented Feb 7, 2019

Sorry, there is no good tutorial. I used to do it some years ago, but my knowledge may be outdated.

And before packaging, we need to redefine the app structure. Cannot let the app running inside /home/pi like now. More things to concern, I already mentioned somewhere in GitHub issues.

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

No branches or pull requests

2 participants