Bug 219966

Summary: [handbook] Section 30.8.2: net/isc-dhcp42-server is deprecated
Product: Documentation Reporter: Guangyuan Yang <ygy>
Component: Books & ArticlesAssignee: freebsd-doc (Nobody) <doc>
Status: Closed FIXED    
Severity: Affects Some People CC: bcr, ygy
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Proposed patch none

Description Guangyuan Yang freebsd_committer freebsd_triage 2017-06-13 21:11:27 UTC
Use net/isc-dhcp43-server instead.
Comment 1 Guangyuan Yang freebsd_committer freebsd_triage 2017-07-04 19:53:05 UTC
Created attachment 184060 [details]
Proposed patch
Comment 2 Benedict Reuschling freebsd_committer freebsd_triage 2017-07-08 11:13:36 UTC
Hi Charly,

the file you submitted is not an actual patch, but the source file itself with your changes. It would be much easier to see where in the file your changes are being done by creating a proper patch using "diff -ruN <oldfile> <filewithyourchanges> > change.diff". You can also find quickstart instructions in the FreeBSD Documentation Primer here: 
https://www.freebsd.org/doc/en_US.ISO8859-1/books/fdp-primer/overview.html#overview-quick-start

Cheers

Benedict
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-07-15 14:36:51 UTC
A commit references this bug:

Author: bcr
Date: Sat Jul 15 14:36:09 UTC 2017
New revision: 50500
URL: https://svnweb.freebsd.org/changeset/doc/50500

Log:
  Update net/isc-dhcp42 to 43, as this is the currently
  supported version in ports.

  PR:		219966
  Submitted by:	Guangyuan (Charlie) Yang (yzgyyang@outlook.com)

Changes:
  head/en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml
Comment 4 Benedict Reuschling freebsd_committer freebsd_triage 2017-07-15 14:37:42 UTC
I fixed it, thanks for submitting the bug. PR closed!
Comment 5 Guangyuan Yang freebsd_committer freebsd_triage 2017-07-16 01:24:36 UTC
(In reply to Benedict Reuschling from comment #4)

It's my first time so I didn't know if I was doing it right. Thank you for the detailed instructions!