[pacman-dev] Patch decisions required

[Dave Reisner]

On Thu, Feb 07, 2013 at 02:31:04PM +1000, Allan McRae wrote:
> Dan/Dave,
> 
> I'd like comments on these patches - which at least to me are not
> obvious whether to include or not.  A simple yes/no/fix would be good.
> 
> 
> From patchwork:
> 
> * pacman: print version information in debug output
> https://patchwork.archlinux.org/patch/114/
> 

Ack.

> 
> * removed duplicate macros SYMEXPORT and SYMHIDDEN from alpm_list.c,
> using them from util.h
> https://patchwork.archlinux.org/patch/304/
> 

I'm siding with whatever Dan decides on this one.

> 
> * Remove "ts=2 sw=2" from Vim modelines for pacman-contrib
> https://patchwork.archlinux.org/patch/689/
> (I think this was approved - Dan?)

No. We "enforce" ts/sw in pacman, why not contrib as well?

> * Remove tab width config from Vim modelines in doc/
> https://patchwork.archlinux.org/patch/690/
> (I think this was approved - Dan?)
> 

See above.

> 
> * Log if dbs are up to date, synced, or failed to sync
> https://patchwork.archlinux.org/patch/760/
> 

No. Too noisy, not useful info.

> * Move logging of sysupgrade start, add log when done
> https://patchwork.archlinux.org/patch/761/
> 

See above.

> 
> * Exit with failure status if download or installation is not confirmed
> https://patchwork.archlinux.org/patch/876/
> 

Ack.

> 
> From the days before patchwork...:
> 
> * Skip reading sync db deltas files if UseDelta is unset
> https://projects.archlinux.org/users/allan/pacman.git/commit/?h=working&id=7c4c418a
> 

Makes sense. Ack.

> * Quieten the build process some more
> https://projects.archlinux.org/users/allan/pacman.git/commit/?h=working&id=11046040
> 
> 

Ack.