fix: use portable ioctl request codes

[rymdbar]

While Zig's std does not set these variants for darwin under std.os., they are available with the exact same values under std.c. Thus there seems to be no reason to not use the portable constants.

[rymdbar]

Pull request updated with integration test change.

Running zig build test, zig build test-integration and zig build test-all completes successfully without any output on FreeBSD 15.0-RELEASE-p10.

[rymdbar]

I see the build error and am a bit embarrassed.

src/pty.zig:15:43: error: struct 'c.T__struct_22974' has no member named 'IOCSWINSZ'
    pub const IOCSWINSZ: c_ulong = std.c.T.IOCSWINSZ;

Turns out the Apple specific code was needed, precisely like the comment said. I was sloppy to assume that std.c.T… contained an independent set of definitions, when actually reading the source shows a slightly more interesting situation. A rookie mistake, I've never seen Zig before.

An updated branch has been pushed.

[kylecarbs]

@rymdbar no biggie at all! Looking again now.

[rymdbar]

Oh. I forgot about the integration tests duplicating the constants, and didn't realize to run zigfmt. Will push an update in a few minutes.

[rymdbar]

I'm having trouble with integration tests failing, even with the same commit which I experienced was working earlier. My train is arriving its station now, so I'll need to get back to it later.

[rymdbar]

Seems my problem with integration test was that I accidentally ran them inside another boo session. When running them in a plain terminal, they work. 🙂

[kylecarbs]

@rymdbar heh ;p

[rymdbar]

Repushed unchanged, after being rebased onto latest main.

[kylecarbs]

Once CI passes I'll merge!