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
Initial Posix build #12
Conversation
7dbd815
to
7ee9c7a
Compare
2964563
to
777aac3
Compare
CMakeLists.txt
Outdated
|
||
# Disable optimizations for interop.cpp for all compilers, to allow optimized | ||
# builds without need for -fno-omit-frame-pointer | ||
set_source_files_properties("${CMAKE_CURRENT_SOURCE_DIR}/src/openloco/interop/interop.cpp" PROPERTIES COMPILE_FLAGS -fno-omit-frame-pointer) | ||
|
||
add_executable(${PROJECT} ${ORCT2_SOURCES} ${OLOCO_MM_SOURCES} ${LOCO_SECTIONS}) | ||
target_compile_definitions(${PROJECT} PRIVATE -D_OPENLOCO_USE_BOOST_FS_) | ||
target_compile_definitions(${PROJECT} PRIVATE ${OPENLOCO_COMPILE_DEFINITIONS}) |
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.
This should be of _OPENLOCO_USE_BOOST_FS_=1
form
d7c615e
to
dd47213
Compare
This fixes compilation only, doesn't let it link just yet
This creates appropriate sections, but only uses default linker script for now.
Based on binutils 2.29.1-2
3bbd0bf
to
b569910
Compare
CMakeLists.txt
Outdated
# include lib | ||
include_directories("lib/") | ||
# add source files | ||
file(GLOB_RECURSE ORCT2_SOURCES "src/*.c" "src/*.cpp" "src/*.h" "src/*.hpp") |
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.
drop *.c
?
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.
Also, ORCT2_SOURCES
?
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.
Victim of copy and paste
7c9705b
to
9b373eb
Compare
CMakeLists.txt
Outdated
) | ||
add_custom_command( | ||
OUTPUT openloco_data | ||
COMMAND dd if="${OLOCO_EXE}" of="${OLOCO_DATA}" bs=4096 skip=215 count=78 |
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.
@marijnvdwerf can you double-check these offsets?
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 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.
text start = 0x1000/4096 = 1
text size: 0xd6000/4096 = 214
rodata start: 0xd7000/4096 = 215
rodata+data size: (0x1f000+0x2f000)/4096 = 78
Prepare window list to vector refactor
No description provided.