Skip to content
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 nxdk for micropython compilation #4

Closed
wants to merge 2 commits into from
Closed

Conversation

JayFoxRox
Copy link
Owner

@JayFoxRox JayFoxRox commented Aug 3, 2018

TODO:

lib/xboxrt/errno.h Outdated Show resolved Hide resolved
lib/xboxrt/startup.c Outdated Show resolved Hide resolved
lib/xboxrt/stdbool.h Outdated Show resolved Hide resolved
lib/xboxrt/unistd.h Outdated Show resolved Hide resolved
lib/xboxrt/assert.h Outdated Show resolved Hide resolved
lib/xboxrt/setjmp.h Outdated Show resolved Hide resolved
@JayFoxRox JayFoxRox added the collection Collection of changes to compile a specific program. Needs review if still required label Oct 4, 2019
@@ -56,18 +56,18 @@ NXDK_CFLAGS = -target i386-pc-win32 -march=pentium3 \
NXDK_ASFLAGS = -target i386-pc-win32 -march=pentium3 \
-nostdlib -I$(NXDK_DIR)/lib -I$(NXDK_DIR)/lib/xboxrt
NXDK_CXXFLAGS = $(NXDK_CFLAGS)
NXDK_LDFLAGS = -subsystem:windows -dll -out:'$@' -entry:XboxCRTEntry \
-stack:$(NXDK_STACKSIZE)
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188


ifeq ($(DEBUG),y)
NXDK_CFLAGS += -g
NXDK_CXXFLAGS += -g
LDFLAGS += -debug
NXDK_LDFLAGS += -debug
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188

@echo "[ LD ] $@"
$(VE) $(LD) $(LDFLAGS) -subsystem:windows -dll -out:'$@' -entry:XboxCRTEntry -stack:$(NXDK_STACKSIZE) $^
$(VE) $(LD) $(NXDK_LDFLAGS) $(LDFLAGS) $^
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188


%.obj: %.cpp
@echo "[ CXX ] $@"
$(VE) $(CXX) $(NXDK_CXXFLAGS) -MD -MT '$@' -MF '$(patsubst %.cpp,%.cpp.d,$<)' -c -o '$@' '$<'
$(VE) $(CXX) $(NXDK_CXXFLAGS) $(CXXFLAGS) -MD -MT '$@' -MF '$(patsubst %.cpp,%.cpp.d,$<)' -c -o '$@' '$<'
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188


%.obj: %.c
@echo "[ CC ] $@"
$(VE) $(CC) $(NXDK_CFLAGS) -MD -MT '$@' -MF '$(patsubst %.c,%.c.d,$<)' -c -o '$@' '$<'
$(VE) $(CC) $(NXDK_CFLAGS) $(CFLAGS) -MD -MT '$@' -MF '$(patsubst %.c,%.c.d,$<)' -c -o '$@' '$<'
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188


%.obj: %.s
@echo "[ AS ] $@"
$(VE) $(AS) $(NXDK_ASFLAGS) -c -o '$@' '$<'
$(VE) $(AS) $(NXDK_ASFLAGS) $(ASFLAGS) -c -o '$@' '$<'
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed in PR 188

@JayFoxRox
Copy link
Owner Author

Upstreamed what I considered interesting, the rest doesn't matter. micropython compiles with upstream nxdk anyway - I assume it still works, too.

@JayFoxRox JayFoxRox closed this Oct 4, 2019
@JayFoxRox JayFoxRox deleted the micropython branch October 4, 2019 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
collection Collection of changes to compile a specific program. Needs review if still required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant