Page MenuHomeFreeBSD

sysutils/bsdstats: Update to 7.0
ClosedPublic

Authored by salvadore on Apr 30 2020, 10:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 5:28 PM
Unknown Object (File)
Wed, Apr 17, 2:30 PM
Unknown Object (File)
Mon, Apr 15, 11:57 AM
Unknown Object (File)
Mon, Apr 15, 11:56 AM
Unknown Object (File)
Fri, Apr 12, 7:35 PM
Unknown Object (File)
Tue, Apr 9, 10:00 AM
Unknown Object (File)
Feb 1 2024, 3:42 PM
Unknown Object (File)
Jan 14 2024, 5:12 PM
Subscribers

Details

Summary
  • Use pkg query %n:%v:%o instead of pkg info -o “*” to get list of installed packages. This simplifies processing and accuracy of split for version numbers on the report server by not having to guess where that split is.
  • Check for existence of /usr/local/etc/bsdstats.conf and if it exists, restrict ports submitted to those listed in there.
  • Add a note to pkg-message.in about /usr/local/etc/bsdstats.conf
  • Get rid of the LIMITED config option: LIMITED is enabled if and only if /usr/local/etc/bsdstats.conf exists.

While here, also modernize port using option helpers.

Submitted by: scrappy

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I propose a verbose commit message (based on scrappy's own draft) because the software is developped in the port itself, so a bit more details about the development might be useful in the future.

Looks okay to me, and the longer commit message definitely is a good idea.

One minor note: "While here, also modernize port using option helpers." lacks a "the" or "this" as in "this port".

And should /usr/local be $PREFIX?

This revision is now accepted and ready to land.Apr 30 2020, 10:33 PM
This revision was automatically updated to reflect the committed changes.

Committed taking into account your comments.

Unfortunately I missed an occurence of /usr/local in pkg-message, so I had to make one more commit (I considered it as already approved from you since it is a forgotten part of this one): https://svnweb.freebsd.org/ports?view=revision&revision=533480

Committed taking into account your comments.

Thanks!

Unfortunately I missed an occurence of /usr/local in pkg-message, so I had to make one more commit (I considered it as already approved from you since it is a forgotten part of this one):

Yes, of course. No worries (and no delays in improving/fixing such things while you wait for me to get up/in front of a computer/...).