Page MenuHomeFreeBSD

debdrup (Daniel Ebdrup)
User

Projects

User Details

User Since
Jan 12 2018, 3:33 PM (356 w, 9 h)

Recent Activity

Sep 27 2024

debdrup closed D46659: Adding QEMU Virtualization to Handbook.
Sep 27 2024, 9:02 PM
debdrup committed R9:3de85d772f3e: virtualization: Add chapter on QEMU (authored by jpb_jimby.name).
virtualization: Add chapter on QEMU
Sep 27 2024, 9:02 PM
debdrup added a comment to D46659: Adding QEMU Virtualization to Handbook.

Seems like the latest diff hasn't been uploaded yet, but once it is uploaded, I think this is ready to land?

Sep 27 2024, 5:14 PM

Sep 9 2024

debdrup added a comment to D46557: First Changes for adding QEMU to Virtualization Chapter in the Handbook.

All of the "Errant period" comments fall into one of two categories:

  1. Ordered lists - these are the "period at beginning of line followed by space" variety. Ordered lists are covered in the FDP Primer, section 6.4.1
  2. Image captions - these are the "period at beginning of line followed by text" variety. Image captions are covered in the FDP Primer in section 7.1 under the "Images" row in the table.

I will mark all of those comments "Done".

Sep 9 2024, 12:03 PM
debdrup resigned from D46571: Remainder of Changes for Adding QEMU to Virtualization Chapter in the Handbook.

Please remember to mark this review as abandoned.

Sep 9 2024, 12:01 PM

Sep 7 2024

debdrup added a comment to D46557: First Changes for adding QEMU to Virtualization Chapter in the Handbook.

Did a pass on the remaining bit that was added, but it's entirely possible I've missed something.

Sep 7 2024, 9:42 PM

Sep 6 2024

debdrup added a comment to D46557: First Changes for adding QEMU to Virtualization Chapter in the Handbook.

I did a quick first pass since I had a bit of energy remaining.

Sep 6 2024, 7:56 PM

Sep 4 2024

debdrup committed R9:920aa82eb14f: sockets(examples): remove unused register, replace bzero with memset (authored by rilysh <nightquick@proton.me>).
sockets(examples): remove unused register, replace bzero with memset
Sep 4 2024, 11:43 AM

Jan 29 2024

debdrup committed rGa84e01f02b15: service(8): direct user to rc(8) for a list of valid commands (authored by lexi_le-fay.org).
service(8): direct user to rc(8) for a list of valid commands
Jan 29 2024, 5:49 PM

Jan 9 2024

debdrup committed rG45486372ca8d: nfsv4(4): Reflow lines to 80 character limit (authored by debdrup).
nfsv4(4): Reflow lines to 80 character limit
Jan 9 2024, 11:16 AM
debdrup committed rGc925f0720f58: nfsv4(4): Belatedly bump .Dd (authored by debdrup).
nfsv4(4): Belatedly bump .Dd
Jan 9 2024, 11:16 AM

Jan 6 2024

debdrup committed rGfbbdfa2b8a42: nfsv4(4): mention the nfsv4_server_only setting (authored by lexi_le-fay.org).
nfsv4(4): mention the nfsv4_server_only setting
Jan 6 2024, 6:23 PM

Dec 22 2023

debdrup accepted D43165: newsyslog(8): Add option to globally override compression method..

mdoc(7) syntax looks fine to me.

Dec 22 2023, 2:27 PM
debdrup added a comment to D42961: Do not compress log files.
In D42961#979432, @0mp wrote:

I think it'd be a good idea to add these as an include - that way, it's easier to enable or disable on a case-by-case basis.

What do you mean by "as an include"?

There are already includes in the file for newsyslog.conf.d/ and the local variant of it.
If I correctly understand how newsyslog works, setting the different values in the include file would override them when the configuration is read by newsyslog?

Dec 22 2023, 2:25 PM

Dec 7 2023

debdrup added a comment to D42961: Do not compress log files.

I think it'd be a good idea to add these as an include - that way, it's easier to enable or disable on a case-by-case basis.

Dec 7 2023, 3:31 PM

Aug 21 2023

debdrup accepted D41522: libc: Document support for binary integers..

Only spotted two minor nits, but they can be fixed before pushing as they don't have to hold up things.

Aug 21 2023, 1:03 PM

Jul 21 2023

debdrup added a comment to D41111: status 2023q3: add pf status report.

Spotted some minor nits, but these can be fixed when it's pushed.

Jul 21 2023, 9:58 AM

Jun 15 2023

debdrup accepted D40566: handbook: use userid example other than `toor`.

Putting aside the debate over the toor user for another time, using it as an example user doesn't seem like the best option, as it's meant to rescue systems.

Jun 15 2023, 2:36 PM

May 30 2023

debdrup added a comment to D40329: rc(8): document how to reload rc's configuration.

I only spotted a minor nit.

May 30 2023, 9:29 AM · manpages, rc

May 23 2023

debdrup accepted D40237: ctags: Support writing to stdout instead of a file..

mdoc(7) changes look good to me, can't speak to the code.

May 23 2023, 6:49 PM
debdrup accepted D40214: FAQ: switch Subversion references to Git.

Looks good to me.

May 23 2023, 7:47 AM

May 3 2023

debdrup committed R9:e7be269af4a9: Added missing -u flag to pdbedit (authored by Anthony Bravolisimo <freebsd@crontab.pw>).
Added missing -u flag to pdbedit
May 3 2023, 10:26 PM

Apr 12 2023

debdrup added a comment to D37881: cxgbetool(8): cleanup.

Need to convert them to .Bl/.It+Ta/.El, but we'd better ask someone from the manpages to be sure.

Apr 12 2023, 11:45 AM

Mar 27 2023

debdrup committed R9:cc7d9533f889: relnotes/12.1: Fix markup breaking rendering (authored by debdrup).
relnotes/12.1: Fix markup breaking rendering
Mar 27 2023, 2:14 PM

Feb 28 2023

debdrup added a comment to D38826: Fix multiple rc.d/jail and jail.conf.d issues.

To add a bit of context from a conversation on IRC, this review is intended to make it much easier to define jail(8) variables globally at the top of jail.conf(5), you only need to instanciate the name of a jail and optionally some per-jail values.

Feb 28 2023, 7:04 PM · Jails

Feb 21 2023

debdrup added a comment to D27959: kldxref: skip .pkgsave files.

Please remember to bump .Dd :)

Feb 21 2023, 12:44 PM

Feb 7 2023

debdrup committed R9:94013543d5ef: Extend my PGP key (authored by debdrup).
Extend my PGP key
Feb 7 2023, 6:48 PM
debdrup added inline comments to D38386: Add nproc(1).
Feb 7 2023, 4:18 PM
debdrup accepted D38398: status: Update report submission methods.

I'm perfectly satisfied with this import, since I've used the sendcalls script and had a hand in crafting the template.

Feb 7 2023, 1:24 PM

Feb 5 2023

debdrup accepted D38382: contributing: Start pointing people at github.

Looks good to me!

Feb 5 2023, 11:40 AM

Feb 4 2023

debdrup added a comment to D38382: contributing: Start pointing people at github.

Just a few nits that I noticed that may be worth considering.

Feb 4 2023, 8:51 PM

Feb 3 2023

debdrup accepted D38370: certctl(8): Remove untrusted certificates from TRUSTPATH.

The mdoc(7) changes look good to me.

Feb 3 2023, 2:09 PM · Contributor Reviews (src), manpages

Dec 20 2022

debdrup added a comment to D37753: Add tarfs, a filesystem backed by tarballs..

Manual page looks good to me, with the exception of one minor nit.

Dec 20 2022, 4:05 PM

Oct 10 2022

debdrup accepted D33677: Mention vendor driver for if_re.

Apologies for taking so long, life got in the way.

Oct 10 2022, 2:33 PM

Sep 18 2022

debdrup added a comment to D36619: Deprecate telnetd.

mdoc(7) looks good

Sep 18 2022, 8:12 AM

Sep 12 2022

debdrup added inline comments to D36513: domains: rewrite documentation to describe present state.
Sep 12 2022, 5:15 PM

Jul 9 2022

debdrup accepted D35756: pkg: Add limited --debug/-d support.

I spotted a few nits, but they're not worth holding up the review for as they can be fixed before the commit is pushed.

Jul 9 2022, 8:00 AM

Jul 5 2022

debdrup added a comment to D35694: Add ts(1) command.

Other than the question, I'm still happy with the mdoc, so will accept once that's been answered.

Jul 5 2022, 10:45 AM

Jul 4 2022

debdrup accepted D35694: Add ts(1) command.

mdoc(7) looks good to me.

Jul 4 2022, 6:04 AM

Jul 2 2022

debdrup added a comment to D35694: Add ts(1) command.

I cant immediately spot any other nits with the mdoc, but perhaps you should add a note about when ts(1) was added to FreeBSD in the history section header?

Jul 2 2022, 4:26 PM

Jun 10 2022

debdrup committed R9:0454aae0d40c: status/2022q1: Fix space issue (authored by debdrup).
status/2022q1: Fix space issue
Jun 10 2022, 11:35 AM
debdrup committed R9:e6cacb6ecf7d: status/README: Add an order of operations (authored by debdrup).
status/README: Add an order of operations
Jun 10 2022, 11:29 AM
debdrup committed R9:3847af12e0c7: status/2022q1: Fix several small nits noticed late (authored by debdrup).
status/2022q1: Fix several small nits noticed late
Jun 10 2022, 3:04 AM

Jun 8 2022

debdrup accepted D35402: status/report-2022-01-2022-03: Add report.

I don't think there's anything else I remarked on that needs to be addressed, so if everyone else would accept it if theirs has been dealt with, we can get the needful done.

Jun 8 2022, 4:57 AM

Jun 6 2022

debdrup accepted D35405: Make SYNOPSIS match DESCRIPTION..

As for the change, it looks good to me - but as I'm sure you know, you need extra approval too. ;)

Jun 6 2022, 1:16 PM

Jun 4 2022

debdrup updated subscribers of D35402: status/report-2022-01-2022-03: Add report.

I've made some suggestions for where various entries belong, but that's about it I think.

Jun 4 2022, 1:35 PM

May 27 2022

debdrup added a comment to D35326: glabel.8: Warn against using generic labels on a shared device.

mdoc(7) syntax looks good to me, so once the other issues pointed out with the manual page have been fixed, I'm happy to accept it.

May 27 2022, 8:57 AM

May 23 2022

debdrup accepted D35283: mutex(9): correct man page, mtx_trylock_spin returns int not void.

I'm happy with mdoc(7) syntax since .Dd was bumped ;)

May 23 2022, 10:36 AM
debdrup accepted D35246: Fix grammar nit in versions chapter as discussed with bsdimp in email..

Looks good to go, go ahead and commit @imp

May 23 2022, 10:33 AM

May 8 2022

debdrup accepted D32551: if(9): Implement support for nvlist-based set- and get- network interface capabilities..

mdoc looks good to me.

May 8 2022, 4:05 PM

May 7 2022

debdrup added a comment to D32551: if(9): Implement support for nvlist-based set- and get- network interface capabilities..

.Dd also needs to be bumped.

May 7 2022, 11:17 PM

Apr 22 2022

debdrup accepted D34933: Import a quoted-printable bidirectional converter.

mdoc(7) looks good to me, but please remember to bump .Dd before you commit. :)

Apr 22 2022, 7:12 PM

Apr 21 2022

debdrup added a comment to D35015: handbook/zfs: Changes zfs send -P into zfs send -p.

I think it's ready to go now, and the summary should work better as a descriptive commit message. :)

Apr 21 2022, 9:55 AM
debdrup retitled D35015: handbook/zfs: Changes zfs send -P into zfs send -p from Handbook/zfs: Changes zfs send -P into zfs send -p to handbook/zfs: Changes zfs send -P into zfs send -p.
Apr 21 2022, 9:54 AM
debdrup accepted D35015: handbook/zfs: Changes zfs send -P into zfs send -p.

You're absolutely right, -P means parsable, whereas -p means properties.
You should probably explain that in the commit log, though - may I edit the summary?

Apr 21 2022, 9:25 AM

Apr 19 2022

debdrup accepted D34969: Tweaks to the software licensing page.

Looks good to me, so assuming it builds, you're good to go. :)

Apr 19 2022, 7:02 PM
debdrup added a comment to D34969: Tweaks to the software licensing page.

For one-sentence-per-line changes, we generally like to keep them as separate commits.
However, I think it's okay for this commit since you're changing so little.

Apr 19 2022, 6:56 PM

Apr 18 2022

debdrup added a comment to D34947: setkey(8): Clarify language around AEAD ciphers..

Ss, like Sh, should be capitalized according to mdoc(7).

Apr 18 2022, 8:06 PM
debdrup accepted D34946: Refactor version list in user handbook intro.

Looks good to me.

Apr 18 2022, 6:56 PM
debdrup committed R9:e8543cd9ce71: arch-handbook: Fix a few nits (authored by debdrup).
arch-handbook: Fix a few nits
Apr 18 2022, 2:35 PM
debdrup committed R9:821ac7145039: arch-handbook: Convert to one sentence per line (authored by debdrup).
arch-handbook: Convert to one sentence per line
Apr 18 2022, 2:35 PM
debdrup committed R9:954bbbabe38e: arch-handbook: Update boot chapter (authored by Isa <isa@isoux.org>).
arch-handbook: Update boot chapter
Apr 18 2022, 2:35 PM

Apr 13 2022

debdrup added a comment to D34896: strcpy(3): Explicitly mention the recommended usage of strlcpy().

I'd prefer if you used .Sy or .Em rather than capitalization.

Apr 13 2022, 9:23 AM

Apr 12 2022

debdrup committed rGbd1eafcdd02a: protect(1): Correct typo and add newline (authored by debdrup).
protect(1): Correct typo and add newline
Apr 12 2022, 10:41 AM

Apr 11 2022

debdrup committed rGc8b6be0f7d1b: protect.1: document existence of _oomprotect (authored by a.wolk_fudosecurity.com).
protect.1: document existence of _oomprotect
Apr 11 2022, 10:27 PM
debdrup closed D30334: protect.1: document existence of _oomprotect.
Apr 11 2022, 10:26 PM · manpages, docs
debdrup commandeered D30334: protect.1: document existence of _oomprotect.

I apologize deeply for losing track of this one (and the change to rc.subr(8), although I think I managed to address that one).

Apr 11 2022, 10:23 PM · manpages, docs
debdrup added a comment to D30336: rc.subr.8: document argument_cmd override pitfalls.

I ended up documenting this in bd6dce978c1, I believe - can you confirm?

Apr 11 2022, 10:19 PM · rc, manpages, docs

Apr 9 2022

debdrup added a comment to D34852: KTR: Add CTR() and TR() macros which accept a variable number of arguments..

Manual page syntax looks good, but it'd be nice to have a mention of the timing in the HISTORY section.

Apr 9 2022, 9:17 AM

Apr 7 2022

debdrup added a comment to D34690: RDMA driver for Intel(R) Ethernet Controller E810.

Have you run the manual page through mandoc -Tlint and igor from textproc/igor?

Apr 7 2022, 2:10 PM
debdrup accepted D34672: siftr: expose t_flags2 in siftr output.

Manual page looks good to me, but please remember to bump .Dd before pushing. ;)

Apr 7 2022, 7:34 AM

Apr 5 2022

debdrup added inline comments to D34347: savecore: add an option to save a live minidump.
Apr 5 2022, 10:26 AM

Apr 2 2022

debdrup accepted D34731: bikeshed.7: the backstory.
Apr 2 2022, 11:00 AM

Apr 1 2022

debdrup accepted D34632: Intel (R) QAT driver.

Manual page looks good to me from a syntax perspective, and I wasn't able to spot any other issues.

Apr 1 2022, 10:04 AM
debdrup requested changes to D34731: bikeshed.7: the backstory.

Please run mandoc -Tlint and igor on the manual page, you'll see that there's a few minor nits that need to be fixed.

Apr 1 2022, 9:43 AM
debdrup added inline comments to D34632: Intel (R) QAT driver.
Apr 1 2022, 9:36 AM

Mar 24 2022

debdrup added a comment to D34662: ifconfig.8: Add capability codes descriptions from handbook.

Syntax looks good, and it matches what I remember of my CCIE Wireless courses.

Mar 24 2022, 8:40 PM
debdrup accepted D34663: handbook: Move capabilities' description to ifconfig(8).

Looks good to me.

Mar 24 2022, 8:40 PM

Mar 23 2022

debdrup committed rG4b1e2c2f77d3: bsdinstall(8): Usage example of tar doesn't work (authored by farhan_farhan.codes).
bsdinstall(8): Usage example of tar doesn't work
Mar 23 2022, 9:02 PM
debdrup closed D34649: bsdinstall.8 tar command order of argument breakage.
Mar 23 2022, 9:01 PM · manpages
debdrup commandeered D34649: bsdinstall.8 tar command order of argument breakage.

Commandeering this now that @rpokala has approved it on IRC.

Mar 23 2022, 8:59 PM · manpages
debdrup accepted D34649: bsdinstall.8 tar command order of argument breakage.

mdoc syntax hasn't really changed and .Dd has been bumped, so this can be commited and pushed.

Mar 23 2022, 8:48 PM · manpages

Mar 22 2022

debdrup added a comment to D34632: Intel (R) QAT driver.

There are also some lines that exceed 80 characters in width, so the manual page needs to be rewrapped.

Mar 22 2022, 4:39 PM

Mar 20 2022

debdrup accepted D34615: sysctl(8): Mention CTLFLAG_SKIP.

Good to go.

Mar 20 2022, 6:25 PM
debdrup accepted D34615: sysctl(8): Mention CTLFLAG_SKIP.

I only spotted one minor nit, so I'll accept this and you can commit and push at your convenience after you've fixed it.

Mar 20 2022, 6:06 PM

Mar 16 2022

debdrup accepted D34576: wpa_supplicant.conf.5: add note about scan_ssid=1 eavesdropping.

Looks good to me.

Mar 16 2022, 8:27 AM

Mar 15 2022

debdrup accepted D34565: tslog.4: Document what TSLOG excels at.

Looks good to me.

Mar 15 2022, 7:07 PM
debdrup accepted D34566: boottrace.4: Document that boottrace is good at tracing one-shot events.

A nice little addition to have. :)

Mar 15 2022, 7:07 PM
debdrup accepted D32551: if(9): Implement support for nvlist-based set- and get- network interface capabilities..

Manual page looks good, I'll let others review the code.

Mar 15 2022, 7:06 PM
debdrup accepted D34570: Porter's Handbook: Document Uses=elfctl.

Looks good to me.

Mar 15 2022, 7:05 PM
debdrup added a comment to D32551: if(9): Implement support for nvlist-based set- and get- network interface capabilities..

I only noticed one thing in the mdoc syntax.

Mar 15 2022, 11:42 AM

Mar 14 2022

debdrup added a reviewer for D34556: Summer of Code page Fixes: docs.

My initial pass of it looks good, but consider waiting for someone else to look over it too.

Mar 14 2022, 11:29 PM

Mar 13 2022

debdrup accepted D34545: Chase mixer(8) command-line interface changes.

The mdoc syntax looks good to me.

Mar 13 2022, 3:39 PM

Mar 9 2022

debdrup added a comment to D34508: diff: add support for --help and --version.

A tiny nit with the mdoc syntax, otherwise it looks good to me.

Mar 9 2022, 12:07 PM
debdrup added a comment to D34502: libmd: export and document *Fd/*FdChunk interfaces.

I think you should add the "nearly" qualifier to identical, as saying they're identical and then mentioning an exception seems contradictory to me.

Mar 9 2022, 11:11 AM
debdrup accepted D34504: kern: add devctl_notifyf().

Manual page changes look good to me.

Mar 9 2022, 11:07 AM

Mar 8 2022

debdrup accepted D34482: pf: support basic L3 filtering in the Ethernet rules.

Please remember to bump .Dd on pf.conf.5 - other than that, manual page looks fine.

Mar 8 2022, 9:26 PM

Mar 4 2022

debdrup accepted D34438: time.3: Update ERRORS section.
Mar 4 2022, 5:47 PM
debdrup accepted D34430: tslog.4: Document TSLOG.

Manual page looks good to me.

Mar 4 2022, 5:46 PM

Mar 3 2022

debdrup added a comment to D34421: diff3: Add support for -A and -m.

I spotted one thing in the manual page, otherwise it looks good to me.

Mar 3 2022, 12:39 PM