Skip to content

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

Merged
merged 1 commit into from
Aug 1, 2016

Conversation

robertsipka
Copy link
Contributor

… (commit ddab1d8).

JerryScript-DCO-1.0-Signed-off-by: Robert Sipka [email protected]

-DFEATURE_VALGRIND=OFF \
-DCMAKE_TOOLCHAIN_FILE=cmake/toolchain_external.cmake \
-DJERRY_LIBC=OFF\
-DJERRY_CMDLINE=OFF\
Copy link
Contributor

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?

Copy link
Member

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.

@zherczeg
Copy link
Member

LGTM after the spaces added.

@robertsipka
Copy link
Contributor Author

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
Copy link
Contributor

@polaroi8d polaroi8d Jul 28, 2016

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)

Copy link
Contributor Author

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.

@LaszloLango
Copy link
Contributor

LGTM

@LaszloLango LaszloLango added the bug Undesired behaviour label Aug 1, 2016
@LaszloLango LaszloLango added this to the Release v1.0 milestone Aug 1, 2016
… (commit ddab1d8).

JerryScript-DCO-1.0-Signed-off-by: Robert Sipka [email protected]
@robertsipka
Copy link
Contributor Author

I've rebased it with master.

@LaszloLango LaszloLango merged commit e6fefd4 into jerryscript-project:master Aug 1, 2016
@robertsipka robertsipka deleted the riot_buildfix branch November 24, 2016 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Undesired behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants