Silent exit instead of error message on DB access problem about pkg HOT 3 OPEN

I think this behavior is intentional(?), as pkgdb_check_access() only returns error codes, and doesn't print standard error messages. This issue also exists in some other commands as well (see #2211).

A simple "fix" can be done by adding a warning message (using warn(3) or perror(3)) whenever faccessat(3) fails.

root@freebsd:/home/rilysh/pkg # ./src/pkg install abc
pkg: faccessat: Operation not permitted
root@freebsd:/home/rilysh/pkg # echo $?
1

I'm not sure, if it's required in most cases, as you can always check for returned errors. Although I agree, having a clear error message is much better than just relying on returned values.

from pkg.

Well, this can't be intentional, because it creates UX from hell for the user. And even if the user is not just user but sysadmin, then bare faccessat: Operation not permitted is stil not enough - yopu want to know which file caused the problem. So that it can be fixed without consulting pkg sources. Because user do not call routines to check for error codes, user calls e.g. pkg upgrade.

So. The only debatable thing here is where to insert reporting to user, and by which function - this is what pkg maintainers know better than bugreporter.

from pkg.

Adding it in pkgdb_check_access() function is just fine. However, before any PR, it will be better to get some attention from maintainers with this issue.

from pkg.