[MacPorts] #15484: post-commit lint script should handle Portfiles anywhere in repository
#15484: post-commit lint script should handle Portfiles anywhere in repository -------------------------------------+-------------------------------------- Reporter: ryandesign@macports.org | Owner: wsiegrist@apple.com Type: defect | Status: new Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Keywords: | -------------------------------------+-------------------------------------- If I commit changes to a Portfile in trunk/dports/*/* a [browser:trunk/base/portmgr/jobs/portfile_lint.pl script] runs which checks the Portfile syntax and emails any errors to the committer and maintainer. But if I have a Portfile elsewhere in the repository, such as under users/*/*, then the script emails me an error instead: {{{ Portfile: pure-devel Error: Unable to open port: Could not find Portfile in /tmp/mp_lint/36446/ryandesign/pure-devel }}} trunk/dports seems to be hardcoded into the script. But the script should support Portfiles, wherever in the repository they may be. William said he doesn't have time to look into this right now so I may take a look. -- Ticket URL: <http://trac.macports.org/ticket/15484> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@macports.org | Owner: ryandesign@macports.org Type: defect | Status: new Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: | Keywords: Port: | --------------------------------------+------------------------------------- Changes (by wsiegrist@apple.com): * owner: wsiegrist@apple.com => ryandesign@macports.org Comment: I've modified the lint script on the server. Please verify this is fixed next time you touch a Portfile outside of trunk/dports? -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: new Priority: Normal | Milestone: Website & Documentation Component: server/hosting | Version: 1.6.0 Keywords: | Port: -------------------------------------+-------------------------------------- Changes (by raimue@…): * milestone: => Website & Documentation -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: closed Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: fixed | Keywords: Port: | --------------------------------------+------------------------------------- Changes (by raimue@…): * status: new => closed * resolution: => fixed Comment: It works for me correctly when committing to my `/users/raimue/ports` directory, so I consider this fixed. -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: | Keywords: Port: | --------------------------------------+------------------------------------- Changes (by ryandesign@…): * status: closed => reopened * resolution: fixed => Comment: You have directories in your ports directory for categories. I don't, because I don't need them. -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: | Keywords: Port: | --------------------------------------+------------------------------------- Changes (by dports@…): * cc: dports@… (added) Comment: I'm not sure it should run lint on portfiles not in trunk/dports at all. r68605 just got me 84 lint report emails, mostly telling me that a portgroup file or dependency couldn't be located. No surprise, they're only in my users directory, and I don't really expect it to look there, but I'd rather not be flooded by emails when it doesn't find them... -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: | Keywords: Port: | --------------------------------------+------------------------------------- Comment(by ryandesign@…): So... you're saying the server ''is'' now running `lint` on Portfiles not in trunk/dports? So... this ticket has been resolved? To prevent the errors about the missing portgroup, feel free to put the new portgroup into the public tree already, even if it isn't quite done yet. -- Ticket URL: <http://trac.macports.org/ticket/15484#comment:7> MacPorts <http://www.macports.org/> Ports system for Mac OS
#15484: post-commit lint script should handle Portfiles anywhere in repository --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: ryandesign@… Type: defect | Status: reopened Priority: Normal | Milestone: Component: server/hosting | Version: 1.6.0 Resolution: | Keywords: Port: | --------------------------------------+------------------------------------- Comment(by ryandesign@…): Replying to [comment:7 ryandesign@…]:
So... you're saying the server ''is'' now running `lint` on Portfiles not in trunk/dports? So... this ticket has been resolved? Oh, wait, I see, you too have category directories whereas I didn't. At the time I filed this ticket, it hadn't occurred to me to create category directories in my user directory. I suppose we could just officially recommend that people use category directories for ports in their user directory (if we don't already) and be done with it.
-- Ticket URL: <http://trac.macports.org/ticket/15484#comment:8> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts