[pacman-dev] [PATCH] libalpm: more verbose error messages on file conflicts
hi http://frugalware.org/~vmiklos/patches/libpacman-proposed/verbose_conflicts.... currently: $ sudo pacman -S foomatic-filters-ppds --noconfirm resolving dependencies... done. looking for inter-conflicts... done. Targets: foomatic-filters-ppds-20051211-1 Total Package Size: 11.5 MB Beginning upgrade process... checking package integrity... done. ERROR: could not prepare transaction error: failed to commit transaction (unexpected error) expected: $ sudo pacman -S foomatic-filters-ppds --noconfirm resolving dependencies... done. looking for inter-conflicts... done. Targets: foomatic-filters-ppds-20051211-1 Total Package Size: 11.5 MB Beginning upgrade process... checking package integrity... done. ERROR: foomatic-filters-ppds: /usr/share/ppd: exists in filesystem ERROR: could not prepare transaction error: failed to commit transaction (conflicting files) the patch mentioned above implements this udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org
VMiklos wrote:
http://frugalware.org/~vmiklos/patches/libpacman-proposed/verbose_conflicts....
currently: $ sudo pacman -S foomatic-filters-ppds --noconfirm resolving dependencies... done. looking for inter-conflicts... done.
Targets: foomatic-filters-ppds-20051211-1
Total Package Size: 11.5 MB
Beginning upgrade process...
checking package integrity... done. ERROR: could not prepare transaction error: failed to commit transaction (unexpected error)
expected: $ sudo pacman -S foomatic-filters-ppds --noconfirm resolving dependencies... done. looking for inter-conflicts... done.
Targets: foomatic-filters-ppds-20051211-1
Total Package Size: 11.5 MB
Beginning upgrade process...
checking package integrity... done. ERROR: foomatic-filters-ppds: /usr/share/ppd: exists in filesystem ERROR: could not prepare transaction error: failed to commit transaction (conflicting files)
the patch mentioned above implements this
Well, with the latest CVS code, this patch does not seem to be needed anymore: src/pacman/add.c is already doing the job to display conflicting files (see the PM_ERR_FILE_CONFLICTS case in the switch(pm_errno) block). It all looks like the PM_ERR_XXX error code was preventing the switch to reach the aforementionned error case. -- Aurelien
On Sat, Jan 07, 2006 at 12:30:57AM +0100, Aurelien Foret <aurelien@archlinux.org> wrote:
Well, with the latest CVS code, this patch does not seem to be needed anymore: src/pacman/add.c is already doing the job to display
yes, the patch is two weeks old :) udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org
On Sat, Jan 07, 2006 at 12:36:03AM +0100, VMiklos <vmiklos@frugalware.org> wrote:
On Sat, Jan 07, 2006 at 12:30:57AM +0100, Aurelien Foret <aurelien@archlinux.org> wrote:
Well, with the latest CVS code, this patch does not seem to be needed anymore: src/pacman/add.c is already doing the job to display
yes, the patch is two weeks old :)
btw you started to use two spaces instead of tabs. why? :) udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org
VMiklos wrote:
btw you started to use two spaces instead of tabs. why? :)
My bad: copy & paste errors ;) Thanks for the heads up. -- Aurelien
VMiklos wrote:
On Sat, Jan 07, 2006 at 12:30:57AM +0100, Aurelien Foret <aurelien@archlinux.org> wrote:
Well, with the latest CVS code, this patch does not seem to be needed anymore: src/pacman/add.c is already doing the job to display
yes, the patch is two weeks old :)
And there are some more patches older than this one, still waiting for review ;) Don't be despaired :) -- Aurelien
participants (2)
-
Aurelien Foret
-
VMiklos