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 install_from_sources.md #2274

Merged

Conversation

KeshavSoni2511
Copy link
Contributor

Link for apache maven is updated. Fix for issue #2272

Link for apache maven is updated
@fgalan
Copy link
Member

fgalan commented Jul 5, 2023

I have check the maven version installed in the generated docker images

# dpkg -l maven
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version      Architecture Description
+++-==============-============-============-=======================================================
ii  maven          3.6.3-5      all          Java software project management and comprehension tool
# 

So I'd suggest to use 3.6.3 version in doc/cygnus-common/installation_and_administration_guide/install_from_sources.md

Resolved as per comment
@KeshavSoni2511
Copy link
Contributor Author

Hi @fgalan, I have done the changes as per your comment.

Copy link
Member

@fgalan fgalan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fgalan fgalan merged commit 9d944bf into telefonicaid:master Jul 5, 2023
5 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