bonsai/harakit
4
4
Fork 3
Code Issues 41 Pull Requests 3 Projects 1 Releases 35 Activity

peek(1) #93

Closed
trinity wants to merge 25 commits from peek into main
pull from: peek
merge into: bonsai:main
Conversation 19 Commits 25 Files Changed 4 +288 -1
trinity commented 2024-04-17 17:18:25 -06:00
Owner
Copy Link

peek(1) is a utility that reads standard input and writes to given outputs without echoing to the terminal. It has a number of features that make safer usage convenient and both the code and the man page are very cleanly written. This tool's utility is obvious in the examples; while shell scripts shouldn't accept secrets, if they were to do so this is a thoughtfully designed utility for that task.

This is imported from my source tree with slight tweaks for this repository. @silt helped review this code when it was first written (four months ago).

mm(1) isn't suited for this task because it doesn't prevent terminal echoing. The shell built-in read doesn't either. The featureset of this utility wouldn't make sense for any other one, either (-1 could be | head -n 1, -e could be | tee /dev/stderr, -n could be | tr -d \\n, -o is a default everywhere, -p is a pipe, and -t doesn't make sense elsewhere).

peek(1) is a utility that reads standard input and writes to given outputs *without* echoing to the terminal. It has a number of features that make safer usage convenient and both the code and the man page are very cleanly written. This tool's utility is obvious in the examples; while shell scripts *shouldn't* accept secrets, if they were to do so this is a thoughtfully designed utility for that task. This is [imported from my source tree](https://git.tebibyte.media/trinity/src/src/branch/main/peek) with slight tweaks for this repository. @silt helped review this code when it was first written (four months ago). mm(1) isn't suited for this task because it doesn't prevent terminal echoing. The shell built-in `read` doesn't either. The featureset of this utility wouldn't make sense for any other one, either (`-1` could be `| head -n 1`, `-e` could be `| tee /dev/stderr`, `-n` could be `| tr -d \\n`, `-o` is a default everywhere, `-p` is a pipe, and `-t` doesn't make sense elsewhere).
trinity added the
enhancement
 label 2024-04-17 17:18:25 -06:00
trinity added 1 commit 2024-04-17 17:18:25 -06:00
trinity requested review from emma 2024-04-17 17:24:29 -06:00
trinity requested review from silt 2024-04-17 17:24:30 -06:00
emma requested changes 2024-04-17 22:34:15 -06:00
docs/peek.1 Outdated
@ -0,0 +19,4 @@
.SH DESCRIPTION
Peek reads input from standard input with terminal echo disabled, which may be
emma commented 2024-04-17 22:30:03 -06:00
Owner
Copy Link

Peek reads input from standard input with terminal echo disabled,

Read input from the standard input with terminal echo disabled.

which may be useful to prevent secrets being spied upon by adversaries watching a user's
screen.

Should this be placed in a RATIONALE section?

> Peek reads input from standard input with terminal echo disabled, Read input from the standard input with terminal echo disabled. > which may be useful to prevent secrets being spied upon by adversaries watching a user's screen. Should this be placed in a RATIONALE section?
emma marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +45,4 @@
.B -o
.RS
Configures peek to output to standard output.
emma commented 2024-04-17 22:30:36 -06:00
Owner
Copy Link

“[...] the standard output”

“[...] the standard output”
trinity marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +48,4 @@
Configures peek to output to standard output.
.RE
.B -p
emma commented 2024-04-17 22:31:40 -06:00
Owner
Copy Link

What is the benefit to this over using a shell pipe?

What is the benefit to this over using a shell pipe?
trinity marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +94,4 @@
.R $ env -i PATH="$PATH" peek -1tp htpasswd -nBi _ | cut -d : -f 2
.RE
This is a POSIX sh(1) command line that lets a user blindly write into a text
emma commented 2024-04-17 22:33:35 -06:00
Owner
Copy Link

Either:

This is a POSIX shell [...]

or:

This is an sh(1p) [...]

Either: > This is a POSIX shell [...] or: > This is an sh(1p) [...]
trinity marked this conversation as resolved
emma commented 2024-04-17 23:05:21 -06:00
Owner
Copy Link

Could this functionality be attained using a combination of existing or hypothetical utilities and shell features? We should consider whether this utility should be necessary for its use case.

Could this functionality be attained using a combination of existing or hypothetical utilities and shell features? We should consider whether this utility should be necessary for its use case.
trinity added 3 commits 2024-04-18 10:04:33 -06:00
trinity commented 2024-04-18 10:08:04 -06:00
Author
Owner
Copy Link

Could this functionality be attained using a combination of existing or hypothetical utilities and shell features? We should consider whether this utility should be necessary for its use case.

Yes, actually. After rethinking it I rewrote the utility to be as simple as possible (and have -t be the default as it should have been). The man page has been updated to match, including incorporating all of your comments which I found to be useful.

> Could this functionality be attained using a combination of existing or hypothetical utilities and shell features? We should consider whether this utility should be necessary for its use case. Yes, actually. After rethinking it I rewrote the utility to be as simple as possible (and have `-t` be the default as it should have been). The man page has been updated to match, including incorporating all of your comments which I found to be useful.
trinity requested review from emma 2024-04-18 10:08:09 -06:00
trinity added 1 commit 2024-04-18 10:15:00 -06:00
emma changed title from peek(1) to WIP: peek(1) 2024-04-24 19:59:50 -06:00
emma changed title from WIP: peek(1) to peek(1) 2024-04-24 19:59:58 -06:00
emma requested changes 2024-05-24 10:37:03 -06:00
docs/peek.1 Outdated
@ -0,0 +27,4 @@
.SH DIAGNOSTICS
Peek prints an error message and exits with the appropriate status from
emma commented 2024-05-24 10:30:44 -06:00
Owner
Copy Link

In the event of an error, a debug message will be printed and the program will
exit with the appropriate sysexits.h(3) error code.

For parity with the man pages in the docs branch. For example, see npc.1.

> In the event of an error, a debug message will be printed and the program will exit with the appropriate sysexits.h(3) error code. For parity with the man pages in the `docs` branch. For example, see [`npc.1`](https://git.tebibyte.media/bonsai/coreutils/src/commit/b7f52902b6bde54876b5fd8efafa433e2cf0df9a/docs/npc.1#L44).
emma marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +30,4 @@
Peek prints an error message and exits with the appropriate status from
sysexits.h(3) if it encounters an unrecoverable error.
Peek tries to handle the signal SIGINT (^C) to ensure the user's terminal is
emma commented 2024-05-24 10:33:18 -06:00
Owner
Copy Link

If it encounters a SIGINT signal (^C), the program will attempt to handle it to ensure the user’s terminal is still usable after premature termination; if the signal can’t be handled, it prints an error message and continues. If the program is interrupted, it exits unsuccessfully and without printing an error message.

If it encounters a SIGINT signal (^C), the program will attempt to handle it to ensure the user’s terminal is still usable after premature termination; if the signal can’t be handled, it prints an error message and continues. If the program is interrupted, it exits unsuccessfully and without printing an error message.
emma marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +44,4 @@
.SH BUGS
This does nothing to prevent others seeing the keyboard being used to input
emma commented 2024-05-24 10:27:14 -06:00
Owner
Copy Link

This being in the BUGS section implies that this can be fixed by the program. Therefore, this belongs in a CAVEATS section because it is definitionally a caveat.

This being in the BUGS section implies that this can be fixed by the program. Therefore, this belongs in a CAVEATS section because it is definitionally a caveat.
emma marked this conversation as resolved
docs/peek.1 Outdated
@ -0,0 +69,4 @@
This is an sh(1p) command line that lets a user blindly write into a text file,
only able to see written lines. Some writers have the habit of prematurely
revising their work and use tools like this to prevent it. It uses mm(1) to
pipe the output of peek to both the standard error and the regular file
emma commented 2024-05-24 10:36:38 -06:00
Owner
Copy Link

It uses mm(1) to pipe the hidden input to both the standard error and the regular file writing.txt.

> It uses mm(1) to pipe the hidden input to both the standard error and the regular file writing.txt.
emma marked this conversation as resolved
emma commented 2024-06-09 00:14:10 -06:00
Owner
Copy Link

Here is the sum of the changes I recommend to the peek.1 manpage as a git diff:

diff --git a/docs/peek.1 b/docs/peek.1
index 1c482a2..ec56a8e 100644
--- a/docs/peek.1
+++ b/docs/peek.1
@@ -1,90 +1,111 @@
 .\" Copyright (c) 2023-2024 DTB <trinity@trinity.moe>
+.\" Copyright (c) 2024 Emma Tebibyte <emma@tebibyte.media>
 .\"
 .\" This work is licensed under CC BY-SA 4.0. To see a copy of this license,
 .\" visit <http://creativecommons.org/licenses/by-sa/4.0/>.
-
-.TH PEEK 1
-
+.\"
+.TH PEEK 1 2024-06-09 "Bonsai Core Utilites 0.13.9"
 .SH NAME
-
 peek \(en read from the standard input, furtively
-
+.\"
 .SH SYNOPSIS
 
 peek
 .RB ( -i )
-
+.\"
 .SH DESCRIPTION
 
-Peek reads input from the standard input with terminal echo disabled.
-
+Read input from the standard input with terminal echo disabled.
+.\"
 .SH OPTIONS
 
-.B -i
-.RS
-Allows input to come from sources other than terminals (i.e. a pipe).
-.RE
-
+.IP \fB-i\fP
+Allows input to come from sources other than terminals (i.e., a pipe).
+.\"
 .SH DIAGNOSTICS
 
-Peek prints an error message and exits with the appropriate status from
-sysexits.h(3) if it encounters an unrecoverable error.
-
-Peek tries to handle the signal SIGINT (^C) to ensure the user's terminal is
-still usable after premature termination; if the signal can't be handled, it
-prints an error message and continues. If peek is interrupted, it exits
-unsuccessfully, without an error message.
+In the event of an error, a debug message will be printed and the program will
+exit with the appropriate
+.BR sysexits.h (3)
+error code.
 
+In order to ensure the user\(cqs terminal is still usable after premature
+termination, the program attempts to handle the SIGINT signal; if it cannot,
+an error message is printed and execution continues. If the program is
+interrupted, it exits unsuccessfully without an error message.
+.\"
 .SH RATIONALE
 
-This tool was originally written to accept passwords in shell scripts, as an
-extremely simple alternative to the GNU Privacy Guard project's pinentry(1).
-Accepting input without showing what is being typed is useful when keying in
-secrets in public settings or places with installed surveillance cameras.
-
-.SH BUGS
+This tool was originally written to accept passwords in shell scripts as an
+extremely simple alternative to the GNU Privacy Guard project\(cqs
+.BR pinentry (1)
+utility.
 
-This does nothing to prevent others seeing the keyboard being used to input
-secrets or mask the sound of typing. Audio or video recordings of typing can be
-used to determine what was input without needing to see the characters appear
-on the screen.
+Accepting input without showing what is being typed is useful when keying in
+secrets in public settings or in places where surveillance cameras are
+installed.
+.\"
+.SH CAVEATS
 
-Accepting secrets in shell scripts is probably not adviseable.
+This program does nothing to prevent others from seeing the key presses inputted
+to a keyboard. It also does not protect against the sound of typing being
+analyzed to determine what was inputted without needing to see screen or
+keyboard.
 
-On systems that support it, the ioctl(2) command TIOCSTI can be used to insert
-characters into the standard input going to peek. This doesn't allow snooping
-but can be used for general mischief.
+Accepting secrets in shell scripts is probably not advisable.
 
+On systems that support it, the
+.BR ioctl (2)
+command TIOCSTI can be used to insert characters into the standard input. This
+doesn't allow snooping but can be used for general mischief.
+.\"
 .SH EXAMPLES
 
-This is an sh(1p) command line that hashes a given password. It uses head(1p)
-to only accept one line of input, xargs(1p) and printf(1p) to strip the
-trailing newline, htpasswd(1) from Apache's utilities to hash the input with
-the bcrypt algorithm, and cut(1p) to print only the resulting hash:
+This is an
+.BR sh (1p)
+command line that hashes a given password. It uses
+.BR head (1p)
+to only accept one line of input,
+.BR xargs (1p)
+and
+.BR printf (1p)
+to strip the trailing newline,
+.BR htpasswd (1) 
+from Apache\(cqs utilities to hash the input with the bcrypt algorithm, and
+.BR cut (1p)
+to print only the resulting hash:
 
 .RS
-.R $ peek | head -n 1 | xargs printf '%s' | htpasswd -nBi _ | cut -d : -f 2
+$ peek | head -n 1 | xargs printf '%s' | htpasswd -nBi _ | cut -d : -f 2
 .RE
 
-This is an sh(1p) command line that lets a user blindly write into a text file,
-only able to see written lines. Some writers have the habit of prematurely
-revising their work and use tools like this to prevent it. It uses mm(1) to
-pipe the output of peek to both the standard error and the regular file
-writing.txt.
+This is an
+.BR sh (1p)
+command line that allows a user to write blindly into a text file but displaying
+only written lines. Some writers have the habit of prematurely revising their 
+work and use tools with functionality similar to this to prevent it. 
+It uses
+.BR mm (1)
+to pipe the output of the program to both the standard error and the regular
+file writing.txt:
 
 .RS
-.R $ echo Input ^D to quit. && peek | mm -eo - >writing.txt
+$ echo Input ^D to quit. && peek | mm -eo - >writing.txt
 .RE
-
+.\"
 .SH AUTHOR
 
-Written by DTB <trinity@trinity.moe>.
-
+Written by DTB
+.MT trinity@trinity.moe
+.ME .
+.\"
 .SH COPYRIGHT
 
-Copyright (c) 2023-2024 DTB. License AGPLv3+: GNU AGPL version 3 or later
+Copyright \(co 2023-2024 DTB. License AGPLv3+: GNU AGPL version 3 or later
 <https://gnu.org/licenses/gpl.html>.
-
+.\"
 .SH SEE ALSO
-
-ioctl(2), ioctl_tty(2), read(1), sh(1)
+.BR ioctl (2),
+.BR ioctl_tty (2),
+.BR read (1),
+.BR sh (1)
Here is the sum of the changes I recommend to the `peek.1` manpage as a git diff: ``` diff --git a/docs/peek.1 b/docs/peek.1 index 1c482a2..ec56a8e 100644 --- a/docs/peek.1 +++ b/docs/peek.1 @@ -1,90 +1,111 @@ .\" Copyright (c) 2023-2024 DTB <trinity@trinity.moe> +.\" Copyright (c) 2024 Emma Tebibyte <emma@tebibyte.media> .\" .\" This work is licensed under CC BY-SA 4.0. To see a copy of this license, .\" visit <http://creativecommons.org/licenses/by-sa/4.0/>. - -.TH PEEK 1 - +.\" +.TH PEEK 1 2024-06-09 "Bonsai Core Utilites 0.13.9" .SH NAME - peek \(en read from the standard input, furtively - +.\" .SH SYNOPSIS peek .RB ( -i ) - +.\" .SH DESCRIPTION -Peek reads input from the standard input with terminal echo disabled. - +Read input from the standard input with terminal echo disabled. +.\" .SH OPTIONS -.B -i -.RS -Allows input to come from sources other than terminals (i.e. a pipe). -.RE - +.IP \fB-i\fP +Allows input to come from sources other than terminals (i.e., a pipe). +.\" .SH DIAGNOSTICS -Peek prints an error message and exits with the appropriate status from -sysexits.h(3) if it encounters an unrecoverable error. - -Peek tries to handle the signal SIGINT (^C) to ensure the user's terminal is -still usable after premature termination; if the signal can't be handled, it -prints an error message and continues. If peek is interrupted, it exits -unsuccessfully, without an error message. +In the event of an error, a debug message will be printed and the program will +exit with the appropriate +.BR sysexits.h (3) +error code. +In order to ensure the user\(cqs terminal is still usable after premature +termination, the program attempts to handle the SIGINT signal; if it cannot, +an error message is printed and execution continues. If the program is +interrupted, it exits unsuccessfully without an error message. +.\" .SH RATIONALE -This tool was originally written to accept passwords in shell scripts, as an -extremely simple alternative to the GNU Privacy Guard project's pinentry(1). -Accepting input without showing what is being typed is useful when keying in -secrets in public settings or places with installed surveillance cameras. - -.SH BUGS +This tool was originally written to accept passwords in shell scripts as an +extremely simple alternative to the GNU Privacy Guard project\(cqs +.BR pinentry (1) +utility. -This does nothing to prevent others seeing the keyboard being used to input -secrets or mask the sound of typing. Audio or video recordings of typing can be -used to determine what was input without needing to see the characters appear -on the screen. +Accepting input without showing what is being typed is useful when keying in +secrets in public settings or in places where surveillance cameras are +installed. +.\" +.SH CAVEATS -Accepting secrets in shell scripts is probably not adviseable. +This program does nothing to prevent others from seeing the key presses inputted +to a keyboard. It also does not protect against the sound of typing being +analyzed to determine what was inputted without needing to see screen or +keyboard. -On systems that support it, the ioctl(2) command TIOCSTI can be used to insert -characters into the standard input going to peek. This doesn't allow snooping -but can be used for general mischief. +Accepting secrets in shell scripts is probably not advisable. +On systems that support it, the +.BR ioctl (2) +command TIOCSTI can be used to insert characters into the standard input. This +doesn't allow snooping but can be used for general mischief. +.\" .SH EXAMPLES -This is an sh(1p) command line that hashes a given password. It uses head(1p) -to only accept one line of input, xargs(1p) and printf(1p) to strip the -trailing newline, htpasswd(1) from Apache's utilities to hash the input with -the bcrypt algorithm, and cut(1p) to print only the resulting hash: +This is an +.BR sh (1p) +command line that hashes a given password. It uses +.BR head (1p) +to only accept one line of input, +.BR xargs (1p) +and +.BR printf (1p) +to strip the trailing newline, +.BR htpasswd (1) +from Apache\(cqs utilities to hash the input with the bcrypt algorithm, and +.BR cut (1p) +to print only the resulting hash: .RS -.R $ peek | head -n 1 | xargs printf '%s' | htpasswd -nBi _ | cut -d : -f 2 +$ peek | head -n 1 | xargs printf '%s' | htpasswd -nBi _ | cut -d : -f 2 .RE -This is an sh(1p) command line that lets a user blindly write into a text file, -only able to see written lines. Some writers have the habit of prematurely -revising their work and use tools like this to prevent it. It uses mm(1) to -pipe the output of peek to both the standard error and the regular file -writing.txt. +This is an +.BR sh (1p) +command line that allows a user to write blindly into a text file but displaying +only written lines. Some writers have the habit of prematurely revising their +work and use tools with functionality similar to this to prevent it. +It uses +.BR mm (1) +to pipe the output of the program to both the standard error and the regular +file writing.txt: .RS -.R $ echo Input ^D to quit. && peek | mm -eo - >writing.txt +$ echo Input ^D to quit. && peek | mm -eo - >writing.txt .RE - +.\" .SH AUTHOR -Written by DTB <trinity@trinity.moe>. - +Written by DTB +.MT trinity@trinity.moe +.ME . +.\" .SH COPYRIGHT -Copyright (c) 2023-2024 DTB. License AGPLv3+: GNU AGPL version 3 or later +Copyright \(co 2023-2024 DTB. License AGPLv3+: GNU AGPL version 3 or later <https://gnu.org/licenses/gpl.html>. - +.\" .SH SEE ALSO - -ioctl(2), ioctl_tty(2), read(1), sh(1) +.BR ioctl (2), +.BR ioctl_tty (2), +.BR read (1), +.BR sh (1) ```
trinity commented 2024-06-09 10:52:20 -06:00
Author
Owner
Copy Link

Could "inputted" be "input"? Otherwise I like these changes and you may merge them into the peek branch, or I can myself when I'm at keyboard.

Could "inputted" be "input"? Otherwise I like these changes and you may merge them into the peek branch, or I can myself when I'm at keyboard.
emma added 1 commit 2024-06-17 22:29:45 -06:00
silt reviewed 2024-06-17 22:43:04 -06:00
docs/peek.1 Outdated
@ -0,0 +20,4 @@
.SH OPTIONS
.IP \fB-i\fP
Allows input to come from sources other than terminals (i.e., a pipe).
silt commented 2024-06-17 22:43:04 -06:00
Owner
Copy Link

This appears to imply that there exists a multitude of other possible sources, but it's not made clear what those are or how to make use of them. Does it support reading input from… files? TCP sockets? IRC messages??

This appears to imply that there exists a multitude of other possible sources, but it's not made clear what those are or how to make use of them. Does it support reading input from… files? TCP sockets? IRC messages??
trinity commented 2024-06-26 09:37:45 -06:00
Author
Owner
Copy Link

The "i.e." stands for "id est" and can be read as "in other words". Perhaps I should use the English. I mean to say that the source other than a terminal, in this case, is a pipe.

The "i.e." stands for "id est" and can be read as "in other words". Perhaps I should use the English. I mean to say that the source other than a terminal, in this case, is a pipe.
trinity commented 2024-06-26 09:57:33 -06:00
Author
Owner
Copy Link

Changed to (pipes) which I think is more clear.

Changed to (pipes) which I think is more clear.
trinity marked this conversation as resolved
emma commented 2024-06-17 22:46:17 -06:00
Owner
Copy Link

I’m still not convinced that this should be a tool instead of a shell feature, like some sort of set analogue that lets you disable echoing.

I’m still not convinced that this should be a tool instead of a shell feature, like some sort of `set` analogue that lets you disable echoing.
trinity commented 2024-06-26 09:34:37 -06:00
Author
Owner
Copy Link

I’m still not convinced that this should be a tool instead of a shell feature, like some sort of set analogue that lets you disable echoing.

It's about the fundamentals.

  • This feature will be available in whatever shell the user wants.
  • Keeping this functionality in a separate program will make it easier to debug it separately.
  • No functionality is lost by making this a separate program.

A program should do one thing well. peek(1) does one thing well. Mixing its functionality into another program is unnecessary, makes it generally less valuable, and makes the other program more complex.

> I’m still not convinced that this should be a tool instead of a shell feature, like some sort of `set` analogue that lets you disable echoing. It's about the fundamentals. - This feature will be available in whatever shell the user wants. - Keeping this functionality in a separate program will make it easier to debug it separately. - No functionality is lost by making this a separate program. A program should do one thing well. peek(1) does one thing well. Mixing its functionality into another program is unnecessary, makes it generally less valuable, and makes the other program more complex.
trinity added 1 commit 2024-06-26 09:43:15 -06:00
trinity added 1 commit 2024-06-26 09:46:38 -06:00
trinity added 1 commit 2024-06-26 09:55:45 -06:00
trinity commented 2024-06-26 09:56:51 -06:00
Author
Owner
Copy Link

My git client was misconfigured; I've signed a child commit to endorse the previous commits, but let me know if there's anything else I should do.

My git client was misconfigured; I've signed a child commit to endorse the previous commits, but let me know if there's anything else I should do.
emma commented 2024-06-26 13:49:04 -06:00
Owner
Copy Link

My git client was misconfigured; I've signed a child commit to endorse the previous commits, but let me know if there's anything else I should do.

You’re good, the merge commit will be signed anyway

> My git client was misconfigured; I've signed a child commit to endorse the previous commits, but let me know if there's anything else I should do. You’re good, the merge commit will be signed anyway
emma commented 2024-06-26 13:49:35 -06:00
Owner
Copy Link

I’m still not convinced that this should be a tool instead of a shell feature, like some sort of set analogue that lets you disable echoing.

It's about the fundamentals.

  • This feature will be available in whatever shell the user wants.
  • Keeping this functionality in a separate program will make it easier to debug it separately.
  • No functionality is lost by making this a separate program.

A program should do one thing well. peek(1) does one thing well. Mixing its functionality into another program is unnecessary, makes it generally less valuable, and makes the other program more complex.

After some thought, I agree with this.

> > I’m still not convinced that this should be a tool instead of a shell feature, like some sort of `set` analogue that lets you disable echoing. > > It's about the fundamentals. > > - This feature will be available in whatever shell the user wants. > - Keeping this functionality in a separate program will make it easier to debug it separately. > - No functionality is lost by making this a separate program. > > A program should do one thing well. peek(1) does one thing well. Mixing its functionality into another program is unnecessary, makes it generally less valuable, and makes the other program more complex. After some thought, I agree with this.
emma reviewed 2024-06-26 17:03:05 -06:00
docs/peek.1 Outdated
@ -0,0 +4,4 @@
.\" This work is licensed under CC BY-SA 4.0. To see a copy of this license,
.\" visit <http://creativecommons.org/licenses/by-sa/4.0/>.
.\"
.TH PEEK 1 2024-06-09 "Bonsai Core Utilites 0.13.9"
emma commented 2024-06-26 17:03:05 -06:00
Owner
Copy Link

.TH PEEK 1 [whenever you make the change] "Harakit X.X.X"

> .TH PEEK 1 [whenever you make the change] "Harakit X.X.X"
trinity marked this conversation as resolved
trinity added 2 commits 2024-06-28 08:37:59 -06:00
trinity added 1 commit 2024-06-28 08:40:40 -06:00
trinity requested review from emma 2024-06-28 08:41:21 -06:00
emma added a new dependency 2024-06-29 22:02:18 -06:00
#96 testing
trinity commented 2024-07-26 07:22:32 -06:00
Author
Owner
Copy Link

I can't find a way to rewrite this in Rust without vendoring a crate or writing termios C bindings.

I can't find a way to rewrite this in Rust without vendoring a crate or writing termios C bindings.
trinity added 1 commit 2024-07-26 07:42:18 -06:00
trinity added 1 commit 2024-07-26 07:58:46 -06:00
trinity added 1 commit 2024-07-27 18:47:02 -06:00
trinity added 1 commit 2024-07-27 18:51:15 -06:00
trinity added 1 commit 2024-08-14 11:06:46 -06:00
trinity commented 2024-08-14 11:07:55 -06:00
Author
Owner
Copy Link

I didn't know about stty(1p) when I wrote this utility but I think for this purpose peek(1) does a better job.

I didn't know about stty(1p) when I wrote this utility but I think for this purpose peek(1) does a better job.
trinity added 1 commit 2024-08-27 21:16:27 -06:00
emma reviewed 2024-08-28 03:25:37 -06:00
src/peek.c Outdated
@ -0,0 +64,4 @@
#ifdef __OpenBSD__
pledge("stdio tty unveil", NULL);
unveil(NULL, NULL);
#endif
emma commented 2024-08-28 03:25:37 -06:00
Owner
Copy Link

Remember to handle the possible errors here!

Remember to handle the possible errors here!
trinity marked this conversation as resolved
trinity added 1 commit 2024-08-28 18:18:14 -06:00
trinity added 2 commits 2024-08-28 18:35:56 -06:00
trinity added 1 commit 2024-08-30 21:29:25 -06:00
trinity added 3 commits 2024-09-01 09:24:25 -06:00
silt approved these changes 2024-09-01 19:06:13 -06:00
silt left a comment
Owner
Copy Link

looks fine to me

looks fine to me
trinity commented 2024-09-01 19:31:50 -06:00
Author
Owner
Copy Link

I requested (in private messages) silt's review because a number of changes were required last-minute to make testing work (peek -i </dev/null >/dev/null would fail).

I requested (in private messages) silt's review because a number of changes were required last-minute to make testing work (`peek -i </dev/null >/dev/null` would fail).
trinity commented 2024-09-01 19:51:52 -06:00
Author
Owner
Copy Link

Merged (via cherry-picking to avoid redundant commits).

Merged (via cherry-picking to avoid redundant commits).
trinity closed this pull request 2024-09-04 20:54:33 -06:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
emma
silt
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
joke
you are silly!
question
More information is needed
wontfix
This won't be fixed
No Label
enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
emma nitori silt trinity
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Depends on
#96 testing
bonsai/harakit
Reference: bonsai/harakit#93
No description provided.
Delete Branch "peek"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?