* 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
* 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
* [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
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