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

["Dmitry V. Levin" <ldv@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 1420 bytes --]

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

> # Automatically added by buildreq on Wed Feb 23 2005
> BuildRequires: [...]

Пора обновить.

> 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.

> %package -n lib%name-devel
> PreReq: lib%name%soname = %version-%release

Почему не простой Requires?
Почему у самого пакета нет такой зависимости?
Почему её нет у пакета lib%name++%cppsoname?

> %{?!_enable_static:export lt_cv_prog_cc_static_works=no}

Это не актуально уже более 2 лет.

> -libFLAC___la_LIBADD = ../libFLAC/libFLAC.la
> +libFLAC___la_LIBADD = ../libFLAC/libFLAC.la -lstdc++

Может это и сработает, но вообще лучше использовать g++ для линковки
плюсового кода.

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


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]