gifconfig_gif0 no longer works, document replacement

[karels]

Summary

rc.conf(5) documents the gifconfig_<interface> keyword, which is
no longer implemented. Document the replacement, which works with
cloned_interfaces as well.

Test Plan

tested with FreeBSD 11.1

[bcr]

This is a content change, so you need to bump the .Dd to the date of the commit.

[karels]

Update Dd date

[wblock]

share/man/man5/rc.conf.5

1804

Not yours, I know, but "is assumed to exist" is terrible. Assumed by who? This kind of says "the code makes assumptions that will probably break and we are just letting you know."

Is there any reason this can't be changed to just

variable exists for each value of

[karels]

share/man/man5/rc.conf.5

1804

Good point. How about "variable should exist for each value"?

[karels]

share/man/man5/rc.conf.5

1804

Or maybe " variable should be defined for each value of"? The variable needs to be defined in /etc/rc.conf.

[wblock]

share/man/man5/rc.conf.5

1804

Hm, I did not take the wording to mean that the user is supposed to define those variables, but that the system did it. An example of the importance of wording, I guess.

Anyway, let's just say that in an imperative way (roughly paraphrasing):

For each tunnel interface, set a variable named
.Va ifconfig_ Ns Aq Ar interface
to the values desired to configure the link layer of the tunnel with the

A complete example would be good here. The one given does not show an actual example of actual naming of the variable.

1810–1812

End this sentence here:

.Xr ifconfig 8 .

1811

And use this for the full example:

For example:
.Bd -literal -offset indent
ifconfig_gif0="blah blah"
.Ed

With real values for the variable name and contents of the quotes. Actually, looking at that, it makes me more confused, and I'm not sure if the name of the variable on line 1820 is correct. All the more reason for a literal example.

[karels]

Another attempt to clarify, flesh out example.

[karels]

share/man/man5/rc.conf.5

1811

Sorry, I hadn't noticed this comment before uploading a new version. I can touch up the macro usage, but will wait for comments on the content.

[karels]

share/man/man5/rc.conf.5

1811

I tried the suggested macros, and they put a blank line before the display but not after, which looks odd. Suggestions for fixing this welcome; i don't think this is a whole line of space in the typeset version.

[karels]

share/man/man5/rc.conf.5

1811

Warren (or anyone), suggestions on how to do this display with reasonable spacing (same before and after)? Any comments on the text?

[bjk]

share/man/man5/rc.conf.5

1811

I've seen some other places that put a '.Pp' after the '.Ed' (on its own line), which seems to add the desired spacing.

[karels]

Fix macro usage, update text

[karels]

Update date.

I forgot to update the date in the last update. I'll optimistically
set it to Nov 30 so I might not have to update if the review is approved.