sanity_checks: fix sanity check on Windows
Metadata
Current evaluation
Merged PR #2919 fixed Windows sanity checks for expected snap directories by allowing backslash paths; landed as commit cc40d5f68.
Resolution: closed resolved
Reason: PR #2919 was approved and merged, correcting expected snap-directory validation on Windows. The change landed as commit cc40d5f68.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh tests/unit`?
-----
The sanity checks for expected snap directories were not checking to see if the path had backslashes in it, so the sanity check was failing on Windows. This fixes it.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2919 fixed Windows sanity checks for expected snap directories by allowing backslash paths; landed as commit cc40d5f68. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2919 fixed snap directory sanity checks on Windows by allowing backslashes in expected paths; landed as commit cc40d5f68. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for Windows sanity check failures in snap directory validation. Updated path checks to handle backslashes correctly. Approved by reviewers, passed CI, and integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for Windows-specific sanity check failures. The update ensures snap directory path validation correctly handles backslashes, resolving test failures on Windows systems. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2279 sanity checks: run properly on build VMs | snapcraft | merged | Merged snapcraft#2279 in commit 6f4ca585f, making snap-directory sanity checks project-relative instead of CWD-relative so they work on build VMs (LP #1792962). | |
| #2268 sanity checks: allow snap/local dir | snapcraft | merged | Merged PR #2268 via commit 10bdb0473 allowed snapcraft-specific files under snap/local/ and added tests, resolving LP #1792203. | |
| #2949 cli: fix clean on Windows | snapcraft | merged | Merged PR #2949 fixed the clean command on Windows in commit 4e19c6174 by avoiding unavailable snapd and CPU-affinity calls. |