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.
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
windows: make more windows tests pass #9370
Conversation
❌ @paperdave 3 files with test failures on linux-x64-baseline: |
❌ @paperdave 2 files with test failures on linux-x64: |
❌ @paperdave 5 files with test failures on bun-darwin-aarch64: |
❌🪟 @paperdave, there are 14 test regressions on Windows x86_64
|
❌ @paperdave 3 files with test failures on bun-darwin-x64: |
// powershell `install.ps1` was used to install. | ||
|
||
const image_path = bun.windows.exePathW(); | ||
const image_dirname = image_path[0..(std.mem.lastIndexOfScalar(u16, image_path, '\\') orelse unreachable)]; | ||
|
||
if (!std.mem.endsWith(u16, image_dirname, comptime bun.strings.literal(u16, "\\bun\\bin"))) | ||
if (!std.mem.endsWith(u16, image_dirname, comptime bun.strings.literal(u16, "bun\\bin"))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.bun\bin
but also handle some form of user-install where it is organized in a folder named bun
const dir = try std.fs.cwd().makeOpenPath(dirname, .{ .access_sub_paths = true }); | ||
break :brk bun.toLibUVOwnedFD(dir.fd); | ||
var dir = try std.fs.cwd().makeOpenPath(dirname, .{ .access_sub_paths = true }); | ||
errdefer dir.close(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. There's a build failure but the code is unreachable
What tests does this fix? |
@@ -17,7 +17,6 @@ | |||
const { chmod, copyFile, lstat, mkdir, opendir, readlink, stat, symlink, unlink, utimes } = require("node:fs/promises"); | |||
const { dirname, isAbsolute, join, parse, resolve, sep } = require("node:path"); | |||
|
|||
const SafePromiseAll = Promise.all; | |||
const PromisePrototypeThen = Promise.prototype.then; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The hilarious thing about this is .$then
exists and so it's less safe than the actual
* run-eval.test.ts * transpiler-cache.test.ts * node net * some open things * a * a * yikes * incredible * run it back * a * this code is what i like to call, incorrect * ok its all worng * remove an assertion that is wrong again * update test things and rebase * performance test * mark filesink with mkfifo as todo. see #8166 * hehe * not done * awa * fs test pass --------- Co-authored-by: Jarred Sumner <[email protected]> Co-authored-by: Jarred Sumner <[email protected]>
some changes to moving spawn({ cmd: [] }) to spawn([], {}) are done because our workspaces type definitions are broken :(
making use our
open
in more places than before because i think our synchronous open is better than libuv'sFixes #6820