-
Notifications
You must be signed in to change notification settings - Fork 108
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
Instructions & build support for NTF #77
Instructions & build support for NTF #77
Conversation
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it wont go here. that will be the Makefile
@krsna1729 do you want to have these instructions on the Makefile and spgwu as comments (instead of install.md)? |
@yiannisy the docker_setup.sh changes was what i was referring to be in Makefile. Since that is where the build happens. Something like this?
user can run do
or
|
I don't have a build environment to try and test, so I will defer to @nevelis for this :) |
d7cb7bf
to
707f907
Compare
707f907
to
6af0f67
Compare
43e7045
to
205d8a8
Compare
retest this please |
205d8a8
to
eef155b
Compare
Hey @krsna1729 I committed your USE_NTF => ENABLE_NTF suggestion after your last review, any chance you could check it off? Thanks! |
@nevelis |
retest this please |
There were some inconsistencies with the |
Hi @nevelis. I am sorry... but can you please rebase? I will merge this PR today. |
eef155b
to
b599604
Compare
retest this please |
1 similar comment
retest this please |
Small note on how to compile & activate NTF within OMEC.