Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2

Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2

[Slava Semushin]

2010/8/13 Girar Builder robot <girar-builder@altlinux.org>:
> http://git.altlinux.org/tasks/27624/task/log
>
> 2010-Aug-13 12:01:01 :: task #27624 for sisyphus started:
> #1 build 0.15.12-alt2 from /people/php_coder/packages/mpd-mini.git
> 2010-Aug-13 12:01:02 :: [x86_64] #1 mpd-mini.git 0.15.12-alt2: build start
> 2010-Aug-13 12:01:02 :: [i586] #1 mpd-mini.git 0.15.12-alt2: build start
> 2010-Aug-13 12:02:10 :: [x86_64] #1 mpd-mini.git 0.15.12-alt2: build OK
> 2010-Aug-13 12:02:10 :: [i586] #1 mpd-mini.git 0.15.12-alt2: build OK
> 2010-Aug-13 12:02:12 :: build check OK
> 2010-Aug-13 12:02:12 :: noarch check OK
> 2010-Aug-13 12:02:13 :: plan OK
> 2010-Aug-13 12:02:13 :: version check OK
> 2010-Aug-13 12:03:02 :: created test repo
> 2010-Aug-13 12:03:04 :: dependencies check OK
> 2010-Aug-13 12:04:22 :: ELF symbols check OK
>        x86_64: mpd-mini=0.15.12-alt2 install failed:
> error: failed dependencies:
>        mpd conflicts with mpd-mini-0.15.12-alt2
> hsh-install: Packages installation failed.
>        i586: mpd-mini=0.15.12-alt2 install failed:
> error: failed dependencies:
>        mpd conflicts with mpd-mini-0.15.12-alt2
> hsh-install: Packages installation failed.
> 2010-Aug-13 12:04:36 :: [i586] install check FAILED
> 2010-Aug-13 12:04:36 :: [x86_64] install check FAILED
> 2010-Aug-13 12:04:36 :: task #27624 for sisyphus FAILED

Попытался исправить баг
https://bugzilla.altlinux.org/show_bug.cgi?id=23881 (пожалуйста,
снимите с него гриф секретности!)

Суть бага в том, что при установке пакета mpd-mini выносится не только
mpd (что логично, так как они конфликтуют), но также и пакеты, которые
зависят от mpd (например, sonata). Тогда я попытался добавить
Provides: mpd, чтобы они не выносились. И оно вроде бы работало. Но
судя по логу сборки стала обламываться установка, так как пакет с
одной стороны конфликтует с mpd, а с другой его же и предоставляет.

Как быть? Может быть завести какой-то виртуальный provides типа
music-player-daemon, от которого бы зависели те программы и который бы
провайдили mpd с mpd-mini? Или можно как-то проще?

Спасибо!

-- 
Slava Semushin

Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2

[Alexey Tourbin]

On Fri, Aug 13, 2010 at 03:14:28PM +0700, Slava Semushin wrote:
> Попытался исправить баг
> https://bugzilla.altlinux.org/show_bug.cgi?id=23881 (пожалуйста,
> снимите с него гриф секретности!)
> 
> Суть бага в том, что при установке пакета mpd-mini выносится не только
> mpd (что логично, так как они конфликтуют), но также и пакеты, которые
> зависят от mpd (например, sonata). Тогда я попытался добавить
> Provides: mpd, чтобы они не выносились. И оно вроде бы работало. Но
> судя по логу сборки стала обламываться установка, так как пакет с
> одной стороны конфликтует с mpd, а с другой его же и предоставляет.

По идее никакой пакет не должен конфликтовать сам с собой, даже если
в нём написано Provides: %name, Obsoletes: %name.  Похоже, что apt так
и считает, а вот исправить rpm на этот счёт сложнее.  Поскольку там
один и тот же код проверяет Requires и Conflicts зависимости.  Только
если Requires зависимость удовлетворена, то это считается хорошо, а если
Conflicts зависимость удовлетворена, то это считается что плохо.-)

> Как быть? Может быть завести какой-то виртуальный provides типа
> music-player-daemon, от которого бы зависели те программы и который бы
> провайдили mpd с mpd-mini? Или можно как-то проще?

Лучше ставить зависимость на e.g. /usr/bin/mpd.

Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2

[Slava Semushin]

13 августа 2010 г. 15:50 пользователь Alexey Tourbin <at@altlinux.ru> написал:
[...]
>> Как быть? Может быть завести какой-то виртуальный provides типа
>> music-player-daemon, от которого бы зависели те программы и который бы
>> провайдили mpd с mpd-mini? Или можно как-то проще?
>
> Лучше ставить зависимость на e.g. /usr/bin/mpd.

Спасибо. Развесил баги на соответствующие пакеты:

https://bugzilla.altlinux.org/show_bug.cgi?id=23885
https://bugzilla.altlinux.org/show_bug.cgi?id=23886


-- 
Slava Semushin

Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2

[Ildar Mulyukov]

On 13.08.2010 14:14:28, Slava Semushin wrote:
> пакеты, которые зависят от mpd (например, sonata).
ИМХО это, скорее, неправильно.
-- 
Ildar  Mulyukov,  free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar.mulyukov@gmail.com
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================

[devel] permit self-conflicting packages?

[Alexey Tourbin]

On Fri, Aug 13, 2010 at 12:50:47PM +0400, Alexey Tourbin wrote:
> По идее никакой пакет не должен конфликтовать сам с собой, даже если
> в нём написано Provides: %name, Obsoletes: %name.  Похоже, что apt так
> и считает, а вот исправить rpm на этот счёт сложнее.  Поскольку там
> один и тот же код проверяет Requires и Conflicts зависимости.  Только
> если Requires зависимость удовлетворена, то это считается хорошо, а если
> Conflicts зависимость удовлетворена, то это считается что плохо.-)

В общем можно разрешать установку пакетов, которые конфликтуют сами
с собой.  Точнее, можно сделать, чтобы конфликт в пределах одного пакета
не срабатывал.  Но это нарушает симметрию Requires и Conflicts
(поскольку Requires может разрешаться в свой собственный пакет,
если есть соответствующий Provides).

Впрочем, симметрия Requires и Conflicts - overgeneralization.

index 304dffe..7d49491 100644
--- a/lib/depends.c
+++ b/lib/depends.c
@@ -534,7 +534,8 @@ int dbSatisfiesDepend(rpmTransactionSet ts,
  * Check key for an unsatisfied dependency.
  * @todo Eliminate rpmrc provides.
  * @param ts		transaction set
- * @param keyType	type of dependency
+ * @param h		header the dependency comes from
+ * @param tag		RPMTAG_REQUIRENAME or RPMTAG_CONFLICTNAME
  * @param keyDepend	dependency string representation
  * @param keyName	dependency name string
  * @param keyEVR	dependency [epoch:]version[-release] string
@@ -543,10 +544,11 @@ int dbSatisfiesDepend(rpmTransactionSet ts,
  */
 static int tsSatisfiesDepend(rpmTransactionSet ts,
 		hashTable dbProvCache,
-		const char * keyType, const char * keyDepend,
+		Header h, rpmTag tag, const char * keyDepend,
 		const char * keyName, const char * keyEVR, int keyFlags)
 	/*@modifies ts @*/
 {
+    const char *keyType = (tag == RPMTAG_CONFLICTNAME) ? " Conflicts" : " Requires"Conflicts;
     /*
      * New features in rpm packaging implicitly add versioned dependencies
      * on rpmlib provides. The dependencies look like "rpmlib(YaddaYadda)".
@@ -561,10 +563,28 @@ static int tsSatisfiesDepend(rpmTransactionSet ts,
 	goto unsatisfied;
     }
 
-    if (alSatisfiesDepend(&ts->addedPackages, keyName, keyEVR, keyFlags)) {
-	rpmMessage(RPMMESS_DEBUG, _("%s: %-45s YES (added provides)\n"),
-			keyType, keyDepend+2);
-	return 0;
+    struct availablePackage **pp =
+	    alAllSatisfiesDepend(&ts->addedPackages, keyName, keyEVR, keyFlags);
+    if (pp) {
+	int ret = 1;
+	if (tag != RPMTAG_CONFLICTNAME)
+	    ret = 0;
+	else {
+	    // Conflicts are special, permit self-conflicting packages.
+	    struct availablePackage **alpp;
+	    for (alpp = pp; *alpp; alpp++) {
+		if ((*alpp)->h != h) {
+		    ret = 0;
+		    break;
+		}
+	    }
+	}
+	pp = _free(pp);
+	if (ret == 0) {
+	    rpmMessage(RPMMESS_DEBUG, _("%s: %-45s YES (added provides)\n"),
+			    keyType, keyDepend+2);
+	    return 0;
+	}
     }
 
     if (dbSatisfiesDepend(ts, dbProvCache, keyName, keyEVR, keyFlags) == 0) {
@@ -629,7 +649,7 @@ static int checkPackageDeps(rpmTransactionSet ts, problemsSet psp,
 	keyDepend = printDepend("R",
 		requires[i], requiresEVR[i], requireFlags[i]);
 
-	rc = tsSatisfiesDepend(ts, dbProvCache, " Requires", keyDepend,
+	rc = tsSatisfiesDepend(ts, dbProvCache, h, RPMTAG_REQUIRENAME, keyDepend,
 		requires[i], requiresEVR[i], requireFlags[i]);
 
 	switch (rc) {
@@ -687,7 +707,7 @@ static int checkPackageDeps(rpmTransactionSet ts, problemsSet psp,
 
 	keyDepend = printDepend("C", conflicts[i], conflictsEVR[i], conflictFlags[i]);
 
-	rc = tsSatisfiesDepend(ts, dbProvCache, "Conflicts", keyDepend,
+	rc = tsSatisfiesDepend(ts, dbProvCache, h, RPMTAG_CONFLICTNAME, keyDepend,
 		conflicts[i], conflictsEVR[i], conflictFlags[i]);
 
 	/* 1 == unsatisfied, 0 == satsisfied */