Changes between Version 1 and Version 2 of Ticket #68035, comment 1


Ignore:
Timestamp:
Aug 26, 2023, 10:19:33 PM (13 months ago)
Author:
ryandesign (Ryan Carsten Schmidt)
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #68035, comment 1

    v1 v2  
    3232}}}
    3333
    34 But it is not mandatory to eliminate all lint warnings; they're only warnings, not errors, and in this case only appear in nitpick mode. This lint warning was originally created back when portgroups were only included at the top of the Portfile. The idea was that the Portfile should begin with the Subversion `$Id$` comment (removed after we moved to GitHub in 2016), the modeline, a blank line, the PortSystem line, any PortGroup lines, a blank line, and then the name, version, etc. The blank line after the PortGroup line is often omitted when it appears later in the Portfile, such as within a variant or subport. Maybe the lint warning should not appear for such later uses.
     34But it is not mandatory to eliminate all lint warnings; they're only warnings, not errors, and in this case only appear in nitpick mode. This lint warning was originally created back when portgroups were only included at the top of the Portfile. The idea was that the Portfile should begin with the modeline, the Subversion `$Id$` comment (removed after we moved to GitHub in 2016), a blank line, the PortSystem line, any PortGroup lines, a blank line, and then the name, version, etc. The blank line after the PortGroup line is often omitted when it appears later in the Portfile, such as within a variant or subport. Maybe the lint warning should not appear for such later uses.
    3535
    3636> Stranger, after install completes, if I invoke nmap -V it still returns 7.93. o.O