Re: [proaudio] CLAM: =libclam-9999 and =NetworkEditor-9999 patch

[Natanael Olaiz <nolaiz@xxxxxxxxx>]

Hi Viktor,

First of all thanks for your reply, your welcome and review of the patch!


El 01/20/11 22:17, Viktor Lazarev escribió:
> Hi Natanael,
>
> Regarding to XML backend:
> libclam-9999 doesnt compile with USE="-xercesc" (I have installed
> xmlpp-1.0.5). 

Oops, sorry about that. I sent the patch without testing all the
possible use flags/dependencies combinations...

> If compare 9999 & 1.4.0, it can be seen that 1.4.0
> already contains lines:
>
> [...]
>
> src_compile() {
> .....
> if use xercesc; then
>  if use xmlpp; then
>   myconf+=" xmlbackend=both"
>  else
>   myconf+=" xmlbackend=xercesc"
>  fi
> else
>  if use xmlpp; then
>   myconf+=" xmlbackend=xmlpp"
>  else
>   myconf+=" xmlbackend=none"
>  fi
> fi
> }
>
> also Sconstruct of svn-version contains:
> opts.Add( EnumVariable( 'xmlbackend', 'XML passivation backend',
> 'xercesc', ('xercesc','xmlpp','both','none')) )
>
> So, I guess live-ebuild should contains the same lines.

You are right. That was about to be my next ebuild patch: allowing the 4
possible combinations, I didn't see that it was already done!


> And one question: which xmlbackend is more preffered? As you can see,
> in 1.4.0 it is xmlpp, is it right?

No, the default option in the SConstruct is in fact xercesc. I'll
include that change in my next ebuild patches...
Also, some time ago (the first time that I tried Gentoo) I wasn't able
to compile clam from the sources because it didn't pass the xerces
tests, and I think it was because xerces-c was not compiled without
iconv use flag, so probably the

    xercesc? ( >=dev-libs/xerces-c-3.1 )

line should be

    xercesc? ( >=dev-libs/xerces-c-3.1[iconv] )


I'll check it ASAP.


Best regards,
Natanael.

> 2011/1/20, Natanael Olaiz <nolaiz@xxxxxxxxx>:
>> Hi,
>>
>>
>> Here is a patch for =media-libs/libclam-9999 (-r1) and
>> =media-sound/NetworkEditor-9999
>>
>> I'm a CLAM developer since a few years, but I'm a pretty new using
>> Gentoo, so probably the ebuilds are not very good...  but at least: I
>> made libclam and NetworkEditor live ebuilds works!
>>
>> See attached changelogs for the changes.
>>
>>
>> One of these days I'll check the release versions of the ebuilds, and
>> the SMSTools, Annotator, Voice2MIDI and chordata... Meanwhile, all
>> suggestions are welcome (I'm learning about the ebuilds and the portage
>> system).
>>
>>
>> And last but not least: thank you all for your work with the overlay!
>>
>>
>> Best regards,
>> Natanael.
>>
>