[devel] просьба поругать новый flac.

[Konstantin Pavlov]

02.02.07, Dmitry V. Levin<ldv.altlinux.org> написал(а):
> On Fri, Feb 02, 2007 at 03:41:27AM +0300, Pavlov Konstantin wrote:
> > Как я уже писал ранее, flac-1.1.3 релизнулся, вот и дошли руки подлатать его
> > до соответствия shared libraries policy.
> >
> > Пожалуйста, поругайте
> > http://git.altlinux.org/people/thresh/packages/?p=flac.git
>
> > %define soname 8
> > %define cppsoname 6
>
> Давайте по возможности использовать точные формулировки.
> 8 -- это soversion, soname -- это в данном случае libFLAC.so.8

Fixed.

> > # Automatically added by buildreq on Wed Feb 23 2005
> > BuildRequires: [...]
>
> Пора обновить.
Fixed.

> > Summary: Free Lossless Audio Codec
> > License: GPL
> > Summary: FLAC shared library
> > License: BSD-like
>
> Действительно библиотека BSD-style а утилиты GPL?
Да, это действительно так.

> > %package -n lib%name%soname
>
> Не хватает
> Provides: lib%name = %version-%release
>
> То же самое касается пакета lib%name++%cppsoname.

Fixed.

> > %package -n lib%name-devel
> > PreReq: lib%name%soname = %version-%release
>
> Почему не простой Requires?
Остатки старого спека.

> Почему у самого пакета нет такой зависимости?
> Почему её нет у пакета lib%name++%cppsoname?
Fixed:
  flac Requires: lib%name = %version-%release
  lib%name++%cppsoversion Requires: lib%name = %version-%release

> > %{?!_enable_static:export lt_cv_prog_cc_static_works=no}
>
> Это не актуально уже более 2 лет.
OK

> > -libFLAC___la_LIBADD = ../libFLAC/libFLAC.la
> > +libFLAC___la_LIBADD = ../libFLAC/libFLAC.la -lstdc++
>
> Может это и сработает, но вообще лучше использовать g++ для линковки
> плюсового кода.

Передам апстриму.

> P.S.  Вообще я люблю ругать чужие спек-файлы. :)

Спасибо, f8ce51b1af7d3773e94ee057d837b55a4b9c8df8