svn.tag should be called svn.rev

[ryandesign@…]

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.

[raimue@…]

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.

[raimue@…]

Now the warning is only printed on lint, see r51565, r51569, r51570, r51572, r51574.

[jmr@…]

Looks fixed to me.