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

Update Linux install instructions #1116

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Conversation

bkmgit
Copy link
Contributor

@bkmgit bkmgit commented Jun 2, 2024

Need to specify INSTALL_ROOT rather than PREFIX for custom install locations.

Need to specify INSTALL_ROOT rather than PREFIX for custom install locations.

Signed-off-by: Benson Muite <[email protected]>
Copy link

what-the-diff bot commented Jun 2, 2024

PR Summary

  • Simplified Installation Instructions
    We've made a major simplification to the installation instructions found in the .github/README-DEVELOPER.md file. Now, building Seamly's application will be easier and less complex! You won't have to use the longer qmake PREFIX=/usr/local Seamly2D.pro command that could get a bit intimidating. Instead, let's just focus on using make INSTALL_ROOT=/usr/local install, so we direct Seamly's installation binary files where they need to be. This change makes everything straightforward and user-friendly. Let's make tech less stressful and more fun! 😊🚀

@bkmgit
Copy link
Contributor Author

bkmgit commented Jun 2, 2024

Closes #1115

@slspencer
Copy link
Collaborator

slspencer commented Jun 3, 2024

@bkmgit You're recommending make instead of qmake. Qmake and make are not equivalent.
https://stackoverflow.com/questions/1368534/why-does-qt-use-its-own-make-tool-qmake
Have you compiled locally on a Linux distro and tested that the app is working (signals & slots are working), and that the executable is in usr/local/bin?

@slspencer slspencer merged commit ee3c3c7 into FashionFreedom:develop Jun 3, 2024
2 checks passed
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