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

Added cmake uninstall target. #360

Closed
wants to merge 1 commit into from

Conversation

tcrass
Copy link

@tcrass tcrass commented Sep 18, 2022

@rncbc
Copy link
Owner

rncbc commented Sep 19, 2022

wonder whether this is really necessary :) you can always run qtractor from the cmake build folder (eg. build/src/qtractor), there's no real need to install it permanently as it's only necessary for packaging, which have their own uninstall procedures that often never rely on any upstream uninstall target ;)

thanks anyway

@tcrass
Copy link
Author

tcrass commented Sep 26, 2022

wonder whether this is really necessary :) you can always run qtractor from the cmake build folder (eg. build/src/qtractor), there's no real need to install it permanently as it's only necessary for packaging, which have their own uninstall procedures that often never rely on any upstream uninstall target ;)

I did in fact install a local qtractor build on various PCs in our household (yes, my kids are using qtractor, too, and I want them to be able to launch the application through the launcher menu :) in order to circumvent #357, so it would be nice if it was possible to easily uninstall it once the fix has made it into an official package.

And more generally (maybe for aesthetic reasons ;) if there's an "install" option I just expect an "uninstall" option to be also available.

Cheers --
Torsten

@rncbc
Copy link
Owner

rncbc commented Sep 26, 2022

so you're free to use your patch yourself, I guess

otoh. the latest builds from yours truly CI/CD upstream is available from these repos;
AppImages included: here's a direct link to the (always) latest snapshot:
https://download.opensuse.org/repositories/home:/rncbc:/apps/AppImage/qtractor-latest-x86_64.AppImage

byee

@tcrass
Copy link
Author

tcrass commented Sep 26, 2022

so you're free to use your patch yourself, I guess

Well then... so be it!

otoh. the latest builds from yours truly CI/CD upstream is available from these repos; AppImages included: here's a direct link to the (always) latest snapshot: https://download.opensuse.org/repositories/home:/rncbc:/apps/AppImage/qtractor-latest-x86_64.AppImage

Good to know, thanks!

Cheers --
Torsten

@tcrass tcrass closed this Sep 26, 2022
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.

2 participants