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

Simplify the installation process #50

Open
spyhunter99 opened this issue Dec 14, 2016 · 7 comments
Open

Simplify the installation process #50

spyhunter99 opened this issue Dec 14, 2016 · 7 comments

Comments

@spyhunter99
Copy link

instead of relying on an ide, adding some maven configuration bits so that the installation process and can scripted without a full dev environment

@lalmeras
Copy link
Member

lalmeras commented Dec 14, 2016

I think I can easily implement a jetty:run maven target to ease application testing. I just need some time and motivation to work on it.

I'll post messages here if there are enhancements on this point.

@spyhunter99
Copy link
Author

I may open a PR for it to have a simple command line util to initialize the database

@lalmeras
Copy link
Member

Update about the installation process, without any other requirements than a running maven:

Build all the codebase (with JDK 7)
mvn -Pdevelopment -DskipTests clean install

Once maven_artifact_notifier database is created, owned by maven_artifact_notifier, with password maven_artifact_notifier: run this command in maven-artifact-notifier-init folder:
mvn exec:java -Dexec.mainClass="fr.openwide.maven.artifact.notifier.init.MavenArtifactNotifierInitFromExcelMain

I continue to work on a jetty:run step. I'll report all this in the documentation once tested, and I would be happy to receive your feedback.

JDK8 build is a later target as it needs to work on owsi-core build also (NB: only build requires JDK7; code can be run with JDK8)

lalmeras pushed a commit that referenced this issue Dec 14, 2016
lalmeras pushed a commit that referenced this issue Dec 14, 2016
@lalmeras
Copy link
Member

Updated process:

Build all the codebase (with JDK7)

mvn -Pdevelopment -DskipTests clean install

Once maven_artifact_notifier database is created, owned by maven_artifact_notifier postgresql user, with password maven_artifact_notifier: run this command in maven-artifact-notifier-init folder:

mvn exec:java -Dexec.mainClass="fr.openwide.maven.artifact.notifier.init.MavenArtifactNotifierInitFromExcelMain

(this step requires feature-50-dev-tooling branch)
Finally, switch to maven-artifact-notifier-webapp folder and run:

mvn jetty-run

Application is available at http://localhost:8080; default login/password is [email protected]/admin.

@spyhunter99 can you provide me some feedback about this process ?

NB: feature-50-dev-tooling branch includes postgresql 9.6 fix.

lalmeras pushed a commit that referenced this issue Dec 14, 2016
lalmeras pushed a commit that referenced this issue Dec 14, 2016
@spyhunter99
Copy link
Author

first of all, thanks for the super quick turn around. mvn jetty-run didn't work for me but mvn jetty:run-war did. great success. i'm up and running for the time being. If I have additional questions, do you prefer new tickets or just post more things to the wiki?

@lalmeras
Copy link
Member

Error on my side. I'm happy you find the correct command. The command is (and jetty:run-war is surely quite similar):

mvn jetty:run

For additional questions, I think ticket is the best way to share and agree on the good solution.

On this subject (project initialization), I'll welcome a PR on README.md or a new wiki page. Once the new content created and added, we'll see how to organize all the existing documentation on initialization.

@spyhunter99
Copy link
Author

lalmeras pushed a commit that referenced this issue Mar 4, 2017
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

No branches or pull requests

2 participants