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.

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

Jump to bottom

compare modified time when running source commands with last compiled binary #66

Closed
progrium opened this issue Jan 23, 2024 · 0 comments · Fixed by #102
Closed

compare modified time when running source commands with last compiled binary #66

progrium opened this issue Jan 23, 2024 · 0 comments · Fixed by #102
Assignees
@Parzival-3141
Labels
good first issue Good for newcomers
Milestone
0.2

Comments

@progrium
Copy link
Contributor

When you run a command that is a source directory, Wanix builds the program and caches it in /sys/bin. If it exists there it will run that binary next time. However, if you make changes to the source, it should recompile next time you run it. Currently it will just use the existing binary. We can do this by checking the creation time of the binary against the modified time of the source files, similar to make.
@progrium progrium added the good first issue Good for newcomers label Jan 23, 2024
@progrium progrium added this to the 0.2 milestone Jan 23, 2024
@progrium progrium changed the title check modified time when running source commands to recompile compare modified time when running source commands with last compiled binary Jan 23, 2024
@Parzival-3141 Parzival-3141 mentioned this issue Jan 24, 2024
Closed
Parzival-3141 added a commit that referenced this issue Feb 8, 2024
@Parzival-3141
kernel: compare mtime's when building source or copying files from …
42dcc87 
…`initfs`

Closes #66

Enables proper live editing within Wanix! `proc.Spawn` will rebuild commands
from source if it's been modified, and `kernel.fs` will copy in `initfs` files
if they've been modified externally *(note this will overwrite local files
inside Wanix)*.
@Parzival-3141 Parzival-3141 mentioned this issue Feb 8, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Parzival-3141 Parzival-3141

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
0.2
Development

Successfully merging a pull request may close this issue.

Include shell source and build source directories in proc.Spawn tractordev/wanix
2 participants
@progrium @Parzival-3141