~rjarry/dlrepo

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
2 2

[PATCH dlrepo v2] cli: new no-print-url option

Details
Message ID
<20241211133506.4156689-1-julien.floret@6wind.com>
DKIM signature
pass
Download raw message
Patch: +20 -6
The --no-print-url option allows disabling the printing of the target
URL to stderr.
That option is useful in scripts, where we want actual errors to
pop in stderr, but not "URL:" lines.

Suggested-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Julien Floret <julien.floret@6wind.com>
Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
---
 dlrepo-cli | 26 ++++++++++++++++++++------
 1 file changed, 20 insertions(+), 6 deletions(-)

diff --git a/dlrepo-cli b/dlrepo-cli
index 1396ba80993d..8222ca1b4154 100755
--- a/dlrepo-cli
+++ b/dlrepo-cli
@@ -80,6 +80,14 @@ def main():
        More verbose output.
        """,
    )
    parser.add_argument(
        "--no-print-url",
        dest="print_url",
        action="store_false",
        help="""
        Do not print the target URL.
        """,
    )
    sub = parser.add_subparsers(title="sub-command help", metavar="SUB_COMMAND")
    sub.required = True

@@ -221,7 +229,8 @@ def get_info(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        for k, v in data["job"].items():
            if isinstance(v, list):
                v = " ".join(v)
@@ -307,7 +316,8 @@ def get_description(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        description = data.get("tag", {}).get("description")
        if description is not None:
            print(description)
@@ -340,7 +350,8 @@ def branches(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.verbose:
            out = []
            branch_len = 0
@@ -396,7 +407,8 @@ def tags(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        for tag in data["branch"]["tags"]:
            print(tag["name"])

@@ -426,7 +438,8 @@ def jobs(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        for job in data["tag"]["jobs"]:
            print(job["name"])

@@ -446,7 +459,8 @@ def status(args):
    if args.raw_json:
        print(json.dumps(data, indent=2))
    else:
        print("URL: %s" % client.make_url(url), file=sys.stderr)
        if args.print_url:
            print("URL: %s" % client.make_url(url), file=sys.stderr)
        print("released=%s" % data["tag"]["released"])
        print("stable=%s" % data["tag"]["stable"])
        print("locked=%s" % data["tag"]["locked"])
-- 
2.39.2

[dlrepo/patches/.build.yml] build success

builds.sr.ht <builds@sr.ht>
Details
Message ID
<D68WTEF8IS3Z.2DKJPU53W45CD@fra02>
In-Reply-To
<20241211133506.4156689-1-julien.floret@6wind.com> (view parent)
DKIM signature
missing
Download raw message
dlrepo/patches/.build.yml: SUCCESS in 43s

[cli: new no-print-url option][0] v2 from [Julien Floret][1]

[0]: https://lists.sr.ht/~rjarry/dlrepo/patches/56375
[1]: julien.floret@6wind.com

✓ #1387533 SUCCESS dlrepo/patches/.build.yml https://builds.sr.ht/~rjarry/job/1387533

Applied: [PATCH dlrepo v2] cli: new no-print-url option

Details
Message ID
<173409262741.2055252.16045473205510856166@ringo>
In-Reply-To
<20241211133506.4156689-1-julien.floret@6wind.com> (view parent)
DKIM signature
pass
Download raw message
Julien Floret <julien.floret@6wind.com> wrote:
> The --no-print-url option allows disabling the printing of the target
> URL to stderr.
> That option is useful in scripts, where we want actual errors to
> pop in stderr, but not "URL:" lines.
>
> Suggested-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> Signed-off-by: Julien Floret <julien.floret@6wind.com>
> Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> ---

Acked-by: Robin Jarry <robin@jarry.cc>

Applied, thanks.

To git@git.sr.ht:~rjarry/dlrepo
   08320496aec8..35d6942b29a0  main -> main
Reply to thread Export thread (mbox)