[pacman-dev] [PATCH] Added a -n option to repo-add to only add new packages

[Allan McRae]

On 12/12/12 03:37, dangets at gmail.com wrote:
> From: Danny George <dangets at gmail.com>
> 
> Packages are already in the pkg db were given a warning, and then
> readded anyway.  With -n specified, the warning is printed, but skips
> readding it.
> 
> Signed-off-by: Danny George <dangets at gmail.com>
> ---

This will be fine with the man page updated and the minor change below.

>  scripts/repo-add.sh.in | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in
> index 5a3bf23..162beec 100644
> --- a/scripts/repo-add.sh.in
> +++ b/scripts/repo-add.sh.in
> @@ -30,6 +30,7 @@ declare -r confdir='@sysconfdir@'
>  
>  QUIET=0
>  DELTA=0
> +ONLYADDNEW=0
>  WITHFILES=0
>  SIGN=0
>  VERIFY=0
> @@ -55,6 +56,7 @@ Multiple packages to add can be specified on the command line.\n")"
>  		printf -- "\n"
>  		printf -- "$(gettext "Options:\n")"
>  		printf -- "$(gettext "  -d, --delta       generate and add delta for package update\n")"
> +		printf -- "$(gettext "  -n, --new         only add packages that aren't already in the database\n")"

are not

>  		printf -- "$(gettext "  -f, --files       update database's file list\n")"
>  	elif [[ $cmd == "repo-remove" ]] ; then
>  		printf -- "$(gettext "Usage: repo-remove [options] <path-to-db> <packagename|delta> ...\n")"
> @@ -290,6 +292,9 @@ db_write_entry() {
>  
>  	if [[ -d $tmpdir/tree/$pkgname-$pkgver ]]; then
>  		warning "$(gettext "An entry for '%s' already existed")" "$pkgname-$pkgver"
> +		if (( ONLYADDNEW )); then
> +			return 0;
> +		fi
>  	else
>  		if (( DELTA )); then
>  			pkgentry=$(find_pkgentry "$pkgname")
> @@ -603,6 +608,7 @@ while (( $# )); do
>  	case $1 in
>  		-q|--quiet) QUIET=1;;
>  		-d|--delta) DELTA=1;;
> +		-n|--new) ONLYADDNEW=1;;
>  		-f|--files) WITHFILES=1;;
>  		-s|--sign)
>  			check_gpg
>