Re: [devel] [SCM] packages/mdadm: tags/4.0-alt2

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

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

Hi,

On Thu, Nov 02, 2017 at 09:11:37AM +0000, Alexey V. Vissarionov wrote:
> Update of /people/gremlin/packages/mdadm.git
> 
> Changes statistics since common ancestor `270b6b59826e674d04a51912755bb47c991dc139' follows:
>  mdadm.spec | 58 ++++++++++++++++++++++++++++++++++++++++++++++++----------
>  1 file changed, 48 insertions(+), 10 deletions(-)
> 
> Changelog since common ancestor `270b6b59826e674d04a51912755bb47c991dc139' follows:
> commit e11470b4cd0ff7cd0548571e359f88dc4ac03f7f
> Author: Gremlin from Kremlin <gremlin@altlinux>
> Date:   Thu Nov 2 11:52:57 2017 +0300
> 
>     split to subpackages to avoid parasitic dependencies;

Заменяем паразитные зависимости на паразитные пакеты?
С какими зависимостями боремся-то?  У mdadm и так зависимостей мало.

>     disable running raidtabtomdadm.sh (there are no raidtabs left)
> 
> Full diff since common ancestor `270b6b59826e674d04a51912755bb47c991dc139' follows:
> diff --git a/mdadm.spec b/mdadm.spec
> index d17566a..9b1cdbc 100644
> --- a/mdadm.spec
> +++ b/mdadm.spec
> @@ -2,7 +2,7 @@
>  
>  Name: mdadm
>  Version: 4.0
> -Release: alt1
> +Release: alt2
>  
>  Summary: A tool for managing Soft RAID under Linux
>  License: GPLv2+
> @@ -15,9 +15,6 @@ Source: %name-%version-%release.tar
>  BuildRequires: binutils-devel
>  %{?_enable_cluster:BuildRequires: libcorosync2-devel libdlm-devel}
>  
> -# due to /lib/udev/rules.d/64-md-raid.rules
> -Conflicts: udev < 151
> -
>  %description
>  mdadm is a program that can be used to create, manage, and monitor
>  Linux MD (Software RAID) devices.
> @@ -28,6 +25,36 @@ program, and it can perform (almost) all functions without a
>  configuration file (that a config file can be used to help with
>  some common tasks).
>  
> +%package tools
> +Summary: Various tools and init script for %name
> +Group: System/Configuration/Hardware
> +Requires: %name = %version
> +%description tools
> +%summary

Ну какие там various tools? init script и cron script.
Зачем отдельный пакет?

> +%package udev
> +Summary: udev rules for %name
> +Group: System/Configuration/Hardware
> +Requires: %name = %version
> +# due to /lib/udev/rules.d/64-md-raid.rules
> +Conflicts: udev < 151
> +%description udev
> +%summary

Зачем отдельный пакет?
Кому могла помешать зависимость на udev-rules?

> +%package systemd
> +Summary: systemd support for %name
> +Group: System/Configuration/Hardware
> +Requires: %name-udev = %version, %name-tools = %version
> +%description systemd
> +%summary

Почему init script и systemd unit files в разных пакетах?

> +%package doc
> +Summary: Optional documentation for %name
> +Group: System/Configuration/Hardware
> +BuildArch: noarch
> +%description doc
> +%summary

Зачем отдельный пакет такого размера (20K) без зависимостей?
Чтобы потерять /usr/share/doc/mdadm*/ANNOUNCE*?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]