Page MenuHomeFreeBSD

build(7): fix references to svn(1) and svnlite(1)
ClosedPublic

Authored by fernape on Apr 23 2021, 5:06 PM.
Tags
None
Referenced Files
F131824013: D29949.id88327.diff
Sat, Oct 11, 11:40 AM
F131824012: D29949.id88052.diff
Sat, Oct 11, 11:40 AM
F131824009: D29949.id88201.diff
Sat, Oct 11, 11:40 AM
F131824008: D29949.id.diff
Sat, Oct 11, 11:40 AM
F131823998: D29949.id88317.diff
Sat, Oct 11, 11:40 AM
F131784905: D29949.diff
Sat, Oct 11, 3:40 AM
F131756938: D29949.diff
Fri, Oct 10, 10:28 PM
Unknown Object (File)
Fri, Sep 26, 1:16 AM

Details

Summary

Replace them with references to Git.

PR: 254937
Reported by: m.bueker@berlin.de

Testing:

  • mandoc -Tlint clean
  • igor clean
  • aspell happy
  • man ./build.7 renders the page properly

Diff Detail

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

Event Timeline

fernape added reviewers: 0mp, gbe.
freebsd_igalic.co added inline comments.
share/man/man7/build.7
46

maybe we should mention https://cgit.freebsd.org/ for further instructions

share/man/man7/build.7
46

I would not object to add the URL, however:

  • The URL references the build(7) man page
  • I don't see clear instructions on how to update src

The problem I try to address with the explanation between the parentheses is that we don't have any tools in base to actually update src. I think a pointer to the documentation on how to install git is useful.

0mp requested changes to this revision.Apr 26 2021, 8:06 PM
0mp added inline comments.
share/man/man7/build.7
46

You are missing a period at the end.

This revision now requires changes to proceed.Apr 26 2021, 8:06 PM
fernape edited the summary of this revision. (Show Details)
  • Add missing period. Spotted by 0mp
0mp requested changes to this revision.Apr 27 2021, 12:12 PM

There are a couple of more formatting issues.

share/man/man7/build.7
48
51
52
This revision now requires changes to proceed.Apr 27 2021, 12:12 PM
Fix more formatting issues

Thanks, remember to bump Dd.

This revision is now accepted and ready to land.Apr 28 2021, 2:14 PM