-
Notifications
You must be signed in to change notification settings - Fork 4
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
Improve build instructions / process #1
Comments
UPD. I was able to build the project using lines from Travis configuration file and installing everything locally. After amending some configuration I was also able to run tests from Eclipse. Should I close this issue now? |
I've renamed the issue to reflect the problem here. @wotopul is there anything you could submit as a PR in terms of better configuration, docs, etc.? |
The MemSAT build process that works on Travis CI uses Maven, and has two steps:
Take a look at the Travis CI configuration for MemSAT if you need more details. |
@juliandolby maybe we should update the MemSAT README with those instructions? That sounds a lot simpler than what is written there now. |
@juliandolby yes, thank you. I've understood that already. But if you aren't going to deploy these WALA artifacts anywhere wouldn't it be better to have this steps explicitly mentioned in README (as suggested by @msridhar )? It will save time for someone else. |
Yes, you are right. I shall update the README if you confirm that these steps work for you as well as me. Thanks.
… On Aug 31, 2017, at 6:27 PM, Dmitry Suvorov ***@***.***> wrote:
@juliandolby <https://github.com/juliandolby> yes, thank you. I've understood that already. But if you aren't going to deploy these WALA artifacts anywhere wouldn't it be better to have this steps explicitly mentioned in README? It will save time for someone else.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABk3fnkcB-22v4VGnfzTmb2P1Vu6A0tZks5sdzNngaJpZM4PH8KO>.
|
I changed the README to have the Maven instructions |
Thank you all for fast responses. |
I'm having a lot of difficulty compiling this project on a linux command line. I'm not a java/maven person so that's possibly not helping. In particular, having built WALA checked out at
Can anyone offer some help and guidance on the current build process? |
I've helped myself. I eroneously checked out WALA to the most recent release, and my setup doesn't set $JAVA_HOME as expected by some parts of the build system. |
In general, MemSAT is kept up to date with the latest WALA, and will likely work only with that version. Thus, getting the latest WALA is the right thing to do. As for JAVA_HOME, I think it is normally set by the shell, so you should not need to set it by hand. |
Hi,
I've run into trouble trying to build MemSAT. As I understand Maven build is not working now because of unsatisfied dependencies. So I've tried importing
memsat.psf
as a team project in Eclipse and got the following error:It seems that the projects listed in
memsat.psf
are inconsistent with current state ofWALA
project. Could you fix this or provide more detailed instructions if I am doing something wrong? More detailed instructions would be helpful anyway as a vanilla Maven building process.BR
The text was updated successfully, but these errors were encountered: