Page MenuHomeFreeBSD

docsProject
ActivePublic

Details

Description

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

Recent Activity

Yesterday

tembun_bk.ru retitled D55333: sh.1: Document which features are non-POSIX from sh.1: Document which features are non-standard to sh.1: Document which features are non-POSIX.
Wed, Feb 18, 7:26 PM · manpages, docs
tembun_bk.ru updated the diff for D55333: sh.1: Document which features are non-POSIX.

Change 'nonstandard' -> 'non-POSIX'.

Wed, Feb 18, 7:24 PM · manpages, docs
tembun_bk.ru added inline comments to D55333: sh.1: Document which features are non-POSIX.
Wed, Feb 18, 6:25 PM · manpages, docs
ziaee requested changes to D55333: sh.1: Document which features are non-POSIX.

I absolutely love the base premise of what you're trying to do here. This will be a major accessibility win for the faction which supports promoting portable programming, which FreeBSD is one of the main champions of.

Wed, Feb 18, 6:15 PM · manpages, docs
marck accepted D55358: website: update ru/events.

looks good, thank you!

Wed, Feb 18, 5:32 PM · docs
vladlen closed D53479: fix translation of ru/articles/.
Wed, Feb 18, 5:03 PM · docs
vladlen requested review of D55358: website: update ru/events.
Wed, Feb 18, 5:02 PM · docs
vladlen closed D55331: website: update ru/news - SA and EN translated .
Wed, Feb 18, 4:17 PM · docs

Tue, Feb 17

andy accepted D55331: website: update ru/news - SA and EN translated .
Tue, Feb 17, 7:33 PM · docs
andy added a comment to D55331: website: update ru/news - SA and EN translated .

Мелкие корректировки непринципиального и рекомендательного характера.

Tue, Feb 17, 7:33 PM · docs
tembun_bk.ru updated the diff for D55333: sh.1: Document which features are non-POSIX.

Grammar improvements:

  • Prefer American English: 'non-standard' -> 'nonstandard', 'behaviour' -> 'behavior'.
  • 'a nonstandard local command' -> 'the nonstandard local command'.
Tue, Feb 17, 6:21 PM · manpages, docs
tembun_bk.ru added inline comments to D55333: sh.1: Document which features are non-POSIX.
Tue, Feb 17, 6:07 PM · manpages, docs
emaste added inline comments to D55333: sh.1: Document which features are non-POSIX.
Tue, Feb 17, 5:59 PM · manpages, docs
tembun_bk.ru added inline comments to D55333: sh.1: Document which features are non-POSIX.
Tue, Feb 17, 5:41 PM · manpages, docs
tembun_bk.ru requested review of D55333: sh.1: Document which features are non-POSIX.
Tue, Feb 17, 5:34 PM · manpages, docs
vladlen requested review of D55331: website: update ru/news - SA and EN translated .
Tue, Feb 17, 4:30 PM · docs

Mon, Feb 16

ziaee closed D55194: sh.1: Fix ordering of Cm and Aq macros.
Mon, Feb 16, 5:24 PM · manpages, docs
ziaee closed D55026: sh.1: Document exit status of the function return command.
Mon, Feb 16, 5:24 PM · docs, manpages
jilles accepted D55026: sh.1: Document exit status of the function return command.

Let's not mention the behaviour when return exits the shell during an EXIT trap action, since it's obscure and probably incorrect (I think it's supposed to work like exit, with a difference between return by itself and return "$?", but in practice it always ignores the argument during an EXIT trap action).

Mon, Feb 16, 3:46 PM · docs, manpages
ziaee accepted D55169: kldload.2: Spell kld in lowercase.

There's more we can do elsewhere, but this one logical change is complete and increases consistency.

Mon, Feb 16, 1:10 PM · manpages, docs

Fri, Feb 13

vladlen closed D55154: fix links in en/articles/ipsec-must.
Fri, Feb 13, 3:57 AM · docs

Mon, Feb 9

tembun_bk.ru added a comment to D55169: kldload.2: Spell kld in lowercase.
In D55169#1262390, @kib wrote:

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Current kld(4) page needs a complete rewrite. Starting from the removal of LKM references, which were not relevant since 4.x.
The renovation for kld(4) is technically complicated work, requiring a lot of code reading to provide good manual.

Mon, Feb 9, 8:30 PM · manpages, docs
kib added a comment to D55169: kldload.2: Spell kld in lowercase.

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Mon, Feb 9, 8:21 PM · manpages, docs
tembun_bk.ru added a comment to D55169: kldload.2: Spell kld in lowercase.
In D55169#1262234, @kib wrote:

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

No, technically we load 'kld file', which is linked into the executing kernel. The file might contain some number of modules (from 0 to arbitrary), which are used as units of initialization.

Mon, Feb 9, 8:06 PM · manpages, docs
kib added a comment to D55169: kldload.2: Spell kld in lowercase.

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

Mon, Feb 9, 5:26 PM · manpages, docs
lwhsu closed D42775: iwm(4): Fix typo.
Mon, Feb 9, 3:58 PM · docs
tembun_bk.ru requested review of D55194: sh.1: Fix ordering of Cm and Aq macros.
Mon, Feb 9, 3:37 PM · manpages, docs
vladlen closed D54904: The article articles/gjournal-desktop is rewritten according to the current version.
Mon, Feb 9, 3:30 PM · docs
tembun_bk.ru updated subscribers of D55169: kldload.2: Spell kld in lowercase.

How about just load kernel module?

Mon, Feb 9, 3:12 PM · manpages, docs
emaste added a comment to D55169: kldload.2: Spell kld in lowercase.

How about just load kernel module?

Mon, Feb 9, 2:28 PM · manpages, docs
kib accepted D55170: kldload.2: Provide more information about file argument.
Mon, Feb 9, 9:59 AM · manpages, docs
tembun_bk.ru updated the diff for D55170: kldload.2: Provide more information about file argument.

Correct wording.

Mon, Feb 9, 9:51 AM · manpages, docs
tembun_bk.ru updated the diff for D55026: sh.1: Document exit status of the function return command.
  • Remove redundant 'of the function' part.
  • Change "it's" -> "it is".
Mon, Feb 9, 9:40 AM · docs, manpages

Sun, Feb 8

kib added inline comments to D55170: kldload.2: Provide more information about file argument.
Sun, Feb 8, 11:57 PM · manpages, docs
ziaee added inline comments to D55169: kldload.2: Spell kld in lowercase.
Sun, Feb 8, 11:28 PM · manpages, docs
dab requested changes to D55169: kldload.2: Spell kld in lowercase.
Sun, Feb 8, 10:55 PM · manpages, docs
jilles added inline comments to D55026: sh.1: Document exit status of the function return command.
Sun, Feb 8, 10:47 PM · docs, manpages
marck accepted D54904: The article articles/gjournal-desktop is rewritten according to the current version.

looks great.

Sun, Feb 8, 3:56 PM · docs
tembun_bk.ru added inline comments to D55170: kldload.2: Provide more information about file argument.
Sun, Feb 8, 3:38 PM · manpages, docs
kib added inline comments to D55170: kldload.2: Provide more information about file argument.
Sun, Feb 8, 10:46 AM · manpages, docs
tembun_bk.ru added inline comments to D55170: kldload.2: Provide more information about file argument.
Sun, Feb 8, 10:36 AM · manpages, docs
kib added inline comments to D55170: kldload.2: Provide more information about file argument.
Sun, Feb 8, 10:29 AM · manpages, docs
tembun_bk.ru updated the diff for D55170: kldload.2: Provide more information about file argument.

Don't touch kldfind(2), leave changes only for kldload(2).

Sun, Feb 8, 9:53 AM · manpages, docs
kib added a comment to D55170: kldload.2: Provide more information about file argument.

This patch does not make sense (to me at least). kldfind(2) looks up kld from the loaded files in kernel, it does not search anything in the file system.
What the syscall does it strips the passed filename from the non-base path components, and then compares the base name with names of all loaded klds, returning the finding, if any.

Sun, Feb 8, 12:46 AM · manpages, docs

Sat, Feb 7

imp added a comment to D55170: kldload.2: Provide more information about file argument.

The system call is a better place for this info. But it's ok to have it botg places in this case

Sat, Feb 7, 11:12 PM · manpages, docs
ziaee added a reviewer for D55170: kldload.2: Provide more information about file argument: markj.

The information is basically taken from kldload(8).

Sat, Feb 7, 10:23 PM · manpages, docs
tembun_bk.ru added inline comments to D55170: kldload.2: Provide more information about file argument.
Sat, Feb 7, 9:50 PM · manpages, docs
tembun_bk.ru requested review of D55170: kldload.2: Provide more information about file argument.
Sat, Feb 7, 9:30 PM · manpages, docs
tembun_bk.ru requested review of D55169: kldload.2: Spell kld in lowercase.
Sat, Feb 7, 8:54 PM · manpages, docs
vladlen requested review of D55154: fix links in en/articles/ipsec-must.
Sat, Feb 7, 12:10 PM · docs