Page MenuHomeFreeBSD

src.conf.5: Cross-reference uname(1) man page
ClosedPublic

Authored by tembun_bk.ru on Tue, Jan 13, 2:32 PM.
Referenced Files
Unknown Object (File)
Fri, Jan 30, 1:53 PM
Unknown Object (File)
Fri, Jan 30, 1:41 PM
Unknown Object (File)
Fri, Jan 30, 12:46 PM
Unknown Object (File)
Fri, Jan 30, 12:38 PM
Unknown Object (File)
Sun, Jan 25, 7:17 AM
Unknown Object (File)
Tue, Jan 20, 9:14 PM
Unknown Object (File)
Tue, Jan 20, 2:27 PM
Unknown Object (File)
Tue, Jan 20, 11:20 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I don't think this is the diff you intended to upload based on the commit message title. Also, please format patches with -U9999 so we can see the context

I don't think this is the diff you intended to upload based on the commit message title.

I'm not sure if I did everything correctly, but as far as I understood, src.conf(5) page is generated by tools/build/options/makeman and descriptions for WITH_ and WITHOUT_ options are taken from files in tools/build/options. The part of the man page that I wanted to change is WITHOUT_REPRODUCIBLE_BUILD option description (use .Xr for uname(1) there), so I edited the corresponding file tools/build/options/WITHOUT_REPRODUCIBLE_BUILD and regenerated the man page with tools/build/options/makeman script.

Also, please format patches with -U9999 so we can see the context

As I can see, this patch does have -U9999 (files in tools/build/options are short and src.conf.5 does have all the context available).

Ah, I misunderstood then, sorry. Over lunch I'll test this.

I still do (nonblocking, if anyone else is free) but I'm swamped at $dayjob and idk when I'll be able to get to it.

I still do (nonblocking, if anyone else is free) but I'm swamped at $dayjob and idk when I'll be able to get to it.

Sure, that's ok, thank you!

Pedantically the non-repro output is in both uname(1) and uname(3), but I think this is fine.

This revision is now accepted and ready to land.Thu, Jan 29, 3:42 PM

Patch doesn't apply to main, please rebase

This revision now requires review to proceed.Thu, Jan 29, 5:42 PM
This revision was not accepted when it landed; it landed in state Needs Review.Thu, Jan 29, 5:45 PM
This revision was automatically updated to reflect the committed changes.