[MacPorts] #14799: reduce 'port lint' pedantry
#14799: reduce 'port lint' pedantry --------------------------------+------------------------------------------- Reporter: ricci@macports.org | Owner: jberry@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Keywords: | --------------------------------+------------------------------------------- Attached patch reduces 'port lint' pedantry by silencing the following: whitespace checks (except for newline at EOF). They're invisible to port and to humans. patchfile naming needs only have 'patch-', no other requirements. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jberry@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment (by raimue@macports.org): We should discuss the patch naming policy on the list before making changes. I don't know when or how it was decided to be 'patch-*.diff', but why change it to 'patch-*' now and not '*.diff'? I think this was introduced to make it easier for committers to use patches with their favorite editor. Therefore '*.diff' would match this requirement more than 'patch-*'. Also, I think this might have reference at other places (the guide?), where it would have to be changed, too. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:1> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jberry@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Changes (by ryandesign@macports.org): * cc: ryandesign@macports.org (added) Comment: This is not to be implemented until [http://lists.macosforge.org/pipermail /macports-dev/2008-March/004833.html discussion on the mailing list] concludes and a decision has been reached. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:2> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jberry@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment (by wsiegrist@apple.com): The server's portlint.tcl has been updated to the latest. This fixes some bugs from v1.6, however, it completely breaks patchfile name warnings as a "file exists" check was added to fix warning about upstream patches. Auto linting does not have access to the files/ directory, so it thinks all patches are remote since they dont exist. Once a final decision/patch is decided on for portlint, the patchfile exists issue should be included in that. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:3> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jberry@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment (by ryandesign@macports.org): Replying to [comment:3 wsiegrist@apple.com]:
it completely breaks patchfile name warnings as a "file exists" check was added to fix warning about upstream patches. Auto linting does not have access to the files/ directory, so it thinks all patches are remote since they dont exist.
Didn't you cause this issue by making the server's port lint checkout non- recursive in r35474? -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:4> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jmpp@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Changes (by wsiegrist@apple.com): * owner: jberry@macports.org => jmpp@macports.org Comment: Replying to [comment:4 ryandesign@macports.org]:
Didn't you cause this issue by making the server's port lint checkout non-recursive in r35474?
The lint process was always non-recursive to minimize the work done by the server. If "file exists" is the only way to enforce the naming policy then I'll fix it, but like I said in my last comment, I am holding off on any further changes until the policy is decided upon. In effect, this disables patch file name errors completely until a decision is made. I'm going to assign to jmpp for a portmgr decision on this. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:5> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jmpp@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts base bugs Component: base | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Changes (by jmpp@macports.org): * component: ports => base Comment: As I said in my latest post to the dev list, we should: "not advise on patchfile naming in any way (either if that's patch-*.diff, patch-*, *.diff or *.patch) at the lint level. From the very beginning this was only a suggestion that most of us agreed on, so we can still keep it in the guide and elsewhere as such, a suggestion; but it was just that, a suggestion, so I really don't think we have any business annoying a maintainer who, after reading the guide and what-not, has still chosen to name his patches in whatever other way. So my take on this particular issue is that we should remove all patch naming checks from lint, and as a direct result any [file exists foo] checks, which would instantly resolve Bill's comments here." So lets wait a little while in case I have to deal with any flames on the list, but otherwise I'd consider the non-recursive nature of the checkouts and no resulting access to patchfiles a non-issue. -jmpp -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:6> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jmpp@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts 1.6.1 Component: base | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Changes (by jmpp@macports.org): * milestone: MacPorts base bugs => MacPorts 1.6.1 -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:7> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jmpp@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts 1.6.1 Component: base | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment (by afb@macports.org): It would be great if someone could add a whitespace option to the "lint" command, so that one can actually use the tool to check the Portfile even if the autonag email doesn't complain anymore... -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:8> MacPorts </projects/macports> Ports system for Mac OS
#14799: reduce 'port lint' pedantry ---------------------------------+------------------------------------------ Reporter: ricci@macports.org | Owner: jmpp@macports.org Type: defect | Status: new Priority: Normal | Milestone: MacPorts 1.6.1 Component: base | Version: 1.7.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment (by afb@macports.org): like in [http://cvsweb.freebsd.org/ports/ports-mgmt/portlint portlint](1) options: {{{ -t Nit pick about use of spaces. }}} -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14799#comment:9> MacPorts </projects/macports> Ports system for Mac OS
participants (1)
-
MacPorts