Page MenuHomeFreeBSD
Differential D16764

www/moinmoincli: A new port to edit MoinMoin wiki pages from a command line
ClosedPublic
Actions

Authored by 0mp on Aug 17 2018, 10:49 AM.
Tags
None
Referenced Files
F115248456: D16764.id46887.diff
Mon, Apr 21, 8:47 PM
Unknown Object (File)
Sun, Apr 20, 7:31 PM
Unknown Object (File)
Wed, Apr 16, 1:27 AM
Unknown Object (File)
Tue, Apr 15, 8:15 PM
Unknown Object (File)
Mon, Apr 14, 1:29 AM
Unknown Object (File)
Sun, Apr 6, 11:55 AM
Unknown Object (File)
Feb 25 2025, 9:20 AM
Unknown Object (File)
Feb 24 2025, 1:59 AM
View All Files
Subscribers
sevan
thj

Details

Reviewers
krion
mat
Commits
rP477785: New port: www/moinmoincli: Edit MoinMoin wiki pages from a command line
Summary

Requested by: thj

Test Plan
  • portlint: OK
  • poudriere: 11.2 i386 OK

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 18912
Build 18565: arc lint + arc unit

Event Timeline

0mp created this revision.Aug 17 2018, 10:49 AM
Harbormaster completed remote builds in B18887: Diff 46828.Aug 17 2018, 10:49 AM
mat added inline comments.Aug 17 2018, 3:44 PM
www/moinmoincli/pkg-plist
4

You need to add an EXAMPLES option to use PORTEXAMPLES.

0mp updated this revision to Diff 46863.Aug 17 2018, 9:21 PM

Remove %%PORTEXAMPLES%% from pkg-plist as the EXAMPLES option is not defined.

Harbormaster completed remote builds in B18912: Diff 46863.Aug 17 2018, 9:21 PM
0mp marked an inline comment as done.Aug 17 2018, 9:22 PM
0mp added inline comments.
www/moinmoincli/pkg-plist
4

The configuration file is actually pretty important so I don't think I should add an EXAMPLES option.

0mp marked an inline comment as done.Aug 17 2018, 9:22 PM
mat added inline comments.Aug 18 2018, 9:52 AM
www/moinmoincli/Makefile
14

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

0mp updated this revision to Diff 46885.Aug 18 2018, 10:31 AM

Move SUB_FILES to the block for standard bsd.port.mk variables.

Thanks mat@!

Harbormaster completed remote builds in B18926: Diff 46885.Aug 18 2018, 10:32 AM
0mp updated this revision to Diff 46887.Aug 18 2018, 10:34 AM

Upload all the changes once again.

It turns out that "arc diff --update" has to reupload all the files everytime it's run.

Harbormaster completed remote builds in B18928: Diff 46887.Aug 18 2018, 10:34 AM
0mp marked 2 inline comments as done.Aug 22 2018, 10:58 AM

@mat, I think it's ready.

mat accepted this revision.Aug 22 2018, 11:34 AM
This revision is now accepted and ready to land.Aug 22 2018, 11:34 AM
Closed by commit rP477785: New port: www/moinmoincli: Edit MoinMoin wiki pages from a command line (authored by 0mp). · Explain WhyAug 22 2018, 1:20 PM
This revision was automatically updated to reflect the committed changes.
0mp added a comment.Aug 22 2018, 1:21 PM

Thanks, @mat!

Revision Contents
Changeset List

PathSize
www/
1 line
27 lines
3 lines
3 lines
5 lines
3 lines

Diff 46863

View Options

www/Makefile

Show All 28 Lines
.\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
.\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY .\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY
.\" OF SUCH DAMAGE. .\" OF SUCH DAMAGE.
.\" .\"
.\" Author: Archie Cobbs <archie@FreeBSD.org> .\" Author: Archie Cobbs <archie@FreeBSD.org>
.\" .\"
.\" $FreeBSD$ .\" $FreeBSD$
.\" .\"
.Dd October 2, 2019 .Dd April 29, 2020
.Dt NG_BRIDGE 4 .Dt NG_BRIDGE 4
.Os .Os
.Sh NAME .Sh NAME
.Nm ng_bridge .Nm ng_bridge
.Nd Ethernet bridging netgraph node type .Nd Ethernet bridging netgraph node type
.Sh SYNOPSIS .Sh SYNOPSIS
.In sys/types.h .In sys/types.h
.In netgraph/ng_bridge.h .In netgraph/ng_bridge.h
▲ Show 20 Lines • Show All 41 Lines • ▼ Show 20 Lines
hooks of one or more hooks of one or more
.Xr ng_ether 4 .Xr ng_ether 4
nodes. nodes.
To connect the host machine to a bridged network, simply connect the To connect the host machine to a bridged network, simply connect the
.Dv upper .Dv upper
hook of an hook of an
.Xr ng_ether 4 .Xr ng_ether 4
node to the bridge node. node to the bridge node.
.Pp
Instead of naming a hook
.Dv linkX
the hook might be also named
.Dv uplinkX .
The node does not learn MAC addresses on uplink hooks, which keeps
dabUnsubmitted
Done
Inline Actions

s/does no/does not/

dab: s/does no/does not/
the internal address table small.
bcrUnsubmitted
Done
Inline Actions

You need to make a line break after a sentence stop.

bcr: You need to make a line break after a sentence stop.
dabUnsubmitted
Done
Inline Actions

I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to capitalize such an abbreviation. Such an abbreviation is usually followed by a comma. Suggest:

the internal address table small; i.e., it's desirable to connect

That would also address the newline after full stop issue. :-)

dab: I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to…
danfeUnsubmitted
Done
Inline Actions

Apart from the above (line break, comma after i.e.), contraction should be expanded: it's -> it is.

danfe: Apart from the above (line break, comma after i.e.), contraction should be expanded: //it's//…
This way it is desirable to connect the
.Dv lower
hook of an
.Xr ng_ether 4
node to an
.Dv uplink
dabUnsubmitted
Done
Inline Actions

s/send out/sent out/

dab: s/send out/sent out/
hook of the bridge, and ignore the complexity of the outside world.
Frames with unknown MACs are always sent out to
.Dv uplink
hooks, so no functionality is lost.
.Sh CONTROL MESSAGES .Sh CONTROL MESSAGES
This node type supports the generic control messages, plus the This node type supports the generic control messages, plus the
following: following:
.Bl -tag -width foo .Bl -tag -width foo
.It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig .It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig
Set the node configuration. Set the node configuration.
This command takes a This command takes a
.Dv "struct ng_bridge_config" .Dv "struct ng_bridge_config"
Show All 32 Lines
Returns the current configuration as a Returns the current configuration as a
.Dv "struct ng_bridge_config" . .Dv "struct ng_bridge_config" .
.It Dv NGM_BRIDGE_RESET Pq Ic reset .It Dv NGM_BRIDGE_RESET Pq Ic reset
Causes the node to forget all hosts and unmute all links. Causes the node to forget all hosts and unmute all links.
The node configuration is not changed. The node configuration is not changed.
.It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats .It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
returns a returns a
.Dv "struct ng_bridge_link_stats" .Dv "struct ng_bridge_link_stats"
containing statistics for the corresponding link, which must be containing statistics for the corresponding
yuripvUnsubmitted
Done
Inline Actions

just a note as .Dv seems to be misused throughout this entire man page -- .Dv does not seem to be the correct choice for struct ng_bridge_link_stats, use .Vt? and for link, may be .Va?

yuripv: just a note as `.Dv` seems to be misused throughout this entire man page -- `.Dv` does not seem…
donnerAuthorUnsubmitted
Done
Inline Actions

I'd like to keep it consistent for the moment and fix this issue in a separate review.

donner: I'd like to keep it consistent for the moment and fix this issue in a separate review.
currently connected: .Dv link ,
which must be currently connected:
.Bd -literal -offset 0n .Bd -literal -offset 0n
/* Statistics structure (one for each link) */ /* Statistics structure (one for each link) */
struct ng_bridge_link_stats { struct ng_bridge_link_stats {
uint64_t recvOctets; /* total octets rec'd on link */ uint64_t recvOctets; /* total octets rec'd on link */
uint64_t recvPackets; /* total pkts rec'd on link */ uint64_t recvPackets; /* total pkts rec'd on link */
uint64_t recvMulticasts; /* multicast pkts rec'd on link */ uint64_t recvMulticasts; /* multicast pkts rec'd on link */
uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */ uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */
uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */ uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */
uint64_t recvRunts; /* pkts rec'd less than 14 bytes */ uint64_t recvRunts; /* pkts rec'd less than 14 bytes */
uint64_t recvInvalid; /* pkts rec'd with bogus source addr */ uint64_t recvInvalid; /* pkts rec'd with bogus source addr */
uint64_t xmitOctets; /* total octets xmit'd on link */ uint64_t xmitOctets; /* total octets xmit'd on link */
uint64_t xmitPackets; /* total pkts xmit'd on link */ uint64_t xmitPackets; /* total pkts xmit'd on link */
uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */ uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */
uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */ uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */
uint64_t loopDrops; /* pkts dropped due to loopback */ uint64_t loopDrops; /* pkts dropped due to loopback */
uint64_t loopDetects; /* number of loop detections */ uint64_t loopDetects; /* number of loop detections */
uint64_t memoryFailures; /* times couldn't get mem or mbuf */ uint64_t memoryFailures; /* times couldn't get mem or mbuf */
}; };
.Ed .Ed
.Pp
Negative numbers refer to the
.Dv uplink
hooks.
So querying for -7 will get the statistics for hook
dabUnsubmitted
Done
Inline Actions

s/quering/querying/

dab: s/quering/querying/
.Dv uplink7 .
.It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats .It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
clears the statistics for that link. clears the statistics for that link.
.It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats .It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats
Same as Same as
.Dv NGM_BRIDGE_GET_STATS , .Dv NGM_BRIDGE_GET_STATS ,
but also atomically clears the statistics as well. but also atomically clears the statistics as well.
.It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable .It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable
Show All 32 Lines
View Options

www/moinmoincli/

Show All 28 Lines
.\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
.\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY .\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY
.\" OF SUCH DAMAGE. .\" OF SUCH DAMAGE.
.\" .\"
.\" Author: Archie Cobbs <archie@FreeBSD.org> .\" Author: Archie Cobbs <archie@FreeBSD.org>
.\" .\"
.\" $FreeBSD$ .\" $FreeBSD$
.\" .\"
.Dd October 2, 2019 .Dd April 29, 2020
.Dt NG_BRIDGE 4 .Dt NG_BRIDGE 4
.Os .Os
.Sh NAME .Sh NAME
.Nm ng_bridge .Nm ng_bridge
.Nd Ethernet bridging netgraph node type .Nd Ethernet bridging netgraph node type
.Sh SYNOPSIS .Sh SYNOPSIS
.In sys/types.h .In sys/types.h
.In netgraph/ng_bridge.h .In netgraph/ng_bridge.h
▲ Show 20 Lines • Show All 41 Lines • ▼ Show 20 Lines
hooks of one or more hooks of one or more
.Xr ng_ether 4 .Xr ng_ether 4
nodes. nodes.
To connect the host machine to a bridged network, simply connect the To connect the host machine to a bridged network, simply connect the
.Dv upper .Dv upper
hook of an hook of an
.Xr ng_ether 4 .Xr ng_ether 4
node to the bridge node. node to the bridge node.
.Pp
Instead of naming a hook
.Dv linkX
the hook might be also named
.Dv uplinkX .
The node does not learn MAC addresses on uplink hooks, which keeps
dabUnsubmitted
Done
Inline Actions

s/does no/does not/

dab: s/does no/does not/
the internal address table small.
bcrUnsubmitted
Done
Inline Actions

You need to make a line break after a sentence stop.

bcr: You need to make a line break after a sentence stop.
dabUnsubmitted
Done
Inline Actions

I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to capitalize such an abbreviation. Such an abbreviation is usually followed by a comma. Suggest:

the internal address table small; i.e., it's desirable to connect

That would also address the newline after full stop issue. :-)

dab: I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to…
danfeUnsubmitted
Done
Inline Actions

Apart from the above (line break, comma after i.e.), contraction should be expanded: it's -> it is.

danfe: Apart from the above (line break, comma after i.e.), contraction should be expanded: //it's//…
This way it is desirable to connect the
.Dv lower
hook of an
.Xr ng_ether 4
node to an
.Dv uplink
dabUnsubmitted
Done
Inline Actions

s/send out/sent out/

dab: s/send out/sent out/
hook of the bridge, and ignore the complexity of the outside world.
Frames with unknown MACs are always sent out to
.Dv uplink
hooks, so no functionality is lost.
.Sh CONTROL MESSAGES .Sh CONTROL MESSAGES
This node type supports the generic control messages, plus the This node type supports the generic control messages, plus the
following: following:
.Bl -tag -width foo .Bl -tag -width foo
.It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig .It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig
Set the node configuration. Set the node configuration.
This command takes a This command takes a
.Dv "struct ng_bridge_config" .Dv "struct ng_bridge_config"
Show All 32 Lines
Returns the current configuration as a Returns the current configuration as a
.Dv "struct ng_bridge_config" . .Dv "struct ng_bridge_config" .
.It Dv NGM_BRIDGE_RESET Pq Ic reset .It Dv NGM_BRIDGE_RESET Pq Ic reset
Causes the node to forget all hosts and unmute all links. Causes the node to forget all hosts and unmute all links.
The node configuration is not changed. The node configuration is not changed.
.It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats .It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
returns a returns a
.Dv "struct ng_bridge_link_stats" .Dv "struct ng_bridge_link_stats"
containing statistics for the corresponding link, which must be containing statistics for the corresponding
yuripvUnsubmitted
Done
Inline Actions

just a note as .Dv seems to be misused throughout this entire man page -- .Dv does not seem to be the correct choice for struct ng_bridge_link_stats, use .Vt? and for link, may be .Va?

yuripv: just a note as `.Dv` seems to be misused throughout this entire man page -- `.Dv` does not seem…
donnerAuthorUnsubmitted
Done
Inline Actions

I'd like to keep it consistent for the moment and fix this issue in a separate review.

donner: I'd like to keep it consistent for the moment and fix this issue in a separate review.
currently connected: .Dv link ,
which must be currently connected:
.Bd -literal -offset 0n .Bd -literal -offset 0n
/* Statistics structure (one for each link) */ /* Statistics structure (one for each link) */
struct ng_bridge_link_stats { struct ng_bridge_link_stats {
uint64_t recvOctets; /* total octets rec'd on link */ uint64_t recvOctets; /* total octets rec'd on link */
uint64_t recvPackets; /* total pkts rec'd on link */ uint64_t recvPackets; /* total pkts rec'd on link */
uint64_t recvMulticasts; /* multicast pkts rec'd on link */ uint64_t recvMulticasts; /* multicast pkts rec'd on link */
uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */ uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */
uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */ uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */
uint64_t recvRunts; /* pkts rec'd less than 14 bytes */ uint64_t recvRunts; /* pkts rec'd less than 14 bytes */
uint64_t recvInvalid; /* pkts rec'd with bogus source addr */ uint64_t recvInvalid; /* pkts rec'd with bogus source addr */
uint64_t xmitOctets; /* total octets xmit'd on link */ uint64_t xmitOctets; /* total octets xmit'd on link */
uint64_t xmitPackets; /* total pkts xmit'd on link */ uint64_t xmitPackets; /* total pkts xmit'd on link */
uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */ uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */
uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */ uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */
uint64_t loopDrops; /* pkts dropped due to loopback */ uint64_t loopDrops; /* pkts dropped due to loopback */
uint64_t loopDetects; /* number of loop detections */ uint64_t loopDetects; /* number of loop detections */
uint64_t memoryFailures; /* times couldn't get mem or mbuf */ uint64_t memoryFailures; /* times couldn't get mem or mbuf */
}; };
.Ed .Ed
.Pp
Negative numbers refer to the
.Dv uplink
hooks.
So querying for -7 will get the statistics for hook
dabUnsubmitted
Done
Inline Actions

s/quering/querying/

dab: s/quering/querying/
.Dv uplink7 .
.It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats .It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
clears the statistics for that link. clears the statistics for that link.
.It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats .It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats
Same as Same as
.Dv NGM_BRIDGE_GET_STATS , .Dv NGM_BRIDGE_GET_STATS ,
but also atomically clears the statistics as well. but also atomically clears the statistics as well.
.It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable .It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable
Show All 32 Lines
View Options

www/moinmoincli/Makefile

Show All 28 Lines
.\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
.\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY .\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY
.\" OF SUCH DAMAGE. .\" OF SUCH DAMAGE.
.\" .\"
.\" Author: Archie Cobbs <archie@FreeBSD.org> .\" Author: Archie Cobbs <archie@FreeBSD.org>
.\" .\"
.\" $FreeBSD$ .\" $FreeBSD$
.\" .\"
.Dd October 2, 2019 .Dd April 29, 2020
.Dt NG_BRIDGE 4 .Dt NG_BRIDGE 4
.Os .Os
.Sh NAME .Sh NAME
.Nm ng_bridge .Nm ng_bridge
.Nd Ethernet bridging netgraph node type .Nd Ethernet bridging netgraph node type
.Sh SYNOPSIS .Sh SYNOPSIS
.In sys/types.h .In sys/types.h
.In netgraph/ng_bridge.h .In netgraph/ng_bridge.h
▲ Show 20 Lines • Show All 41 Lines • ▼ Show 20 Lines
hooks of one or more hooks of one or more
.Xr ng_ether 4 .Xr ng_ether 4
nodes. nodes.
To connect the host machine to a bridged network, simply connect the To connect the host machine to a bridged network, simply connect the
.Dv upper .Dv upper
hook of an hook of an
.Xr ng_ether 4 .Xr ng_ether 4
node to the bridge node. node to the bridge node.
.Pp
Instead of naming a hook
.Dv linkX
the hook might be also named
.Dv uplinkX .
The node does not learn MAC addresses on uplink hooks, which keeps
dabUnsubmitted
Done
Inline Actions

s/does no/does not/

dab: s/does no/does not/
the internal address table small.
bcrUnsubmitted
Done
Inline Actions

You need to make a line break after a sentence stop.

bcr: You need to make a line break after a sentence stop.
dabUnsubmitted
Done
Inline Actions

I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to capitalize such an abbreviation. Such an abbreviation is usually followed by a comma. Suggest:

the internal address table small; i.e., it's desirable to connect

That would also address the newline after full stop issue. :-)

dab: I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to…
danfeUnsubmitted
Done
Inline Actions

Apart from the above (line break, comma after i.e.), contraction should be expanded: it's -> it is.

danfe: Apart from the above (line break, comma after i.e.), contraction should be expanded: //it's//…
This way it is desirable to connect the
.Dv lower
hook of an
.Xr ng_ether 4
node to an
.Dv uplink
dabUnsubmitted
Done
Inline Actions

s/send out/sent out/

dab: s/send out/sent out/
hook of the bridge, and ignore the complexity of the outside world.
Frames with unknown MACs are always sent out to
.Dv uplink
hooks, so no functionality is lost.
.Sh CONTROL MESSAGES .Sh CONTROL MESSAGES
This node type supports the generic control messages, plus the This node type supports the generic control messages, plus the
following: following:
.Bl -tag -width foo .Bl -tag -width foo
.It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig .It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig
Set the node configuration. Set the node configuration.
This command takes a This command takes a
.Dv "struct ng_bridge_config" .Dv "struct ng_bridge_config"
Show All 32 Lines
Returns the current configuration as a Returns the current configuration as a
.Dv "struct ng_bridge_config" . .Dv "struct ng_bridge_config" .
.It Dv NGM_BRIDGE_RESET Pq Ic reset .It Dv NGM_BRIDGE_RESET Pq Ic reset
Causes the node to forget all hosts and unmute all links. Causes the node to forget all hosts and unmute all links.
The node configuration is not changed. The node configuration is not changed.
.It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats .It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
returns a returns a
.Dv "struct ng_bridge_link_stats" .Dv "struct ng_bridge_link_stats"
containing statistics for the corresponding link, which must be containing statistics for the corresponding
yuripvUnsubmitted
Done
Inline Actions

just a note as .Dv seems to be misused throughout this entire man page -- .Dv does not seem to be the correct choice for struct ng_bridge_link_stats, use .Vt? and for link, may be .Va?

yuripv: just a note as `.Dv` seems to be misused throughout this entire man page -- `.Dv` does not seem…
donnerAuthorUnsubmitted
Done
Inline Actions

I'd like to keep it consistent for the moment and fix this issue in a separate review.

donner: I'd like to keep it consistent for the moment and fix this issue in a separate review.
currently connected: .Dv link ,
which must be currently connected:
.Bd -literal -offset 0n .Bd -literal -offset 0n
/* Statistics structure (one for each link) */ /* Statistics structure (one for each link) */
struct ng_bridge_link_stats { struct ng_bridge_link_stats {
uint64_t recvOctets; /* total octets rec'd on link */ uint64_t recvOctets; /* total octets rec'd on link */
uint64_t recvPackets; /* total pkts rec'd on link */ uint64_t recvPackets; /* total pkts rec'd on link */
uint64_t recvMulticasts; /* multicast pkts rec'd on link */ uint64_t recvMulticasts; /* multicast pkts rec'd on link */
uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */ uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */
uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */ uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */
uint64_t recvRunts; /* pkts rec'd less than 14 bytes */ uint64_t recvRunts; /* pkts rec'd less than 14 bytes */
uint64_t recvInvalid; /* pkts rec'd with bogus source addr */ uint64_t recvInvalid; /* pkts rec'd with bogus source addr */
uint64_t xmitOctets; /* total octets xmit'd on link */ uint64_t xmitOctets; /* total octets xmit'd on link */
uint64_t xmitPackets; /* total pkts xmit'd on link */ uint64_t xmitPackets; /* total pkts xmit'd on link */
uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */ uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */
uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */ uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */
uint64_t loopDrops; /* pkts dropped due to loopback */ uint64_t loopDrops; /* pkts dropped due to loopback */
uint64_t loopDetects; /* number of loop detections */ uint64_t loopDetects; /* number of loop detections */
uint64_t memoryFailures; /* times couldn't get mem or mbuf */ uint64_t memoryFailures; /* times couldn't get mem or mbuf */
}; };
.Ed .Ed
.Pp
Negative numbers refer to the
.Dv uplink
hooks.
So querying for -7 will get the statistics for hook
dabUnsubmitted
Done
Inline Actions

s/quering/querying/

dab: s/quering/querying/
.Dv uplink7 .
.It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats .It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
clears the statistics for that link. clears the statistics for that link.
.It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats .It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats
Same as Same as
.Dv NGM_BRIDGE_GET_STATS , .Dv NGM_BRIDGE_GET_STATS ,
but also atomically clears the statistics as well. but also atomically clears the statistics as well.
.It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable .It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable
Show All 32 Lines
View Options

www/moinmoincli/distinfo

Show All 28 Lines
.\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
.\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY .\" THIS SOFTWARE, EVEN IF WHISTLE COMMUNICATIONS IS ADVISED OF THE POSSIBILITY
.\" OF SUCH DAMAGE. .\" OF SUCH DAMAGE.
.\" .\"
.\" Author: Archie Cobbs <archie@FreeBSD.org> .\" Author: Archie Cobbs <archie@FreeBSD.org>
.\" .\"
.\" $FreeBSD$ .\" $FreeBSD$
.\" .\"
.Dd October 2, 2019 .Dd April 29, 2020
.Dt NG_BRIDGE 4 .Dt NG_BRIDGE 4
.Os .Os
.Sh NAME .Sh NAME
.Nm ng_bridge .Nm ng_bridge
.Nd Ethernet bridging netgraph node type .Nd Ethernet bridging netgraph node type
.Sh SYNOPSIS .Sh SYNOPSIS
.In sys/types.h .In sys/types.h
.In netgraph/ng_bridge.h .In netgraph/ng_bridge.h
▲ Show 20 Lines • Show All 41 Lines • ▼ Show 20 Lines
hooks of one or more hooks of one or more
.Xr ng_ether 4 .Xr ng_ether 4
nodes. nodes.
To connect the host machine to a bridged network, simply connect the To connect the host machine to a bridged network, simply connect the
.Dv upper .Dv upper
hook of an hook of an
.Xr ng_ether 4 .Xr ng_ether 4
node to the bridge node. node to the bridge node.
.Pp
Instead of naming a hook
.Dv linkX
the hook might be also named
.Dv uplinkX .
The node does not learn MAC addresses on uplink hooks, which keeps
dabUnsubmitted
Done
Inline Actions

s/does no/does not/

dab: s/does no/does not/
the internal address table small.
bcrUnsubmitted
Done
Inline Actions

You need to make a line break after a sentence stop.

bcr: You need to make a line break after a sentence stop.
dabUnsubmitted
Done
Inline Actions

I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to capitalize such an abbreviation. Such an abbreviation is usually followed by a comma. Suggest:

the internal address table small; i.e., it's desirable to connect

That would also address the newline after full stop issue. :-)

dab: I don't think it is common usage to start a sentence with "i.e.", nor is it common usage to…
danfeUnsubmitted
Done
Inline Actions

Apart from the above (line break, comma after i.e.), contraction should be expanded: it's -> it is.

danfe: Apart from the above (line break, comma after i.e.), contraction should be expanded: //it's//…
This way it is desirable to connect the
.Dv lower
hook of an
.Xr ng_ether 4
node to an
.Dv uplink
dabUnsubmitted
Done
Inline Actions

s/send out/sent out/

dab: s/send out/sent out/
hook of the bridge, and ignore the complexity of the outside world.
Frames with unknown MACs are always sent out to
.Dv uplink
hooks, so no functionality is lost.
.Sh CONTROL MESSAGES .Sh CONTROL MESSAGES
This node type supports the generic control messages, plus the This node type supports the generic control messages, plus the
following: following:
.Bl -tag -width foo .Bl -tag -width foo
.It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig .It Dv NGM_BRIDGE_SET_CONFIG Pq Ic setconfig
Set the node configuration. Set the node configuration.
This command takes a This command takes a
.Dv "struct ng_bridge_config" .Dv "struct ng_bridge_config"
Show All 32 Lines
Returns the current configuration as a Returns the current configuration as a
.Dv "struct ng_bridge_config" . .Dv "struct ng_bridge_config" .
.It Dv NGM_BRIDGE_RESET Pq Ic reset .It Dv NGM_BRIDGE_RESET Pq Ic reset
Causes the node to forget all hosts and unmute all links. Causes the node to forget all hosts and unmute all links.
The node configuration is not changed. The node configuration is not changed.
.It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats .It Dv NGM_BRIDGE_GET_STATS Pq Ic getstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
returns a returns a
.Dv "struct ng_bridge_link_stats" .Dv "struct ng_bridge_link_stats"
containing statistics for the corresponding link, which must be containing statistics for the corresponding
yuripvUnsubmitted
Done
Inline Actions

just a note as .Dv seems to be misused throughout this entire man page -- .Dv does not seem to be the correct choice for struct ng_bridge_link_stats, use .Vt? and for link, may be .Va?

yuripv: just a note as `.Dv` seems to be misused throughout this entire man page -- `.Dv` does not seem…
donnerAuthorUnsubmitted
Done
Inline Actions

I'd like to keep it consistent for the moment and fix this issue in a separate review.

donner: I'd like to keep it consistent for the moment and fix this issue in a separate review.
currently connected: .Dv link ,
which must be currently connected:
.Bd -literal -offset 0n .Bd -literal -offset 0n
/* Statistics structure (one for each link) */ /* Statistics structure (one for each link) */
struct ng_bridge_link_stats { struct ng_bridge_link_stats {
uint64_t recvOctets; /* total octets rec'd on link */ uint64_t recvOctets; /* total octets rec'd on link */
uint64_t recvPackets; /* total pkts rec'd on link */ uint64_t recvPackets; /* total pkts rec'd on link */
uint64_t recvMulticasts; /* multicast pkts rec'd on link */ uint64_t recvMulticasts; /* multicast pkts rec'd on link */
uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */ uint64_t recvBroadcasts; /* broadcast pkts rec'd on link */
uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */ uint64_t recvUnknown; /* pkts rec'd with unknown dest addr */
uint64_t recvRunts; /* pkts rec'd less than 14 bytes */ uint64_t recvRunts; /* pkts rec'd less than 14 bytes */
uint64_t recvInvalid; /* pkts rec'd with bogus source addr */ uint64_t recvInvalid; /* pkts rec'd with bogus source addr */
uint64_t xmitOctets; /* total octets xmit'd on link */ uint64_t xmitOctets; /* total octets xmit'd on link */
uint64_t xmitPackets; /* total pkts xmit'd on link */ uint64_t xmitPackets; /* total pkts xmit'd on link */
uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */ uint64_t xmitMulticasts; /* multicast pkts xmit'd on link */
uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */ uint64_t xmitBroadcasts; /* broadcast pkts xmit'd on link */
uint64_t loopDrops; /* pkts dropped due to loopback */ uint64_t loopDrops; /* pkts dropped due to loopback */
uint64_t loopDetects; /* number of loop detections */ uint64_t loopDetects; /* number of loop detections */
uint64_t memoryFailures; /* times couldn't get mem or mbuf */ uint64_t memoryFailures; /* times couldn't get mem or mbuf */
}; };
.Ed .Ed
.Pp
Negative numbers refer to the
.Dv uplink
hooks.
So querying for -7 will get the statistics for hook
dabUnsubmitted
Done
Inline Actions

s/quering/querying/

dab: s/quering/querying/
.Dv uplink7 .
.It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats .It Dv NGM_BRIDGE_CLR_STATS Pq Ic clrstats
This command takes a four byte link number as an argument and This command takes a four byte link number as an argument and
clears the statistics for that link. clears the statistics for that link.
.It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats .It Dv NGM_BRIDGE_GETCLR_STATS Pq Ic getclrstats
Same as Same as
.Dv NGM_BRIDGE_GET_STATS , .Dv NGM_BRIDGE_GET_STATS ,
but also atomically clears the statistics as well. but also atomically clears the statistics as well.
.It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable .It Dv NGM_BRIDGE_GET_TABLE Pq Ic gettable
Show All 32 Lines
View Options

www/moinmoincli/files/

Loading...
View Options

www/moinmoincli/files/pkg-message.in

Loading...
View Options

www/moinmoincli/pkg-descr

Loading...
View Options

www/moinmoincli/pkg-plist

Loading...