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

Clarify usage of Maven configuration #94

Closed
jmesnil opened this issue Jul 6, 2022 · 5 comments · Fixed by #96
Closed

Clarify usage of Maven configuration #94

jmesnil opened this issue Jul 6, 2022 · 5 comments · Fixed by #96

Comments

@jmesnil
Copy link
Member

jmesnil commented Jul 6, 2022

I find the usage of the Maven configuration ambiguous:

./prospero install --help
Install a new application server instance.

      --channel=<channel>   Channel file URL.
      --channel-repo=<channelRepositories>
                            URL of a remote Maven repository that contains artifacts required to build an application container.

-> I find the name channel-repo confusing. aiui, this parameter allows to configure any Maven repositories to fetch the application server bits. Could it be renamed to remote-repositories ?
-> The help message should be updated to something like: `URLs of remote Maven repositories that contains the artifacts required to install the application server ( multiple URLs are separated by comma).

      --local-repo=<localRepo>
                            Path to a local Maven repository.

-> should be path to the local Maven repository (there is always a single local maven repo).
-> could it default to $HOME/.m2/repository?

      --offline             Perform installation from local Maven repository only. Offline installation requires --local-repo to be configured.

-> if we provide a default value for local-repo, offline installation would not required it to be configured. wdyt?

If I want to perform an offline installation, ideally, I should be able to run

 ./prospero install --fpl=eap-8.0 --dir=eap8 --channel-repo=file:///Users/jmesnil/tmp/jboss-eap-8.0.0.Beta-maven-repository/maven-repository/ --offline
@jmesnil
Copy link
Member Author

jmesnil commented Jul 6, 2022

.setChannelRepo(channelRepositories == null || channelRepositories.isEmpty() ?
null : channelRepositories.get(0).toString())

Only the first URL from channel-repo is used, is that correct?

@spyrkob
Copy link
Collaborator

spyrkob commented Jul 6, 2022

Only the first URL from channel-repo is used, is that correct?

That seems to be a mistake from when we moved to Picocli. I'll fix that

@spyrkob
Copy link
Collaborator

spyrkob commented Jul 6, 2022

if we provide a default value for local-repo, offline installation would not required it to be configured. wdyt?

At the moment the default is to use a temporary folder for the local repository as there were some concerns about disk usage.
But we can make it so that if --local-repo is present without value it defaults to ~/.m2/repository, and if value is present it uses the specified folder?
Or maybe using ~/.m2/repository should be default, --local-repo allows to change this location and a new flag (eg --no-local-repo) would use the temp?

spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 6, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 6, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 6, 2022
@jmesnil
Copy link
Member Author

jmesnil commented Jul 7, 2022

Or maybe using ~/.m2/repository should be default, --local-repo allows to change this location and a new flag (eg --no-local-repo) would use the temp?

That sounds a good approach. --no-local-repo could be renamed to --no-resolve-local-cache or something similar?

spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 7, 2022
@spyrkob
Copy link
Collaborator

spyrkob commented Jul 8, 2022

cc @marekkopecky

spyrkob added a commit to spyrkob/prospero that referenced this issue Jul 8, 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 a pull request may close this issue.

2 participants