From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34402) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ejLtf-0004v9-Tt for guix-patches@gnu.org; Wed, 07 Feb 2018 04:20:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ejLta-0006Zj-T0 for guix-patches@gnu.org; Wed, 07 Feb 2018 04:20:07 -0500 Received: from debbugs.gnu.org ([208.118.235.43]:52982) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ejLta-0006Zd-OV for guix-patches@gnu.org; Wed, 07 Feb 2018 04:20:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ejLta-0007pc-E0 for guix-patches@gnu.org; Wed, 07 Feb 2018 04:20:02 -0500 Subject: [bug#30328] [PATCH] gnu: Add emacs-browse-at-remote-gnu. Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <20180202214457.14147-1-go.wigust@gmail.com> <87k1vq5ec3.fsf@gnu.org> <871shynjzi.fsf@gmail.com> Date: Wed, 07 Feb 2018 10:18:58 +0100 In-Reply-To: <871shynjzi.fsf@gmail.com> (Oleg Pykhalov's message of "Tue, 06 Feb 2018 19:47:13 +0300") Message-ID: <874lmt17jx.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Oleg Pykhalov Cc: 30328@debbugs.gnu.org Hi Oleg, Oleg Pykhalov skribis: > ludo@gnu.org (Ludovic Court=C3=A8s) writes: >=20=20 > > What about simply adding the patch to =E2=80=98emacs-browse-at-remote= =E2=80=99 if > > it=E2=80=99s committed upstream way? > > I'm sorry, what do you mean by =E2=80=9Ccommitted upstream way=E2=80=9D? I mean that the patch for Savannah support is already in the repository of =E2=80=98browse-at-remote=E2=80=99, isn=E2=80=99t it? Which is to say t= hat the next version of =E2=80=98browse-at-remote=E2=80=99 will have that functionality. > I kinda agree to move the patch to =E2=80=98emacs-browse-at-remote=E2=80= =99, but if > somebody want to upgrade it, maybe he will need to recreate a patch. (=E2=80=9CHe=E2=80=9D or =E2=80=9Cshe=E2=80=9D.) I don=E2=80=99t know, won=E2=80=99t the next =E2=80=98browse-at-remote=E2= =80=99 include it? > > Also please make the .patch file name a bit shorter; see =E2=80=98gui= x lint=E2=80=99. > > Ouch, I missed this because of =E2=80=9CGUIX_PACKAGE_PATH=E2=80=9D. Usua= lly I unset > this when prepare a package for Guix collection, but forget this time. > > $ ./pre-inst-env env guix lint emacs-browse-at-remote-gnu > > $ ./pre-inst-env env GUIX_PACKAGE_PATH=3D guix lint emacs-browse-at-remot= e-gnu > /home/natsu/src/guix-wip-emacs-browse-at-remote-gnu/gnu/packages/emacs.sc= m:6474:2: > emacs-browse-at-remote-gnu@0.9.0-1.31dcf77: > emacs-browse-at-remote-gnu-add-support-for-gnu-repositories.patch: file > name is too long > fetching CVE database for 2018...nu@0.9.0-1.31dcf77 [cve]... Oh I see, interesting. The output of =E2=80=98guix lint=E2=80=99 seems to be messy, too. Thanks, Ludo=E2=80=99.