Re: musepack-tools sv8

On Wed, Nov 24, 2010 at 10:08 AM, Paulo Cavalcanti <[email protected]> wrote:

>
>
> On Wed, Nov 24, 2010 at 7:47 AM, Axel Thimm <[email protected]> wrote:
>
>> Hi Paulo,
>>
>> On Sat, 2010-11-20 at 09:45 -0200, Paulo Cavalcanti wrote:
>> > when I was trying to fix mppdec for F14,
>> > I saw that musepack-tools sv8 was released last year, and contained a
>> > new
>> > mpcdec and other tools (it is compatible with the previous sv7
>> > version).
>>
>> Thanks!
>>
>> > The recipe is simple:
>> >
>> > 1) Rebuild cuetools from Fedora, so it has the missing devel bits
>> > (cuetools-devel).
>>
>> This is the part that has some issues, the packages you submitted built
>> for f14 only and for f13/x86_64. Other platforms fail with (example from
>> f13/i386):
>>
>> gcc -shared  .libs/cd.o .libs/cdtext.o .libs/time.o .libs/cuefile.o
>> .libs/cue_print.o .libs/toc_print.o .libs/cue_parse.o .libs/cue_scan.o
>> .libs/toc_parse.o .libs/toc_scan.o   -m32 -march=i686 -mtune=atom
>> -Wl,-soname -Wl,libcuefile.so.0 -o .libs/libcuefile.so.0.0.0
>> .libs/toc_parse.o:(.data+0x300): multiple definition of `yysindex'
>> .libs/cue_parse.o:(.data+0x200): first defined here
>> /usr/bin/ld: Warning: size of symbol `yysindex' changed from 180 in
>> .libs/cue_parse.o to 310 in .libs/toc_parse.o
>> collect2: ld returned 1 exit status
>> make[4]: *** [libcuefile.la] Error 1
>>
>>
> Weird. Its the package from F14. No difference, other than adding
> some .h bits and keeping the symbolic link of the devel package.
>
> In the worst scenario, we can get the version from F13.
> I compiled the 64 bit version I sent you for F10, F11, F12, and F14,
> but the 32 bits only for F14 ...
>
>
>

Well, it built fine for F13, 32 bits, using mock  (I cleaned the cache),
and also built for RHEL5, 32 bits, recompiling the .src.rpm directly.

gcc -shared  .libs/cd.o .libs/cdtext.o .libs/time.o .libs/cuefile.o
.libs/cue_print.o .libs/toc_print.o .libs/cue_parse.o .libs/cue_scan.o
.libs/toc_parse.o .libs/toc_scan.o   -m32 -march=i686 -mtune=atom
-Wl,-soname -Wl,libcuefile.so.0 -o .libs/libcuefile.so.0.0.0
(cd .libs && rm -f libcuefile.so.0 && ln -s libcuefile.so.0.0.0
libcuefile.so.0)
(cd .libs && rm -f libcuefile.so && ln -s libcuefile.so.0.0.0 libcuefile.so)
creating libcuefile.la

I only found in the source:

[lua:~/redhat/SOURCES/cuetools-1.3.1] grep -R yysindex *
src/lib/toc_parse_prefix.h:#define yysinde        toc_yysindex
src/lib/cue_parse_prefix.h:#define yysinde        cue_yysindex

The package has no BuildRequires. Therefore, I do not see any reason
for it to fail in ATrpms.

Would you have any suggestion?

-- 
Paulo Roma Cavalcanti
LCG - UFRJ