In D28402#714642, @bapt wrote:What would be the point for it to be an option? I think if it is safe and faster we should just use that code, instead of complexifying diff code with 2 differents code path
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Aug 25 2021
Aug 25 2021
diff3: document and test -T
Could we make this (mmap) into an option?
Aug 24 2021
Aug 24 2021
Apart from inline comments I would like to ask you to consider adding at least one test case.
Aug 23 2021
Aug 23 2021
Like in another differential, I'm making a style-related note from a quick glance just to subscribe to this review. But I want to test this more thoroughly.
I want to test this soon, but for now I'll just make a style-related note.
pstef committed rG3cbf98e2bee9: diff: read whole files to determine if they are ASCII text (authored by pstef).
diff: read whole files to determine if they are ASCII text
diff3: implement --strip-trailing-cr
diff3.1: update manual page to match code
diff3: improve style
diff3: sync with upstream
Register /usr/tests/usr.bin/diff3
pstef committed rGaf2f01643184: diff: don't output carriage returns that were stripped on input (authored by pstef).
diff: don't output carriage returns that were stripped on input
Aug 22 2021
Aug 22 2021
pstef committed rG108633bb1556: diff: Use unprivileged_user with report_identical test (authored by olivier).
diff: Use unprivileged_user with report_identical test
Improve --strip-trailing-cr implementation
add a test case
register diff3 tests with mtree
diff --git a/usr.bin/diff/diffreg.c b/usr.bin/diff/diffreg.c index 651ec88df90..113b7b62125 100644 --- a/usr.bin/diff/diffreg.c +++ b/usr.bin/diff/diffreg.c @@ -1273,7 +1273,16 @@ fetch(long *f, int a, int b, FILE *lb, int ch, int oldfile, int flags) } col = 0; for (j = 0, lastc = '\0'; j < nc; j++, lastc = c) { - if ((c = getc(lb)) == EOF) { + c = getc(lb); + if (flags & D_STRIPCR && c == '\r') { + if ((c = getc(lb)) == '\n') + j++; + else { + ungetc(c, lb); + c = '\r'; + } + } + if (c == EOF) { if (diff_format == D_EDIT || diff_format == D_REVERSE || diff_format == D_NREVERSE)
Aug 21 2021
Aug 21 2021
Add --strip-trailing-cr
Aug 20 2021
Aug 20 2021
Include a commit that I omitted by mistake.
Aug 19 2021
Aug 19 2021
pstef committed rGced2dcadccfc: ls: prevent no-color build from complaining when COLORTERM is non-empty (authored by pstef).
ls: prevent no-color build from complaining when COLORTERM is non-empty
In D29493#712586, @pstef wrote:In D29493#712580, @bdrewery wrote:By the way it does not build with -DNO_HISTORY.
That's because of D29361, not this one. Will work on it, thank you for the report.
sh: fix NO_HISTORY build
Aug 18 2021
Aug 18 2021
In D29493#712580, @bdrewery wrote:By the way it does not build with -DNO_HISTORY.
Aug 17 2021
Aug 17 2021
pstef committed rGf49931c1423e: style.9: remove an outdated comment about indent(1) (authored by pstef).
style.9: remove an outdated comment about indent(1)
mount.h: improve a comment about flags
fstyp: add BeFS support
Aug 16 2021
Aug 16 2021
How do I obtain befs.img.bz2? I can see it neither in tree nor in this patch.
Aug 15 2021
Aug 15 2021
pstef added a comment to D18921: Prevent buffer overrun in tzsetup (for -C with overly large parameter).
This is a truncation that in my opinion should result in program error.
Also note that for a non-chrooted run the paths could already be in the arrays.
So I would do it more or less like below (untested, compiled):
82455a3319a9 corrected the word but the space is still there.
I tested this and it works for me. The patch as a whole will look good to me when synopsis is updated to reflect the change.
Consider adding a cross reference to re_format.
pstef added a comment to D13965: Bug 217149 - seq(1) inconsistently omits 'last' when using float increment .
This was fixed by 3049d4cc (r330086).
pstef closed D2266: I can find no reason to allow packets with both SYN and FIN bits set
past this point in the code. The packet should be dropped and not massaged
as it is here..
This has been committed as 916e17fd5678.
Aug 14 2021
Aug 14 2021
Jul 15 2021
Jul 15 2021
In D31146#701876, @gbe wrote:The man page part looks good, but I am a little bit confused, on my FreeBSD systems I see
root 66156 0.0 0.0 12800 1680 - Is 7Jul21 0:04.38 /usr/sbin/cron -s
when executing 'ps axuw'. So it seems, that some kind of a '-s' option is already implemented.
Jun 17 2021
Jun 17 2021
May 20 2021
May 20 2021
May 8 2021
May 8 2021
May 5 2021
May 5 2021
getdirentries.2: remove unnecessary space
Balance parentheses in sysctl descriptions
sh: fix debug build
sh: silence sanitizer
sh(1): autocomplete commands
May 3 2021
May 3 2021
May 2 2021
May 2 2021
if (fd == -1 || (f = fdopen(fd, "w")) == NULL) and it will LGTM.
Apr 30 2021
Apr 30 2021
I tested the save/restore history capabilities that are built into libedit and so far it looks good to me.
Apr 11 2021
Apr 11 2021
getdirentries.2: remove unnecessary space
Balance parentheses in sysctl descriptions
sh: fix debug build
Apr 7 2021
Apr 7 2021
In D26447#664004, @rgrimes wrote:Hum, not a core system functionality? May I remind you that ftp://ftp.freebsd.org exists? Though it probably does not run the base system ftpd, it DOES run that functionality.
Apr 6 2021
Apr 6 2021
If there are security bugs in ftpd that will force us to release errata. It won't be the vocal users' responsibility to handle it. On the other hand, it wouldn't painful to pkg install ftpd unless you're emotionally attached. The existence of ftpd in base also provokes suggestions to maintain and enhance it (for example with encryption support).
Apr 5 2021
Apr 5 2021
Apr 1 2021
Apr 1 2021
sh: silence sanitizer
Mar 31 2021
Mar 31 2021
Mar 30 2021
Mar 30 2021
Mar 29 2021
Mar 29 2021
sh(1): autocomplete commands
Mar 28 2021
Mar 28 2021
I believe that all reported issues (and more) have been addressed.
Mar 27 2021
Mar 27 2021
security.7: fix typo in sysctl name
Mar 26 2021
Mar 26 2021
security.7: fix typo in sysctl name
Mar 24 2021
Mar 24 2021
Add code to remove duplicates.
Mar 22 2021
Mar 22 2021
Apply some of the fixes suggested by Jilles.
Mar 21 2021
Mar 21 2021
Thanks, I tried to fix all the obvious mistakes. I will have to test and think about the rest of your comments.
Address mistakes pointed out by Alex.
Mar 16 2021
Mar 16 2021
@kevans pointed out to me that my previous patch replaces the name with a dash character, while login(1) only prepends the dash to the original program name, so that's probably what shells expect. I did try to fix that, it seems to work for the few cases that I tested, but in the meantime someone asked about Linux jails, which I looked into. Those can't be properly supported since our getpwnam won't work with the underlying system's structure; so while it is possible to spawn a login shell, the shell would always be /bin/sh which under standard Ubuntu seems to translate into dash, not a great shell for interactive use.
Mar 14 2021
Mar 14 2021
Agree with jamie's comments.
Aug 14 2020
Aug 14 2020
pstef added inline comments to D26064: Check XHCI endpoint state before issuing XHCI endpoint commands.
Aug 3 2020
Aug 3 2020
MFC 363621: sh(1): print a newline when ^D quits sh
Jul 31 2020
Jul 31 2020
Jul 27 2020
Jul 27 2020
sh(1): print a newline when ^D quits sh
I would also remove the "most interesting" comment.