[pacman-dev] [PATCH] Clarify the "failed to add target" errors.

Nagy Gabor ngaba at bibl.u-szeged.hu
Mon Feb 4 11:00:52 EST 2008


Hi!

Well, this error message problem is not critical at all, but I don't like your
preferred fix, because:
- all "transaction steps" (init, release, addtarget, prepare, commit) use the
same format now: "error: failed to foo transaction (ERRORLAST)\n" and the patch
would break this simmetry.
- I'm not sure that alpm_strerrorlast() is always informative and set properly
- we can get the same ERRORLAST so foo can be helpful in debugging in these cases
- Personally I liked this front-end + back-end error message "concatenation"

So I prefer rephase this "failed to add target" message, Dan's suggestion seems
OK to me: http://www.archlinux.org/pipermail/pacman-dev/2008-February/011045.html

Bye


----------------------------------------------------
SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu
This mail sent through IMP: http://horde.org/imp/





More information about the pacman-dev mailing list