~sircmpwn/hare-dev

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
2 2

[PATCH hare] os+freebsd: Do not invert CLOEXEC/CTTY twice

Details
Message ID
<20230921110352.3912-1-yyp@disroot.org>
DKIM signature
pass
Download raw message
Patch: +0 -2
It's already inverted in fsflags_to_bsd and shouldn't be inverted once again in
open or create.

Signed-off-by: Alexey Yerin <yyp@disroot.org>
---
 os/+freebsd/dirfdfs.ha | 2 --
 1 file changed, 2 deletions(-)

diff --git a/os/+freebsd/dirfdfs.ha b/os/+freebsd/dirfdfs.ha
index 637fa51c..60acdd39 100644
--- a/os/+freebsd/dirfdfs.ha
+++ b/os/+freebsd/dirfdfs.ha
@@ -174,7 +174,6 @@ fn fs_open_file(
	for (let i = 0z; i < len(flags); i += 1z) {
		oflags |= flags[i];
	};
	oflags ^= fs::flag::CTTY | fs::flag::NOCLOEXEC; // invert NOCTTY/CLOEXEC
	return _fs_open(fs, path, fsflags_to_bsd(oflags), 0);
};

@@ -197,7 +196,6 @@ fn fs_create_file(
	for (let i = 0z; i < len(flags); i += 1z) {
		oflags |= flags[i];
	};
	oflags ^= fs::flag::CTTY | fs::flag::NOCLOEXEC; // invert NOCTTY/CLOEXEC
	oflags |= fs::flag::CREATE;
	return _fs_open(fs, path, fsflags_to_bsd(oflags), mode)?;
};
-- 
2.42.0

[hare/patches] build success

builds.sr.ht <builds@sr.ht>
Details
Message ID
<CVOJOPLYGL02.I2AMP8H9T51Y@cirno2>
In-Reply-To
<20230921110352.3912-1-yyp@disroot.org> (view parent)
DKIM signature
missing
Download raw message
hare/patches: SUCCESS in 1m34s

[os+freebsd: Do not invert CLOEXEC/CTTY twice][0] from [Alexey Yerin][1]

[0]: https://lists.sr.ht/~sircmpwn/hare-dev/patches/44920
[1]: yyp@disroot.org

✓ #1060732 SUCCESS hare/patches/freebsd.yml https://builds.sr.ht/~sircmpwn/job/1060732
✓ #1060731 SUCCESS hare/patches/alpine.yml  https://builds.sr.ht/~sircmpwn/job/1060731
Details
Message ID
<CVOLPXADWT1N.5OZVJOZ1FNG8@taiga>
In-Reply-To
<20230921110352.3912-1-yyp@disroot.org> (view parent)
DKIM signature
pass
Download raw message
Thanks!

To git@git.sr.ht:~sircmpwn/hare
   4e3586a1..cb2477b0  master -> master
Reply to thread Export thread (mbox)