Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: don't attempt to write and stat the lock file during dry runs #2440

Merged
merged 1 commit into from
May 6, 2024

Conversation

jdrouhard
Copy link
Contributor

Fixes #2431

@jhasse
Copy link
Collaborator

jhasse commented May 6, 2024

Thanks! We're really should have more (integration) tests for this stuff ...

@jhasse jhasse added this to the 1.13.0 milestone May 6, 2024
@jhasse jhasse merged commit 554579d into ninja-build:master May 6, 2024
11 checks passed
@jhasse jhasse modified the milestones: 1.13.0, 1.12.1 May 11, 2024
@jdrouhard jdrouhard deleted the dry-run-fix branch May 13, 2024 19:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dry run (-n) error if the builddir directory does not exist in 1.12.0
2 participants