From debbugs-submit-bounces@debbugs.gnu.org Mon Aug 15 04:32:25 2022 Received: (at 57118) by debbugs.gnu.org; 15 Aug 2022 08:32:25 +0000 Received: from localhost ([127.0.0.1]:40809 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oNVWL-0005pE-DE for submit@debbugs.gnu.org; Mon, 15 Aug 2022 04:32:25 -0400 Received: from eggs.gnu.org ([209.51.188.92]:38664) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oNVWK-0005p2-5L for 57118@debbugs.gnu.org; Mon, 15 Aug 2022 04:32:24 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]:35072) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oNVWE-0003ST-Uj; Mon, 15 Aug 2022 04:32:18 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-Version:In-Reply-To:Date:References:Subject:To: From; bh=23yPGvGOocHjVeQlbLavc68mhMoqEdbLpS8S4KPr388=; b=doUFr9/Xqg7k1e5OME9B uL5iZwk2ZtpEJR63mEvvmqPZZNwk8tChdwqGmHyPgbI3jnJAyKppED+VlVGiTBOP668yy1bjAAwPg ftS/RX1U0v2cZ/TBrFANCk/qs69jUiLcln5iAncer4TdlxLWpRb/c3hi8K5qnbyip+ZCIdNeFci/E lAOXGkybtUfBDXBK4x/8RfanE+OXcWCvvXPoyahmTtnGW+uXYu79vabdfF/i8atg7FxB0oT0a3MWP jVeSxO/3ouaODOEg3QAxW/y28MUE00Y7pucCceJbth/nQZS48S+CnxKM6zz/UHMtxogUfhbPz4xAR PY6baFm9rzomnQ==; Received: from 2a02-8429-81d2-3d01-94c9-8097-ea5c-2775.rev.sfr.net ([2a02:8429:81d2:3d01:94c9:8097:ea5c:2775]:45846 helo=meije) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oNVWC-0004T4-OA; Mon, 15 Aug 2022 04:32:18 -0400 From: Mathieu Othacehe To: raingloom Subject: Re: bug#57118: [PATCH] gnu: Add linode-cli. References: <20220810141843.21051-1-raingloom@riseup.net> Date: Mon, 15 Aug 2022 10:32:14 +0200 In-Reply-To: <20220810141843.21051-1-raingloom@riseup.net> (raingloom@riseup.net's message of "Wed, 10 Aug 2022 16:18:43 +0200") Message-ID: <87y1vpgb5d.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -2.3 (--) X-Debbugs-Envelope-To: 57118 Cc: 57118@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -3.3 (---) Hey, > * gnu/packages/python-xyz.scm (linode-cli): New variable. You also need to mention the linode-openapi-spec variable. > + (synopsis "Tool for managing Linode resources") > + (home-page "https://www.linode.com/docs/products/tools/cli/") These are usually at the end of the package definition. > + (assoc-ref inputs > + "linode-openapi-spec") It would be preferable to use the gexp mechanism. > + (assoc-ref outputs "out") Here too. > + "A wrapper around the Linode API, which gives you the ability to manage > +your Linode account from the command line. Almost any task that can be done > +through the Cloud Manager can also be performed through the CLI.") This package provides a wrapper around Linode API which gives the ability to manage Linode accounts from the command line. Almost ... Sentences must be separated by two spaces. Could you please send a v2? Thanks, Mathieu