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.
