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

zig build test-cases --watch does not pick up changes to source files #20606

Open
andrewrk opened this issue Jul 13, 2024 · 0 comments
Open

zig build test-cases --watch does not pick up changes to source files #20606

andrewrk opened this issue Jul 13, 2024 · 0 comments
Labels
bug Observed behavior contradicts documented or intended behavior contributor friendly This issue is limited in scope and/or knowledge of Zig internals. enhancement Solving this issue will likely involve adding new logic or components to the codebase. zig build system std.Build, the build runner, `zig build` subcommand, package management
Milestone
0.14.0

Comments

@andrewrk
Copy link
Member

Extracted from #20580.

Currently the test-cases test suite walks the filesystem during the configure phase, loads the source files into memory, and adds WriteFile steps to write out the source code always to tmp.zig files, using those files as the inputs to Compile steps.

This issue is to rework the test cases to create the Compile steps directly using the source files from test/cases/*. This will make the file system watching mechanism pick up the changes properly.

Related:
@andrewrk andrewrk added bug Observed behavior contradicts documented or intended behavior enhancement Solving this issue will likely involve adding new logic or components to the codebase. contributor friendly This issue is limited in scope and/or knowledge of Zig internals. zig build system std.Build, the build runner, `zig build` subcommand, package management labels Jul 13, 2024
@andrewrk andrewrk added this to the 0.14.0 milestone Jul 13, 2024
@andrewrk andrewrk mentioned this issue Jul 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Observed behavior contradicts documented or intended behavior contributor friendly This issue is limited in scope and/or knowledge of Zig internals. enhancement Solving this issue will likely involve adding new logic or components to the codebase. zig build system std.Build, the build runner, `zig build` subcommand, package management
Projects
None yet
Milestone
0.14.0
Development

No branches or pull requests
1 participant
@andrewrk