[PR PATCH] xbps-src: let -I imply -N

[PR PATCH] xbps-src: let -I imply -N

[paper42]

[-- Attachment #1: Type: text/plain, Size: 386 bytes --]

There is a new pull request by paper42 against master on the void-packages repository

https://github.com/paper42/void-packages xbps-src-I-implies-C
https://github.com/void-linux/void-packages/pull/30974

xbps-src: let -I imply -N
don't synchronize repositories when not installing any packages

A patch file from https://github.com/void-linux/void-packages/pull/30974.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xbps-src-I-implies-C-30974.patch --]
[-- Type: text/x-diff, Size: 982 bytes --]

From e568e5a88bc569ee608049bd0d8820153c2a60c5 Mon Sep 17 00:00:00 2001
From: Paper <paper@tilde.institute>
Date: Mon, 17 May 2021 21:15:11 +0200
Subject: [PATCH] xbps-src: let -I imply -N

don't synchronize repositories when not installing any packages
---
 xbps-src | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xbps-src b/xbps-src
index c3cd7e5db10b..056e01ee1bdf 100755
--- a/xbps-src
+++ b/xbps-src
@@ -375,7 +375,7 @@ while getopts "$XBPS_OPTSTRING" opt; do
         H) XBPS_ARG_HOSTDIR="$OPTARG"; XBPS_OPTIONS+=" -H $OPTARG";;
         h) usage && exit 0;;
         i) XBPS_ARG_INFORMATIVE_RUN=1; XBPS_OPTIONS+=" -i";;
-        I) XBPS_ARG_SKIP_DEPS=1; XBPS_OPTIONS+=" -I";;
+        I) XBPS_ARG_SKIP_DEPS=1 XBPS_SKIP_REMOTEREPOS=1; XBPS_OPTIONS+=" -I -N";;
         j) XBPS_ARG_MAKEJOBS="$OPTARG"; XBPS_OPTIONS+=" -j $OPTARG";;
         L) export NOCOLORS=1; XBPS_OPTIONS+=" -L";;
         m) XBPS_ARG_MASTERDIR="$OPTARG"; XBPS_OPTIONS+=" -m $OPTARG";;

Re: [PR REVIEW] xbps-src: let -I imply -N

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 259 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/30974#discussion_r642635548

Comment:
```suggestion
        I) XBPS_ARG_SKIP_DEPS=1; XBPS_SKIP_REMOTEREPOS=1; XBPS_OPTIONS+=" -I -N";;
```

Right?

Re: [PR PATCH] [Updated] xbps-src: let -I imply -N

[paper42]

[-- Attachment #1: Type: text/plain, Size: 391 bytes --]

There is an updated pull request by paper42 against master on the void-packages repository

https://github.com/paper42/void-packages xbps-src-I-implies-C
https://github.com/void-linux/void-packages/pull/30974

xbps-src: let -I imply -N
don't synchronize repositories when not installing any packages

A patch file from https://github.com/void-linux/void-packages/pull/30974.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xbps-src-I-implies-C-30974.patch --]
[-- Type: text/x-diff, Size: 988 bytes --]

From 4a0f0bdece95e09326cc16096d4b2e3fe2478b15 Mon Sep 17 00:00:00 2001
From: Michal Vasilek <michal@vasilek.cz>
Date: Mon, 17 May 2021 21:15:11 +0200
Subject: [PATCH] xbps-src: let -I imply -N

don't synchronize repositories when not installing any packages
---
 xbps-src | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xbps-src b/xbps-src
index 806c6480f21c..086a0fe212fc 100755
--- a/xbps-src
+++ b/xbps-src
@@ -375,7 +375,7 @@ while getopts "$XBPS_OPTSTRING" opt; do
         H) XBPS_ARG_HOSTDIR="$OPTARG"; XBPS_OPTIONS+=" -H $OPTARG";;
         h) usage && exit 0;;
         i) XBPS_ARG_INFORMATIVE_RUN=1; XBPS_OPTIONS+=" -i";;
-        I) XBPS_ARG_SKIP_DEPS=1; XBPS_OPTIONS+=" -I";;
+        I) XBPS_ARG_SKIP_DEPS=1; XBPS_SKIP_REMOTEREPOS=1; XBPS_OPTIONS+=" -I -N";;
         j) XBPS_ARG_MAKEJOBS="$OPTARG"; XBPS_OPTIONS+=" -j $OPTARG";;
         L) export NOCOLORS=1; XBPS_OPTIONS+=" -L";;
         m) XBPS_ARG_MASTERDIR="$OPTARG"; XBPS_OPTIONS+=" -m $OPTARG";;

Re: [PR REVIEW] xbps-src: let -I imply -N

[paper42]

[-- Attachment #1: Type: text/plain, Size: 193 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/30974#discussion_r663275273

Comment:
sorry, I missed your review, fixed now, thanks

Re: [PR PATCH] [Merged]: xbps-src: let -I imply -N

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 222 bytes --]

There's a merged pull request on the void-packages repository

xbps-src: let -I imply -N
https://github.com/void-linux/void-packages/pull/30974

Description:
don't synchronize repositories when not installing any packages