summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAaron Griffin <aaron@archlinux.org>2007-04-17 02:31:04 +0000
committerAaron Griffin <aaron@archlinux.org>2007-04-17 02:31:04 +0000
commite0f5d4efbeadd9921ec7628888877d8f7e36f8b1 (patch)
tree6e392a411b505e27da2175c4d61b4db78ca8c5d5
parentec83c93ea4a7d4742c5d92438b0da1c469704257 (diff)
downloadpacman-e0f5d4efbeadd9921ec7628888877d8f7e36f8b1.tar.xz
* Nagy Gabor <ngaba@petra.hos.u-szeged.hu>
Make sure ldconfig runs on upgrade * Added 3 pactests to verify (by output only) that ldconfig is run
-rw-r--r--lib/libalpm/add.c2
-rw-r--r--pactest/tests/ldconfig001.py12
-rw-r--r--pactest/tests/ldconfig002.py16
-rw-r--r--pactest/tests/ldconfig003.py14
4 files changed, 43 insertions, 1 deletions
diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c
index 5236b6ab..ddeca317 100644
--- a/lib/libalpm/add.c
+++ b/lib/libalpm/add.c
@@ -837,7 +837,7 @@ int _alpm_add_commit(pmtrans_t *trans, pmdb_t *db)
}
/* run ldconfig if it exists */
- if((trans->type != PM_TRANS_TYPE_UPGRADE) && (handle->trans->state != STATE_INTERRUPTED)) {
+ if(handle->trans->state != STATE_INTERRUPTED) {
_alpm_log(PM_LOG_DEBUG, _("running \"ldconfig -r %s\""), handle->root);
_alpm_ldconfig(handle->root);
}
diff --git a/pactest/tests/ldconfig001.py b/pactest/tests/ldconfig001.py
new file mode 100644
index 00000000..aced413b
--- /dev/null
+++ b/pactest/tests/ldconfig001.py
@@ -0,0 +1,12 @@
+# quick note here - chroot() is expected to fail. We're not checking the
+# validity of the scripts, only that they fire (or try to)
+self.description = "Make sure ldconfig runs on an add operation"
+
+p = pmpkg("dummy")
+self.addpkg(p)
+
+# --debug is necessary to check PACMAN_OUTPUT
+self.args = "--debug -A %s" % p.filename()
+
+self.addrule("PACMAN_RETCODE=0")
+self.addrule("PACMAN_OUTPUT=ldconfig")
diff --git a/pactest/tests/ldconfig002.py b/pactest/tests/ldconfig002.py
new file mode 100644
index 00000000..162f5831
--- /dev/null
+++ b/pactest/tests/ldconfig002.py
@@ -0,0 +1,16 @@
+# quick note here - chroot() is expected to fail. We're not checking the
+# validity of the scripts, only that they fire (or try to)
+self.description = "Make sure ldconfig runs on an upgrade operation"
+
+lp = pmpkg("dummy")
+self.addpkg2db("local", lp)
+
+p = pmpkg("dummy", "1.0-2")
+self.addpkg(p)
+
+# --debug is necessary to check PACMAN_OUTPUT
+self.args = "--debug -U %s" % p.filename()
+
+self.addrule("PACMAN_RETCODE=0")
+self.addrule("PKG_VERSION=dummy|1.0-2")
+self.addrule("PACMAN_OUTPUT=ldconfig")
diff --git a/pactest/tests/ldconfig003.py b/pactest/tests/ldconfig003.py
new file mode 100644
index 00000000..1b48efad
--- /dev/null
+++ b/pactest/tests/ldconfig003.py
@@ -0,0 +1,14 @@
+# quick note here - chroot() is expected to fail. We're not checking the
+# validity of the scripts, only that they fire (or try to)
+self.description = "Make sure ldconfig runs on a sync operation"
+
+sp = pmpkg("dummy")
+self.addpkg2db("sync", sp)
+
+self.args = "-S %s" % sp.name
+
+# --debug is necessary to check PACMAN_OUTPUT
+self.args = "--debug -S %s" % sp.name
+
+self.addrule("PACMAN_RETCODE=0")
+self.addrule("PACMAN_OUTPUT=ldconfig")