xbps-install: download-only misleading information and explanation. close or see #341

Author: zlice

bin/xbps-install/xbps-install.1

@@ -99,10 +99,10 @@ If the first character is not '/' then it's a relative path of
 .It Fl d , Fl -debug
 Enables extra debugging shown to stderr.
 .It Fl D , Fl -download-only
-Only download packages to the cache, do not do any other installation steps.
-This may be useful for doing system upgrades while offline, or automatically
-downloading updates while leaving you with the option of still manually running
-the update.
+Only download packages and dependencies to the cache, do not do any other
+installation steps. This may be useful for doing system upgrades while offline, or
+automatically downloading updates while leaving you with the option of still
+manually running the update.
 .It Fl f , Fl -force
 Force installation (downgrade if package version in repos is less than installed version),
 or reinstallation (if package version in repos is the same) to the target

lib/transaction_prepare.c

@@ -88,9 +88,11 @@ compute_transaction_stats(struct xbps_handle *xhp)
 
 		if (ttype == XBPS_TRANS_REMOVE) {
 			rm_pkgcnt++;
-		} else if (ttype == XBPS_TRANS_CONFIGURE) {
+		} else if (!(xhp->flags & XBPS_FLAG_DOWNLOAD_ONLY) &&
+		          ttype == XBPS_TRANS_CONFIGURE) {
 			cf_pkgcnt++;
-		} else if (ttype == XBPS_TRANS_INSTALL || ttype == XBPS_TRANS_REINSTALL) {
+		} else if (!(xhp->flags & XBPS_FLAG_DOWNLOAD_ONLY) &&
+              (ttype == XBPS_TRANS_INSTALL || ttype == XBPS_TRANS_REINSTALL)) {
 			inst_pkgcnt++;
 		} else if (ttype == XBPS_TRANS_UPDATE) {
 			up_pkgcnt++;