Skip to content

Win32 support #118

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.

Sign up for GitHub

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

Draft
cyanidle wants to merge 13 commits into
base: master
Choose a base branch
from
Draft

Win32 support #118

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
0f462e0
Work on porting to Windows
cyanidle May 13, 2025
06d75cf
Rework API for IOCP
cyanidle May 13, 2025
82833ee
Fix posix side for new API. Fix memory leaks
cyanidle May 13, 2025
0e81a7c
Revert to old clang-format
cyanidle May 13, 2025
7a29310
It segfaults for some reason on depsparse
cyanidle May 13, 2025
a1e5387
Minor improvements
cyanidle May 14, 2025
9dc4f7e
Fix deps/log file renaming. Now subprocesses work
cyanidle May 14, 2025
6b320ca
Fix warning message on Pipe write end opening
cyanidle May 14, 2025
169b235
Run clang-format
cyanidle May 16, 2025
564bc31
Resolve unused var errors + fix types for some ints
cyanidle May 16, 2025
9c2fc18
Change NamedPipe pattern
cyanidle May 16, 2025
ce3fa9f
Fix variables escaping
cyanidle May 27, 2025
9c91b09
Canonpath: handle windows path
cyanidle May 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
*.o
.*.sw*
/samu
out/
.vs/
.vscode/
.cache/
build/
CMakeUserPresets.json
CMakePresets.json
CMakeSettings.json
*.user
Copy link
Copy Markdown
Owner

@michaelforney michaelforney Apr 8, 2026

Choose a reason for hiding this comment

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

These are probably better to put in your .git/info/exclude instead.

Copy link
Copy Markdown
Author

@cyanidle cyanidle Apr 8, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Agree, will revert later probably

29 changes: 29 additions & 0 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
cmake_minimum_required(VERSION 3.16)
Copy link
Copy Markdown
Owner

@michaelforney michaelforney Apr 8, 2026

Choose a reason for hiding this comment

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

Do you think it's possible to get the Makefile to work with nmake?

Copy link
Copy Markdown
Author

@cyanidle cyanidle Apr 8, 2026

Choose a reason for hiding this comment

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

NMake is not very compatible with Make, easier to maintain both, considering project simplicity

project(samurai C)

set(SOURCE
build.c
deps.c
env.c
graph.c
htab.c
log.c
parse.c
samu.c
scan.c
tool.c
tree.c
util.c
)

if (WIN32)
list(APPEND SOURCE os-win32.c samu.manifest)
elseif(UNIX)
list(APPEND SOURCE os-posix.c)
endif()

add_executable(samu ${SOURCE})

if (UNIX)
target_link_libraries(samu PRIVATE rt)
endif()
2 changes: 1 addition & 1 deletion arg.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ extern const char *argv0;
switch (*opt_)

#define ARGEND \
} \
} \
}

#define EARGF(x) \
Expand Down
Loading