docs: revise INSTALL.md with updated Maven setup#11025
Merged
DaanHoogland merged 2 commits intoapache:mainfrom Jun 16, 2025
Merged
docs: revise INSTALL.md with updated Maven setup#11025DaanHoogland merged 2 commits intoapache:mainfrom
DaanHoogland merged 2 commits intoapache:mainfrom
Conversation
DaanHoogland
approved these changes
Jun 14, 2025
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11025 +/- ##
============================================
- Coverage 16.60% 16.60% -0.01%
- Complexity 13924 13925 +1
============================================
Files 5730 5730
Lines 508224 508224
Branches 61789 61789
============================================
- Hits 84388 84387 -1
Misses 414401 414401
- Partials 9435 9436 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
bernardodemarco
approved these changes
Jun 15, 2025
Member
bernardodemarco
left a comment
There was a problem hiding this comment.
Only docs update, lgtm
b1136ac to
465b635
Compare
dhslove
pushed a commit
to ablecloud-team/ablestack-cloud
that referenced
this pull request
Jun 19, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Description
This PR updates the Maven installation instructions in INSTALL.md.
While following the setup steps to build CloudStack locally as a new contributor, I found that the provided download link for Maven 3.9.9 is no longer active, and the version itself is not listed on the official Apache Maven distribution site.
I’ve updated the instructions to use Maven 3.9.10, which is the most up-to-date version at the time of writing.
Additionally, I added sudo to the tar and ln commands since those require root permission when extracting to or linking under /usr/local.
This small patch improves the onboarding experience for new developers setting up their environment.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
I manually tested by following the updated instructions to set up Maven 3.9.10 on a fresh environment.
How did you try to break this feature and the system with this change?
Maven 3.x on official Apache Maven distribution site.