Page MenuHomeFreeBSD

Documenting fetchReqHTTP in libfetch manual
ClosedPublic

Authored by farhan_farhan.codes on Jan 9 2019, 7:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 9:16 AM
Unknown Object (File)
Feb 5 2024, 10:29 PM
Unknown Object (File)
Jan 30 2024, 4:39 PM
Unknown Object (File)
Jan 30 2024, 4:36 PM
Unknown Object (File)
Dec 20 2023, 1:02 AM
Unknown Object (File)
Dec 15 2023, 7:40 PM
Unknown Object (File)
Dec 13 2023, 3:37 AM
Unknown Object (File)
Nov 5 2023, 10:44 AM
Subscribers

Details

Summary

Adding documenting for the fetchReqHTTP function in fetch(3)/libfetch.
Also added:
A comment to lib/libfetch/http.c to be consistent with other functions.
Manual page symlink for fetchReqHTTP in Makefile

Test Plan

Review the man page for fetch(3) to ensure that the changes were correctly done.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Adding des; I think this looks OK, but there may be a reason it was undocumented.

Hi,
Any updates on this? It should be a pretty quick documentation update.
Thanks!

Any updates on this? Its been sitting here for quite a while now...

markj added inline comments.
lib/libfetch/Makefile
81

This should be sorted before fetchStat.3.

lib/libfetch/fetch.3
400

Quotes should be formatted with .Ql. So,

to the intended verb, such as
.Ql POST ,
and
.Fa body
to the content.
0mp added a subscriber: 0mp.

OK from manpages assuming that the comments made by @markj will get addressed.

This revision is now accepted and ready to land.Aug 20 2019, 8:14 AM

Updated per @markj 's comments.

I should add, I also have some changes to fetch.3 in D18908 that may conflict. However, it should be easy to reconcile.

This revision now requires review to proceed.Aug 20 2019, 7:49 PM
markj added inline comments.
lib/libfetch/fetch.3
406

This last fragment is duplicated.

This revision is now accepted and ready to land.Aug 21 2019, 3:12 PM

Silly mistake, removed the duplicate.

This revision now requires review to proceed.Aug 23 2019, 5:55 AM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2019, 5:01 PM
Closed by commit rS351573: Document fetchReqHTTP(). (authored by markj). · Explain Why
This revision was automatically updated to reflect the committed changes.