Comprehensive Installation guide #46
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit improves the OpenModScan installation script by: - Adding Robustness Checks: Implemented initial system checks for OS distribution (Debian/Ubuntu) and essential commands like `sudo`, `git`, `cmake`, and `make` to provide clearer diagnostics and prevent failures on incompatible systems. - Correcting Build System: Identified and switched the build process from QMake to CMake, as indicated by the project's `CMakeLists.txt` file. - Adjusting Directory Structure: Updated paths to correctly navigate into the `omodscan/` subdirectory, where the primary source and build files reside. - **Implementing Out-of-Source Build:** Configured CMake to perform an out-of-source build, which is a standard practice for CMake projects to keep source and build artifacts separate. - **Including Missing Dependencies:** Added `libqt5serialbus5-dev` to the dependency list, resolving the `Qt::SerialBus` target not found error during CMake configuration. - Correcting Executable Name: Adjusted the `APP_NAME` variable to `omodscan`, matching the actual executable name generated by the build process, ensuring correct installation to `/usr/local/bin`. - Enhancing Error Handling: Improved error messages for various stages of the installation process, providing more specific guidance for troubleshooting.
|
Some notes:
|
sanny32
requested changes
Jul 23, 2025
| @@ -0,0 +1,167 @@ | |||
| #!/bin/bash | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a shell script to improve the installation process by:
sudo,git,cmake, andmaketo provide clearer diagnostics and prevent failures on incompatible systems.CMakeLists.txtfile.omodscan/subdirectory, where the primary source and build files reside.libqt5serialbus5-devto the dependency list, resolving theQt::SerialBustarget not found error during CMake configuration.APP_NAMEvariable toomodscan, matching the actual executable name generated by the build process, ensuring correct installation to/usr/local/bin.