[pacman-dev] [PATCH] New pactest: unresolvable001.py
To test the regression of commit a4084b91. Signed-off-by: Nagy Gabor <ngaba@bibl.u-szeged.hu> --- pactest/tests/unresolvable001.py | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) create mode 100644 pactest/tests/unresolvable001.py diff --git a/pactest/tests/unresolvable001.py b/pactest/tests/unresolvable001.py new file mode 100644 index 0000000..c98a4c0 --- /dev/null +++ b/pactest/tests/unresolvable001.py @@ -0,0 +1,21 @@ +self.description = "An unresolvable dependency" + +packageA1 = pmpkg("dep") +self.addpkg2db("local", packageA1) + +packageA1up = pmpkg("dep", "2.0-1") +packageA1up.depends = ["fake"]; +self.addpkg2db("sync", packageA1up) + +packageA2up = pmpkg("package") +packageA2up.depends = ["dep"]; +self.addpkg2db("sync", packageA2up) + +self.args = "-S package dep --ask=32" + +self.addrule("PACMAN_RETCODE=0") +self.addrule("PKG_EXIST=package") +self.addrule("PKG_EXIST=dep") +self.addrule("PKG_VERSION=dep|1.0-1") + +self.expectfailure = True -- 1.7.1
On Mon, May 17, 2010 at 2:41 PM, Nagy Gabor <ngaba@bibl.u-szeged.hu> wrote:
To test the regression of commit a4084b91.
This might not be the ID on the main branch so that might not work. But with that said, I thought you said forget about it completely in that other email? What is the plan here?
Signed-off-by: Nagy Gabor <ngaba@bibl.u-szeged.hu> --- pactest/tests/unresolvable001.py | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) create mode 100644 pactest/tests/unresolvable001.py
diff --git a/pactest/tests/unresolvable001.py b/pactest/tests/unresolvable001.py new file mode 100644 index 0000000..c98a4c0 --- /dev/null +++ b/pactest/tests/unresolvable001.py @@ -0,0 +1,21 @@ +self.description = "An unresolvable dependency" + +packageA1 = pmpkg("dep") +self.addpkg2db("local", packageA1) + +packageA1up = pmpkg("dep", "2.0-1") +packageA1up.depends = ["fake"]; +self.addpkg2db("sync", packageA1up) + +packageA2up = pmpkg("package") +packageA2up.depends = ["dep"]; +self.addpkg2db("sync", packageA2up) + +self.args = "-S package dep --ask=32" + +self.addrule("PACMAN_RETCODE=0") +self.addrule("PKG_EXIST=package") +self.addrule("PKG_EXIST=dep") +self.addrule("PKG_VERSION=dep|1.0-1") + +self.expectfailure = True -- 1.7.1
On Mon, May 17, 2010 at 2:41 PM, Nagy Gabor <ngaba@bibl.u-szeged.hu> wrote:
To test the regression of commit a4084b91.
This might not be the ID on the main branch so that might not work.
But with that said, I thought you said forget about it completely in that other email? What is the plan here?
Yes, this is an "attachment" for that mail. We are discussing this with Xavier on jabber atm. Possible solutions: 1. Revert the phonon/qt fix. 2. Remove the whole remove_unresolvable stuff. :-) 3. Don't do anything with this regression. 4. Better fix for the phonon/qt issue. [???] I vote for 3. NG
On Tue, May 18, 2010 at 1:09 AM, Dan McGee <dpmcgee@gmail.com> wrote:
On Mon, May 17, 2010 at 2:41 PM, Nagy Gabor <ngaba@bibl.u-szeged.hu> wrote:
To test the regression of commit a4084b91.
This might not be the ID on the main branch so that might not work.
But with that said, I thought you said forget about it completely in that other email? What is the plan here?
In that test, the user asks explicitly to install/upgrade package AND dep. dep is unresolvable and could be skipped. remove unresolvable feature used to allow that, but after phonon/qt fix, it is no longer be allowed. However, if the user asked explicitly to upgrade dep and this cannot be done, a failure in such a situation doesn't sound too bad. And he can still install package if he wants with pacman -S package. And in the -Su situation, maybe dep could be ignored or something ? Anyway, if we are just talking about unresolvable situations, those are just failure situations that should not happen frequently, and when they happen, they should be fixed right away. So no big deal imo. That said, I might have misunderstood something, and I didn't do any real testing to confirm any of this :)
participants (3)
-
Dan McGee
-
Nagy Gabor
-
Xavier Chantry