[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