-
Notifications
You must be signed in to change notification settings - Fork 68
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
Migrate RelEx to maven #267
Comments
Anything else to do here? |
It is needed to create maven artifacts to replace manually written scripts like |
Well, the *.sh scripts were meant to be "examples", that the user could
modify to suit their purposes.
So whatever they are replaced with, they should also be "examples", and be
labeled as such, and
should somehow be easy and obvious to understand.
…--linas
On Sun, Sep 9, 2018 at 9:27 PM, Vitaly Bogdanov ***@***.***> wrote:
It is needed to create maven artifacts to replace manually written scripts
like opencog-server.sh and relation-extractor.sh. In fact all of *.sh
scripts in the root project folder.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#267 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAFwoBKhOlZdItbTOlpuNSV3cPAD5ji-ks5uZRcwgaJpZM4VPwmP>
.
--
cassette tapes - analog TV - film cameras - you
|
Actually I thought about it again and looks like the simplest way is to collect all of the dependencies in one folder using maven and then use them as classpath in existing scripts. Then scripts themselves will not be changed, only classpath will be. I have raised #276 to collect dependencies. And I see the following further steps:
|
Is the work here finished? Can this be closed? |
Yes, thanks |
See second part of the #100 (comment).
Maven can manage most of dependencies automatically so it will simplify build procedure.
It has a plugin to automate ready to run environment createion (see #266)
The text was updated successfully, but these errors were encountered: