search for: db_dump

We went around the houses a few times last year in order to try to fix this old Debian bug: https://bugzilla.redhat.com/show_bug.cgi?id=1409024 My last attempt was: https://www.redhat.com/archives/libguestfs/2017-October/msg00058.html which I believe was neither reviewed nor rejected, so I'm reposting the same patch again, simply rebased against current git. Rich.

We do this by defining DB_DUMP unconditionally and then testing the special exit code given by the shell if the command is not found (see http://www.tldp.org/LDP/abs/html/exitcodes.html). Packagers may either: (1) Provide db_dump as a build requirement, but make it a weak dependency at runtime, or: (2) Not provide db_dump at...

...stfs/2017-October/msg00032.html This takes a completely different approach. It turns out that POSIX / the shell already defines a special exit code 127 for ‘command not found’. We can make a small adjustment to lib/command.c to return this exit code in that case. Then we just have to modify the db_dump code to test for this exit code. I also made db_dump unconditional, allowing the packager to either build without db_dump present at all, and/or to make db_dump into a weak runtime dependency which will be silently ignored if not present by the mechanism above. Rich.

.../lib/inspect-apps.c > +++ b/lib/inspect-apps.c > @@ -122,9 +122,14 @@ guestfs_impl_inspect_list_applications2 (guestfs_h *g, const char *root) > if (STREQ (type, "linux") || STREQ (type, "hurd")) { > if (STREQ (package_format, "rpm")) { > #ifdef DB_DUMP > - ret = list_applications_rpm (g, root); > - if (ret == NULL) > - return NULL; > + /* Allow db_dump to be missing at runtime so that it can be > + * installed as a weak dependency. > + */ > + if (system (DB_DUMP " -V >/dev/null 2...

Fix for: https://bugzilla.redhat.com/show_bug.cgi?id=1409024

...> +++ b/lib/inspect-apps.c > > @@ -122,9 +122,14 @@ guestfs_impl_inspect_list_applications2 (guestfs_h *g, const char *root) > > if (STREQ (type, "linux") || STREQ (type, "hurd")) { > > if (STREQ (package_format, "rpm")) { > > #ifdef DB_DUMP > > - ret = list_applications_rpm (g, root); > > - if (ret == NULL) > > - return NULL; > > + /* Allow db_dump to be missing at runtime so that it can be > > + * installed as a weak dependency. > > + */ > > + if (system...

...8..020a3332c 100644 --- a/lib/inspect-apps.c +++ b/lib/inspect-apps.c @@ -122,9 +122,14 @@ guestfs_impl_inspect_list_applications2 (guestfs_h *g, const char *root) if (STREQ (type, "linux") || STREQ (type, "hurd")) { if (STREQ (package_format, "rpm")) { #ifdef DB_DUMP - ret = list_applications_rpm (g, root); - if (ret == NULL) - return NULL; + /* Allow db_dump to be missing at runtime so that it can be + * installed as a weak dependency. + */ + if (system (DB_DUMP " -V >/dev/null 2>&1") == 0) { +...

...ct-apps.c > > > @@ -122,9 +122,14 @@ guestfs_impl_inspect_list_applications2 (guestfs_h *g, const char *root) > > > if (STREQ (type, "linux") || STREQ (type, "hurd")) { > > > if (STREQ (package_format, "rpm")) { > > > #ifdef DB_DUMP > > > - ret = list_applications_rpm (g, root); > > > - if (ret == NULL) > > > - return NULL; > > > + /* Allow db_dump to be missing at runtime so that it can be > > > + * installed as a weak dependency. > > > +...

...9; which AFAICT leads to a binary build that could never work except with a fixed appliance [fixed appliance doesn't need supermin-helper]. This is a bug, I think. - msgmerge: If not present, localization is disabled. It doesn't really make sense to enable this "at runtime". - db_dump: We conditionally compile some inspection support based on whether db_dump is found or not. This may or may not be a bug, depending on whether you think it's worth inspection carrying around a lot of code that has to be tested in the db_dump present/not-present- at-runtime cases. Also the run...

libguestfs and supermin use 'no' instead of either the basename of the desired application, or 'false', as helper string. This happens when configure does not find things like rpm or supermin during build. Wouldnt it be more logical to use the basename instead of 'no' so that at runtime one has the chance to install the required packages and in addition get an obvious

Brought to you by the "I haven't rebuilt the libguestfs universe in a while" saga -- now with working test suite. Pino Toscano (5): build: remove extra gnulib submodule build: remove extra checks and submodules build: stop using gnulib in test-harness build: remove unused gnulib modules Remove extra entries from podfiles .gitmodules | 3 -- Makefile.am

Brought to you by the "I haven't rebuilt the libguestfs universe in a while" saga. Pino Toscano (5): build: remove extra gnulib submodule build: remove extra checks and submodules build: stop using gnulib in test-harness build: remove unused gnulib modules Remove extra entries from podfiles .gitmodules | 3 -- Makefile.am | 2 - bootstrap

I got a recent report of virt-v2v (via virt-p2v) failing to migrate a Fedora guest. The issue was that its /var/lib/rpm/Packages was bigger than our current limit (~410M vs 300M), hence the inspection failed. I took the liberty to refactor the limits of the these files, bumping the problematic one, and reducing the ones of the others (as they are supposed to be way smaller than the limit). Pino

Inspired by libvirt's virCommand* internal mini-library, this adds some internal APIs for running commands. The first patch contains the new APIs. The subsequent patches change various parts of the library over to use it. Rich.

Hi, the libguestfs library uses a number of external tools; for some of them, we search for them at build time, enabling some feature only if found, and later on assuming at runtime they are installed. However, the situation is more complex than that: - hardcoding the full path means that there is an incoherency in the way some of the tools are used, as some other tools (e.g. qemu-img) are

Hi, continuing what started a couple of weeks ago [1], the attached patch series continues the work in making it easier to build and run libguestfs (in fixed appliance mode) on OSes different than Linux. The provided changes should cause no functional changes on Linux. [1] https://www.redhat.com/archives/libguestfs/2014-October/msg00176.html Thanks, -- Pino Pino Toscano (9): build: check

Hi, the libguestfs library uses a number of external tools; for some of them, we search for them at build time, enabling some feature only if found, and later on assuming at runtime they are installed. However, the situation is more complex than that: - hardcoding the full path means that there is an incoherency in the way some of the tools are used, as some other tools (e.g. qemu-img) are

...STATUS_SIZE (300 * 1000 * 1000) +#define MAX_DPKG_STATUS_SIZE (50 * 1000 * 1000) /* Maximum APK 'installed' file we will download to /tmp. */ -#define MAX_APK_INSTALLED_SIZE (300 * 1000 * 1000) +#define MAX_APK_INSTALLED_SIZE (50 * 1000 * 1000) #ifdef DB_DUMP static struct guestfs_application2_list *list_applications_rpm (guestfs_h *g, const char *root); -- 2.17.2

I'm puzzled by such statements as diff /var/clamav/daily.cvd /var/clamav/daily.cvd.rpmnew Binary files /var/clamav/daily.cvd and /var/clamav/daily.cvd.rpmnew differ I thought the point of rpmnew files was so that we could check what has been changed? Anne -- New to KDE4? - get help from http://userbase.kde.org Just found a cool new feature? Add it to UserBase -------------- next part

...ccounts LDA is exim. For maildir exim writes a maildirsize file and I can see the IMAP contingent in my mailclient. But I can't find a solution for mbox. I tried dict with mysql, db, and pgsql - but notihing works for me. - db: a file is written but it seems to be empty: 1013 [mail] =db_dump-4.6 -p quota.db VERSION=3 format=print type=btree db_pagesize=4096 HEADER=END priv/quota/messages 0 priv/quota/storage 0 DATA=END - mysql: the dict-server connects, but nothing is written to the table dovecot: Jul 02 11:24:13 Info: dict: mysql: Connected to mail the mysql log shows no entr...