* Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 @ 2010-08-13 8:14 ` Slava Semushin 2010-08-13 8:50 ` Alexey Tourbin 2010-08-13 14:28 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Ildar Mulyukov 0 siblings, 2 replies; 5+ messages in thread From: Slava Semushin @ 2010-08-13 8:14 UTC (permalink / raw) To: ALT Linux Team development discussions 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 2010-08-13 8:14 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Slava Semushin @ 2010-08-13 8:50 ` Alexey Tourbin 2010-08-13 9:31 ` Slava Semushin 2010-08-14 19:55 ` [devel] permit self-conflicting packages? Alexey Tourbin 2010-08-13 14:28 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Ildar Mulyukov 1 sibling, 2 replies; 5+ messages in thread From: Alexey Tourbin @ 2010-08-13 8:50 UTC (permalink / raw) To: ALT Linux Team development discussions 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. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 2010-08-13 8:50 ` Alexey Tourbin @ 2010-08-13 9:31 ` Slava Semushin 2010-08-14 19:55 ` [devel] permit self-conflicting packages? Alexey Tourbin 1 sibling, 0 replies; 5+ messages in thread From: Slava Semushin @ 2010-08-13 9:31 UTC (permalink / raw) To: ALT Linux Team development discussions 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [devel] permit self-conflicting packages? 2010-08-13 8:50 ` Alexey Tourbin 2010-08-13 9:31 ` Slava Semushin @ 2010-08-14 19:55 ` Alexey Tourbin 1 sibling, 0 replies; 5+ messages in thread From: Alexey Tourbin @ 2010-08-14 19:55 UTC (permalink / raw) To: ALT Linux Team development discussions 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 */ ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 2010-08-13 8:14 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Slava Semushin 2010-08-13 8:50 ` Alexey Tourbin @ 2010-08-13 14:28 ` Ildar Mulyukov 1 sibling, 0 replies; 5+ messages in thread From: Ildar Mulyukov @ 2010-08-13 14:28 UTC (permalink / raw) To: devel 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 ========================================= ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-08-14 19:55 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2010-08-13 8:14 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Slava Semushin 2010-08-13 8:50 ` Alexey Tourbin 2010-08-13 9:31 ` Slava Semushin 2010-08-14 19:55 ` [devel] permit self-conflicting packages? Alexey Tourbin 2010-08-13 14:28 ` [devel] [#27624] FAILED mpd-mini.git=0.15.12-alt2 Ildar Mulyukov
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git