cleanup: set generic version for devel/subversion

[michaelo]

Summary

PR: 253566
Approved by: jrm (mentor), otis (mentor), maintainer timeout

[michaelo]

Add review URL

[michaelo]

This is a followup to https://reviews.freebsd.org/D42348.

[otis]

devel/p5-SVN-Hooks/Makefile

20

I wonder why are > and >= intermixed here.

[michaelo]

devel/p5-SVN-Hooks/Makefile

20

Looking at the Porter's Handbook it should be >0. I will fix for hunks I have changed. Good catch.

[michaelo]

Address comments from otis

[michaelo]

Issue addressed.

[michaelo]

Guys, please have a look at this again. It is straight forward.

[jrm]

devel/p5-SVN-Hooks/Makefile

20

If no particular version is required, just omit the version range specifier. That is, just use svn:devel/subversion as you did below for portshaker.

devel/p5-SVN-Look/Makefile

18

Same comment as above.

devel/statsvn/Makefile

14

Same comment as above.

[michaelo]

Drop portshaker because it is an invalid change.

[michaelo]

While you are right, there is a difference. Portshaker depends on the executable svn, but the other ports depend on the package. I don't know whether they do use the executable or not. Having said that, I think this is still correct. The Portshaker hunk is wrong, antoine@ pointed me to this.