RFE: "lint" action for Portfile verification

[afb@…]

port needs a target to verify that Portfiles are following set criteria, such as required variables or character encoding. It also helps to check that whitespace use and declaration order is uniform.

USAGE: $ sudo port lint MacPorts zlib ---> Verifying Portfile for MacPorts ---> 0 errors and 0 warnings found. ---> Verifying Portfile for zlib Warning: Line 4 should be a newline (after PortSystem) Warning: Line 16 should be a newline (after long_description) Warning: Line 33 has trailing whitespace before newline ---> 0 errors and 3 warnings found.

[afb@…]

macports-portlint.patch

[afb@…]

Should be good to go, for collecting some feedback...

Opinions on script gotten so far on the mailing list:

• tabs and whitespace in Portfiles should be free-form
• newline after long_description might be optional ?
• "knowing" the location of Portfile is bad practice

[jmpalacios (Juan Manuel Palacios)]

Milestone Needs developer review deleted

[nox@…]

This is a duplicate of #463.

[afb@…]

This is not a duplicate of #463, it's the start of a fix/implementation for it.

It was requested that it was posted to Trac as a patch, rather than commited...

[afb@…]

Applied in r28170

[afb@…]

Well, it's in. Enhancements should be new tickets.

(whitespace-checks and long_desc-newline removed, ignoring that knowing Portfile is "bad practice")