[pacman-dev] [PATCH] A variety of small changes

[Dan McGee]

On Fri, May 16, 2008 at 8:06 AM, Allan McRae <mcrae_allan at hotmail.com> wrote:
> 1. makepkg - Reduces the missing arch error to a warning when only
> generating intergity checks (-g or --geninteg flag)
Probably smart, this seems good.

> 2. libalpm - remove unused handle->uid from pmhandle_t.  The need to
> check permissions should be determined by the frontend (and is in pacman).
Hmm- I almost think the backend should do something (such as verifying
we have read/write perms on the relevant dbs if necessary), but you
are right, it is unused now so it should go. Any headers that can be
dropped in handle.c because of this change?

> 3. libalpm - fix comment on noextract in pmhandle_t.
So we definitely use this and it still works? Heh. We might need to
beef up pactests in this area.

> 4. pacman - only ask for removal confirmation when the recusre or
> cascade options add packages to the removal list
I thought this came up a few months ago, and I believe I at least
thought it wasn't the greatest of ideas. I wanted behavior to be
consistent with the option, as some people might use these flags as "I
know it will ask me for confirmation" step. I know it would confuse me
(and scripts) if it asked sometimes and other times did not, and it is
not immediately apparent to the end user why this is happening.

-Dan