[82148] trunk/dports/security/medusa

[Ryan Schmidt]

On Aug 8, 2011, at 16:29, Rainer Müller wrote:

> On 2011-08-08 22:03 , ryandesign at macports.org wrote:
>> Revision: 82148
>>          http://trac.macports.org/changeset/82148
>> Author:   ryandesign at macports.org
>> Date:     2011-08-08 13:03:50 -0700 (Mon, 08 Aug 2011)
>> Log Message:
>> -----------
>> medusa: fix replacing /usr/local with prefix; add pcre dependency; force subversion variant always on since subversion libraries are always used even when not requested
>> 
>> Modified Paths:
>> --------------
>>    trunk/dports/security/medusa/Portfile
>>    trunk/dports/security/medusa/files/patch-configure.in.diff
> 
>> [...]
> 
>> +# All modules end up linked with libsvn_client-1.0.dylib, even if the svn
>> +# module was not requested.
>> +default_variants +subversion
>> +if {![variant_isset subversion]} {
>> +    variant_set subversion
>> +}
>> +
> 
> This variant seems useless. I think it should be removed as we can't
> make a choice here at all.
> 
> Or do you intend to get the inadvertently linking with subversion fixed
> and this is just a temporary workaround?

I left the variant there for now because I hope it can be fixed. I took a quick look at it but my knowledge of autoconfery is not very good.