-
Notifications
You must be signed in to change notification settings - Fork 684
Build fix for RIOT target after the modifications of the build system… #1228
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
Build fix for RIOT target after the modifications of the build system… #1228
Conversation
-DFEATURE_VALGRIND=OFF \ | ||
-DCMAKE_TOOLCHAIN_FILE=cmake/toolchain_external.cmake \ | ||
-DJERRY_LIBC=OFF\ | ||
-DJERRY_CMDLINE=OFF\ |
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.
There is need a whitespace, or not?
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.
Probably optional, but I agree that it looks better with white space.
LGTM after the spaces added. |
c34322c
to
139027e
Compare
Thanks, I've updated the patch. |
cp `cat $(INTERM)/$(TYPE).external/list` $(OUTPUT)/. | ||
cp $(OUTPUT)/lib$(TYPE).jerry-core.a $(COPYTARGET)/libjerrycore.a | ||
make -C$(BUILD_DIR) jerry-core | ||
cp $(BUILD_DIR)/lib/libjerry-core.a $(COPYTARGET)/libjerrycore.a |
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.
$(BUILD_DIR) = build/riotstm32f4/
So after the cp command ..//lib/libjerry-core.a ... is it okey?
edit: Ditto at $(COPYTARGET)
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 is, but really ugly, I fixed it, thanks.
139027e
to
554f739
Compare
LGTM |
… (commit ddab1d8). JerryScript-DCO-1.0-Signed-off-by: Robert Sipka [email protected]
554f739
to
e6fefd4
Compare
I've rebased it with master. |
… (commit ddab1d8).
JerryScript-DCO-1.0-Signed-off-by: Robert Sipka [email protected]