Page MenuHomeFreeBSD

docsProject
ActivePublic

Details

Description

People who are willing to review documentation changes (opt-in)

Recent Activity

Today

temcbun_gmail.com updated the diff for D58329: pw.conf.5: Make the .Nd more informative.

Get rid of .Xr in .Nd and make the .Nd itself more informative.

Tue, Jul 21, 9:17 AM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com added a comment to D58329: pw.conf.5: Make the .Nd more informative.

Thank you for the review!

Tue, Jul 21, 9:14 AM · Contributor Reviews (src), manpages, docs
ziaee updated subscribers of D55417: ls.1: Explain that color sequences can not be used with tabs.
Tue, Jul 21, 4:45 AM · Contributor Reviews (src), docs, manpages

Yesterday

ziaee updated subscribers of D55417: ls.1: Explain that color sequences can not be used with tabs.

I have not reviewed this because all I know about colors is how to turn them off, drives me crazy. Maybe... @dteske might know?

Mon, Jul 20, 10:41 PM · Contributor Reviews (src), docs, manpages
ziaee requested changes to D58329: pw.conf.5: Make the .Nd more informative.
Mon, Jul 20, 10:38 PM · Contributor Reviews (src), manpages, docs
ziaee closed D55333: sh.1: Document which features are non-POSIX.
Mon, Jul 20, 9:54 PM · manpages, docs
temcbun_gmail.com added a comment to D55333: sh.1: Document which features are non-POSIX.

@jilles Could you please accept this revision once again, just to move it into 'ready to land' category?

That is not necessary.

  1. We do not really have tooling around that, so it does not make it any more discoverable.
  2. Whoever lands it is responsible for handling the judgement. So, jilles approval stands unless he rescinds it or a change is made which we would deem requires his re-review per the context.
  3. Someone might do that as a form of ping, since it sends out an email.

For example, since jilles is reviewing it in the context of being the lead shell maintainer, we could still make mechanical changes at this stage like fixing a typo or improving text alignment. However a content change that changes the meaning would, to me, invalidate his previous approval and require his re-review.

Mon, Jul 20, 9:53 PM · manpages, docs
ziaee added a comment to D55333: sh.1: Document which features are non-POSIX.

Thank you both, I think this is a great new feature for the sh manual and I am so excited to land this now!

Mon, Jul 20, 9:49 PM · manpages, docs
temcbun_gmail.com added a comment to D55333: sh.1: Document which features are non-POSIX.

@jilles Could you please accept this revision once again, just to move it into 'ready to land' category?

Mon, Jul 20, 6:30 PM · manpages, docs
mhorne closed D58350: vfs_unmountall.9: Fix an outdated .Xr to boot(9).
Mon, Jul 20, 5:33 PM · Contributor Reviews (src), manpages, docs
mhorne accepted D58350: vfs_unmountall.9: Fix an outdated .Xr to boot(9).
Mon, Jul 20, 5:29 PM · Contributor Reviews (src), manpages, docs
emaste accepted D58350: vfs_unmountall.9: Fix an outdated .Xr to boot(9).
Mon, Jul 20, 12:45 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com added a comment to D55169: kldload.2: Spell kld in lowercase.

Ping.

Mon, Jul 20, 12:00 PM · manpages, docs

Sun, Jul 19

kib accepted D58350: vfs_unmountall.9: Fix an outdated .Xr to boot(9).
Sun, Jul 19, 8:37 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com requested review of D58350: vfs_unmountall.9: Fix an outdated .Xr to boot(9).
Sun, Jul 19, 8:13 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com updated the diff for D55333: sh.1: Document which features are non-POSIX.

Rebase the patch and bump .Dd.

Sun, Jul 19, 12:54 PM · manpages, docs
jilles accepted D55333: sh.1: Document which features are non-POSIX.
Sun, Jul 19, 12:37 PM · manpages, docs

Sat, Jul 18

temcbun_gmail.com requested review of D58329: pw.conf.5: Make the .Nd more informative.
Sat, Jul 18, 9:37 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com added a comment to D55333: sh.1: Document which features are non-POSIX.

Ping.

Sat, Jul 18, 9:14 PM · manpages, docs
temcbun_gmail.com added a comment to D55417: ls.1: Explain that color sequences can not be used with tabs.

Ping.

Sat, Jul 18, 9:13 PM · Contributor Reviews (src), docs, manpages
temcbun_gmail.com added a comment to D56648: users.7: Standard user account names.

Ping.

Sat, Jul 18, 9:12 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com requested review of D58328: mktemp.1: Improve the .Nd description.
Sat, Jul 18, 9:12 PM · Contributor Reviews (src), manpages, docs

Tue, Jul 7

vladlen closed D58046: documentation: fix crossref macro with comma in parameter.
Tue, Jul 7, 7:49 PM · docs

Mon, Jul 6

carlavilla accepted D58046: documentation: fix crossref macro with comma in parameter.
Mon, Jul 6, 8:35 PM · docs
ziaee accepted D58046: documentation: fix crossref macro with comma in parameter.
Mon, Jul 6, 8:28 PM · docs
ziaee added a comment to D57720: man: iwlwifi, rtw8{8,9}: Fix linuxkpi{_wlan}(4) .Xr's.

@ziaee Could you please MFC this commit (I see the commit message says it is to be MFCed after 3 days, but it's still not).

Mon, Jul 6, 8:27 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com added a comment to D57720: man: iwlwifi, rtw8{8,9}: Fix linuxkpi{_wlan}(4) .Xr's.

@ziaee Could you please MFC this commit (I see the commit message says it is to be MFCed after 3 days, but it's still not).

Mon, Jul 6, 2:20 PM · Contributor Reviews (src), manpages, docs

Sun, Jul 5

olce closed D58038: ps.1: Fix broken comment line.
Sun, Jul 5, 8:04 PM · Contributor Reviews (src), manpages, docs
vladlen closed D58045: books/dev-model: change bibliography links.
Sun, Jul 5, 7:14 PM · docs
vladlen requested review of D58046: documentation: fix crossref macro with comma in parameter.
Sun, Jul 5, 7:10 PM · docs
carlavilla accepted D58045: books/dev-model: change bibliography links.
Sun, Jul 5, 6:19 PM · docs
vladlen added a comment to D57999: documentation: fix WARN after hugo setings change.

Yes, as you said, change it.

https://docs.asciidoctor.org/asciidoc/latest/sections/bibliography/

Sun, Jul 5, 5:48 PM · docs
vladlen requested review of D58045: books/dev-model: change bibliography links.
Sun, Jul 5, 5:47 PM · docs
temcbun_gmail.com added a comment to D58038: ps.1: Fix broken comment line.

Removing the hyphen in the first line is fine, but then that should be mentioned in the commit message. I can add a sentence to that effect myself.

Sun, Jul 5, 4:53 PM · Contributor Reviews (src), manpages, docs
olce accepted D58038: ps.1: Fix broken comment line.

Removing the hyphen in the first line is fine, but then that should be mentioned in the commit message. I can add a sentence to that effect myself. Which mail do you want to use? The one that can be inferred from your Phabricator login? I also see a mail at "zohomail" in the tree that could match. If you have to disclose the full name and don't want to do that here, then please mail it to me (olce@).

Sun, Jul 5, 4:49 PM · Contributor Reviews (src), manpages, docs

Sat, Jul 4

temcbun_gmail.com updated the diff for D58038: ps.1: Fix broken comment line.

Remove hyphen from the first comment line. Thanks @ziaee!

Sat, Jul 4, 6:46 PM · Contributor Reviews (src), manpages, docs
ziaee accepted D58038: ps.1: Fix broken comment line.
Sat, Jul 4, 6:43 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com added a comment to D56648: users.7: Standard user account names.

Ping for review.

Sat, Jul 4, 3:36 PM · Contributor Reviews (src), manpages, docs
temcbun_gmail.com requested review of D58038: ps.1: Fix broken comment line.
Sat, Jul 4, 3:32 PM · Contributor Reviews (src), manpages, docs
vladlen closed D57999: documentation: fix WARN after hugo setings change.
Sat, Jul 4, 2:58 PM · docs
carlavilla added a comment to D57999: documentation: fix WARN after hugo setings change.

Do you see any problems with them?

It seems you pushed this commit already and it is live on https://docs.freebsd.org/en/books/dev-model/, where the links work. However local build with cd documentation; make, the links do not work, whereas they links do work for local build in handbook. Also, the appearance is wrong for that page because there are padded spaces some places and not other places. Maybe we can do something like \[crossref:dev-model[ref-freebsd-trenches, Lehey 2002]\] for the appearance? I do not know why the links are broken in offline build though.

I will check it. I checked it locally with cd documentation; make run and
looked at it in a browser.

I tried using \[ , but the output rendered as \[ literally.

Probably we could change the link style. Links like [1] were used in old
books, and now they are mostly used in scientific articles. Square brackets
are special symbols in AsciiDoc, and we will spend more time fighting with
them than using them comfortably.

Sat, Jul 4, 10:17 AM · docs
vladlen added a comment to D57999: documentation: fix WARN after hugo setings change.

Do you see any problems with them?

It seems you pushed this commit already and it is live on https://docs.freebsd.org/en/books/dev-model/, where the links work. However local build with cd documentation; make, the links do not work, whereas they links do work for local build in handbook. Also, the appearance is wrong for that page because there are padded spaces some places and not other places. Maybe we can do something like \[crossref:dev-model[ref-freebsd-trenches, Lehey 2002]\] for the appearance? I do not know why the links are broken in offline build though.

Sat, Jul 4, 10:13 AM · docs

Fri, Jul 3

vladlen added a comment to D58001: fix typos in documentation.
In D58001#1330236, @bcr wrote:

We can't make changes to past release notes once they are published. They become part of the release hash and would change in a number of places that reference them (even if they contain typos).

Fri, Jul 3, 10:54 AM · docs

Thu, Jul 2

ziaee added a comment to D58001: fix typos in documentation.
In D58001#1330236, @bcr wrote:

We can't make changes to past release notes once they are published. They become part of the release hash and would change in a number of places that reference them (even if they contain typos).

Thu, Jul 2, 6:46 PM · docs
ziaee added a comment to D57999: documentation: fix WARN after hugo setings change.

Do you see any problems with them?

Thu, Jul 2, 6:43 PM · docs
bcr added a comment to D58001: fix typos in documentation.

We can't make changes to past release notes once they are published. They become part of the release hash and would change in a number of places that reference them (even if they contain typos).

Thu, Jul 2, 5:13 PM · docs
vladlen closed D57722: website: add RSS feed to status pages.
Thu, Jul 2, 12:31 PM · docs
vladlen closed D57725: website: fix RSS feed for security page.
Thu, Jul 2, 12:28 PM · docs
carlavilla added a comment to D57999: documentation: fix WARN after hugo setings change.

Dude you rock! Aside from dev-model, all the changes LGTM.

if the macro is broken, I need to fix it. But I created this macros coz we have these scenarios:

  • Online website (freebsd.org, docs.freebsd.org)
  • Offline website
  • PDFs

It works everywhere else I checked except dev-model.

This links had 2 problems:
[crossref:dev-model[freebsd-send-pr, FreeBSD, 2002D]]

  1. The same, as in website - it started and finished by square bracket

and simply disappeared from output (considered as AsciiDoc command).
I fixed this, they appeared, and I saw second problem:

  1. The text FreeBSD, 2002D rendered as FreeBSD, everything

after comma were ignored. I removed comma and text is renderd correctly.
This comma is only for refference style, it does not have any special meaning.
With this changes all these links work correctly at least in HTML.

Do you see any problems with them?

P.S. Original article used reference style [FreeBSD, 2002D] long before AsciiDoc
was choosen for FreeBSD documentation, and now [ and ] chars need special treatment
in adoc files, or we are free to rename all these links.

Thu, Jul 2, 11:41 AM · docs
carlavilla accepted D57725: website: fix RSS feed for security page.
Thu, Jul 2, 11:39 AM · docs