subversion 1.9.0+ causes port lint Portfile property checks to fail

[dbevans (David B. Evans)]

This concerns the behavior of the port lint command when the standard svn properties have not been set on a given Portfile.

Using subversion 1.8.14 and earlier, failure to set these properties on the Portfile results in the useful message:

--->  Verifying Portfile for argyll
Error: Missing subversion property on Portfile, please execute: svn ps svn:keywords Id Portfile
--->  1 errors and 0 warnings found.

However, after upgrading to subversion 1.9.0, the message is less helpful:

$ port lint
--->  Verifying Portfile for argyll
Warning: Unable to check for svn:keywords property: svn: warning: W200017: Property 'svn:keywords' not found on '/opt/macports/trunk/dports/graphics/argyll/Portfile'
svn: E200000: A problem occurred; see other errors for details
child process exited abnormally
--->  0 errors and 0 warnings found.

The cause is due to the fact that, in 1.9.0, subversion now considers it an error if propget is executed on a file when the target property does not exist.

In the earlier subversion versions, subversion does not generate an error and just returns with null output.

The current code depends on this latter behavior to identify when a property has not been set. If subversion returns an error, the error is printed as shown in the 1.9.0 case. The new 1.9.0 behavior in this situation needs to be accommodated as well as that of the older versions.

Tested in MacPorts 2.3.3 but examination of current subversion HEAD shows that the problem has not yet been dealt with.

[Ionic (Mihai Moldovan)]

Cc Me!

[dbevans (David B. Evans)]

Still an issue with MacPorts 2.3.4

[ryandesign (Ryan Carsten Schmidt)]

Since we're moving from Subversion to Git and won't be using Subversion properties anymore, this lint check should be removed.

[ryandesign (Ryan Carsten Schmidt)]

r153865

This still needs to be added to the changelog (if we want it listed there), and merged to the 2.3 branch.