[MacPorts] #56345: Make it easier to determine which ports have lint problems
MacPorts
noreply at macports.org
Tue Apr 24 00:26:11 UTC 2018
#56345: Make it easier to determine which ports have lint problems
-------------------------+--------------------
Reporter: ryandesign | Owner: (none)
Type: enhancement | Status: new
Priority: Normal | Milestone:
Component: base | Version: 2.4.99
Keywords: | Port:
-------------------------+--------------------
`port lint` does not make it easy to display output only for ports that
have lint warnings or errors and to be silent about those that don't, or
to construct a list of the names of such ports. For example:
{{{
$ port lint maintainer:jberry
---> Verifying Portfile for cronolog-devel
Warning: no license set
---> 0 errors and 1 warnings found.
---> Verifying Portfile for dovecot
---> 0 errors and 0 warnings found.
---> Verifying Portfile for mrtg
---> 0 errors and 0 warnings found.
---> Verifying Portfile for mrtg-ping-probe
Warning: Line 3 has trailing whitespace before newline
Warning: Patchfile mrtg-ping-probe.patch does not follow the source patch
naming policy "patch-*.diff"
---> 0 errors and 2 warnings found.
---> Verifying Portfile for tag
Warning: Line 12 seems to hardcode the version number, consider using
${version} instead
Warning: Line 31 seems to hardcode the version number, consider using
${version} instead
---> 0 errors and 2 warnings found.
}}}
There is a lot of superfluous output about ports that don't have problems.
This is inconvenient if one is trying to find lint problems in a large
number of ports, such as all of the ports a maintainer maintains, or even
the entire set of all ports.
More useful output can be obtained with postprocessing, but it would be
better if MacPorts would produce such output without postprocessing:
{{{
$ port lint maintainer:jberry 2>&1 | perl -ne 'if (/Verifying Portfile for
(.+)$/) {$PORT=$1} elsif (!/\d+ errors and \d+ warnings found/) {print
"$PORT: $_"}'
cronolog-devel: Warning: no license set
mrtg-ping-probe: Warning: Line 3 has trailing whitespace before newline
mrtg-ping-probe: Warning: Patchfile mrtg-ping-probe.patch does not follow
the source patch naming policy "patch-*.diff"
tag: Warning: Line 12 seems to hardcode the version number, consider using
${version} instead
tag: Warning: Line 31 seems to hardcode the version number, consider using
${version} instead
}}}
--
Ticket URL: <https://trac.macports.org/ticket/56345>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list