[MacPorts] #17349: svn.tag should be called svn.rev
#17349: svn.tag should be called svn.rev -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: MacPorts base enhancements Component: base | Version: 1.6.0 Keywords: | Port: -------------------------------------+-------------------------------------- The `svn.tag` keyword should really be called `svn.rev` because it is where you put the revision of the repository you would like to check out. I'm sure it's just called "tag" since it was probably copied from (or maybe even shares code with) the cvs checkout feature, where `cvs.tag` makes sense, since it's the tag name you want to check out. We should allow `svn.rev` as a synonym of `svn.tag`. Then, once we release a version of MacPorts with that change, we should mass-update all ports to use `svn.rev` instead. Then we can modify `port lint` to issue a warning if you use `svn.tag` and suggest you use `svn.rev` instead. -- Ticket URL: <http://trac.macports.org/ticket/17349> MacPorts <http://www.macports.org/> Ports system for Mac OS
#17349: svn.tag should be called svn.rev -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: MacPorts Future Component: base | Version: 1.6.0 Keywords: | Port: -------------------------------------+-------------------------------------- Changes (by ryandesign@…): * cc: raimue@… (added) -- Ticket URL: <http://trac.macports.org/ticket/17349#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#17349: svn.tag should be called svn.rev --------------------------------------------+------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: MacPorts 1.8.0 Component: base | Version: 1.6.0 Keywords: svn.tag svn.revision deprecate | Port: --------------------------------------------+------------------------------- Changes (by raimue@…): * keywords: => svn.tag svn.revision deprecate * milestone: MacPorts Future => MacPorts 1.8.0 Comment: Oh, I have not been aware of this ticket before doing r48223. I made it a ui_warn message now. I will take a look if we can also make it count as a warning in lint by adding another option_proc there. -- Ticket URL: <http://trac.macports.org/ticket/17349#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#17349: svn.tag should be called svn.rev --------------------------------------------+------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: MacPorts 1.8.0 Component: base | Version: 1.6.0 Keywords: svn.tag svn.revision deprecate | Port: --------------------------------------------+------------------------------- Comment(by raimue@…): Now the warning is only printed on lint, see r51565, r51569, r51570, r51572, r51574. -- Ticket URL: <http://trac.macports.org/ticket/17349#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#17349: svn.tag should be called svn.rev --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: closed Priority: Normal | Milestone: MacPorts 1.8.0 Component: base | Version: 1.6.0 Resolution: fixed | Keywords: svn.tag svn.revision deprecate Port: | --------------------------------------+------------------------------------- Changes (by jmr@…): * status: new => closed * resolution: => fixed Comment: Looks fixed to me. -- Ticket URL: <http://trac.macports.org/ticket/17349#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts