GNU bug report logs - #62664
[PATCH] [WIP] gnu: Add emacs-copilot.

Previous Next

Package: guix-patches;

Reported by: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>

Date: Tue, 4 Apr 2023 14:32:02 UTC

Severity: normal

Tags: patch

To reply to this bug, email your comments to 62664 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 04 Apr 2023 14:32:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Rostislav Svoboda <rostislav.svoboda <at> gmail.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 04 Apr 2023 14:32:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Rostislav Svoboda <Rostislav.Svoboda <at> gmail.com>
Subject: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Tue,  4 Apr 2023 16:30:43 +0200
---
 gnu/packages/emacs-xyz.scm | 46 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 2b96c60057..d037fa9075 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -36689,6 +36689,52 @@ (define-public emacs-tintin-mode
       (description "This major mode focuses on highlighting as many aspects of the TinTin++ scripting language as possible, organizing commands into functional categories and highlighting specific modes that many commands use to accomplish different tasks.")
       (license license:asl2.0))))
 
+(define-public emacs-copilot
+  (let ((commit "e11847ab0c3b183a1e53fbc1ac587de82912b9cf")
+        (revision "0"))
+    (package
+      (name "emacs-copilot")
+      (version (git-version "0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/zerolfx/copilot.el.git")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "0gsxb268vqyim65ag8d7dlgdqyxqrjcjirlnbfbfq0pdr1y2158q"))))
+      (build-system emacs-build-system)
+      (arguments
+       (list
+        #:include #~(cons "^dist/" %default-include)
+        #:phases
+        #~(modify-phases %standard-phases
+            (add-after 'unpack 'substitute-copilot-node-executable
+              (lambda* (#:key inputs #:allow-other-keys)
+                (emacs-substitute-variables "copilot.el"
+                  ("copilot-node-executable"
+;;; Copilot requires Node 16+, however packaging v16 and/or v18 is not
+;;; trivial. See https://issues.guix.gnu.org/53414,
+;;; https://issues.guix.gnu.org/59188 .
+;;;
+;;; As a hack, download and compile recent Node version manually.
+                   (format #f "%s/node-v18.15.0/out/Release/node" (getenv "dev")) ; Remove this line when Node 16+ becomes available as a Guix package.
+                   ;; (search-input-file inputs "/bin/node")                      ; Activate this line when Node 16+ becomes available as a Guix package.
+                   )))))))
+      ;; (inputs (list node))                                                     ; Activate this line when Node 16+ becomes available as a Guix package.
+      (propagated-inputs
+       (list emacs-dash emacs-editorconfig emacs-s))
+      (home-page
+       "https://github.com/zerolfx/copilot.el.git")
+      (synopsis "An unofficial Copilot plugin for Emacs")
+      (description
+       "Copilot.el is an Emacs plugin for GitHub Copilot. This plugin is
+unofficial and based on binaries provided by copilot.vim. Note: You need
+access to GitHub Copilot to use this plugin.")
+      (license license:gpl3+))))
+
 ;;;
 ;;; Avoid adding new packages to the end of this file. To reduce the chances
 ;;; of a merge conflict, place them above by existing packages with similar
-- 
2.39.2





Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 04 Apr 2023 14:37:02 GMT) Full text and rfc822 format available.

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Tue, 4 Apr 2023 16:35:47 +0200
As hinted in the commit message, this patch is Work-In-Progress,
waiting for Node v16+ to become available as a Guix package. I.e. this
patch is not ready for integration in the upstream, yet
However I assume some folks may benefit from it, anyway.

Cheers Bost

Le mar. 4 avr. 2023 à 16:31, Rostislav Svoboda
<rostislav.svoboda <at> gmail.com> a écrit :
>
> ---
>  gnu/packages/emacs-xyz.scm | 46 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index 2b96c60057..d037fa9075 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -36689,6 +36689,52 @@ (define-public emacs-tintin-mode
>        (description "This major mode focuses on highlighting as many aspects of the TinTin++ scripting language as possible, organizing commands into functional categories and highlighting specific modes that many commands use to accomplish different tasks.")
>        (license license:asl2.0))))
>
> +(define-public emacs-copilot
> +  (let ((commit "e11847ab0c3b183a1e53fbc1ac587de82912b9cf")
> +        (revision "0"))
> +    (package
> +      (name "emacs-copilot")
> +      (version (git-version "0" revision commit))
> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/zerolfx/copilot.el.git")
> +               (commit commit)))
> +         (file-name (git-file-name name version))
> +         (sha256
> +          (base32
> +           "0gsxb268vqyim65ag8d7dlgdqyxqrjcjirlnbfbfq0pdr1y2158q"))))
> +      (build-system emacs-build-system)
> +      (arguments
> +       (list
> +        #:include #~(cons "^dist/" %default-include)
> +        #:phases
> +        #~(modify-phases %standard-phases
> +            (add-after 'unpack 'substitute-copilot-node-executable
> +              (lambda* (#:key inputs #:allow-other-keys)
> +                (emacs-substitute-variables "copilot.el"
> +                  ("copilot-node-executable"
> +;;; Copilot requires Node 16+, however packaging v16 and/or v18 is not
> +;;; trivial. See https://issues.guix.gnu.org/53414,
> +;;; https://issues.guix.gnu.org/59188 .
> +;;;
> +;;; As a hack, download and compile recent Node version manually.
> +                   (format #f "%s/node-v18.15.0/out/Release/node" (getenv "dev")) ; Remove this line when Node 16+ becomes available as a Guix package.
> +                   ;; (search-input-file inputs "/bin/node")                      ; Activate this line when Node 16+ becomes available as a Guix package.
> +                   )))))))
> +      ;; (inputs (list node))                                                     ; Activate this line when Node 16+ becomes available as a Guix package.
> +      (propagated-inputs
> +       (list emacs-dash emacs-editorconfig emacs-s))
> +      (home-page
> +       "https://github.com/zerolfx/copilot.el.git")
> +      (synopsis "An unofficial Copilot plugin for Emacs")
> +      (description
> +       "Copilot.el is an Emacs plugin for GitHub Copilot. This plugin is
> +unofficial and based on binaries provided by copilot.vim. Note: You need
> +access to GitHub Copilot to use this plugin.")
> +      (license license:gpl3+))))
> +
>  ;;;
>  ;;; Avoid adding new packages to the end of this file. To reduce the chances
>  ;;; of a merge conflict, place them above by existing packages with similar
> --
> 2.39.2
>




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 04 Apr 2023 14:55:02 GMT) Full text and rfc822 format available.

Message #11 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: "jgart" <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Tue, 04 Apr 2023 14:54:46 +0000
Hi Rostislav,

Thanks for this patch. It builds fine for me. Linter complains about a few nitpicks regarding line lengths and article of synopsis.

I added your patch as a pre-release to Guix "R" Us:

https://git.sr.ht/~whereiseveryone/guixrus/commit/4fe9054f758c022ef757773b9870cbd548bde26d

I'll let you know if we run into any other issues when trying the package from Guix "R" Us.

all best,

jgart

https://whereis.みんな/
https://toys.whereis.みんな/




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 04 Apr 2023 19:21:02 GMT) Full text and rfc822 format available.

Message #14 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>, 62664 <at> debbugs.gnu.org
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Tue, 04 Apr 2023 21:19:54 +0200
Am Dienstag, dem 04.04.2023 um 16:30 +0200 schrieb Rostislav Svoboda:
> +       "Copilot.el is an Emacs plugin for GitHub Copilot. This
> plugin is
> +unofficial and based on binaries provided by copilot.vim. Note: You
> need
> +access to GitHub Copilot to use this plugin.")
> +      (license license:gpl3+))))
Given that this package is based on binaries, and the fact that Github
Copilot has attracted a wide variety of concerns regarding licensing,
privacy and autocompleted vulnerabilities, I think we can do better
without it :)

Cheers




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Wed, 05 Apr 2023 13:09:02 GMT) Full text and rfc822 format available.

Message #17 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Giovanni Biscuolo <g <at> xelera.eu>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>, 62664 <at> debbugs.gnu.org
Subject: Re: [bug#62664] [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Wed, 05 Apr 2023 15:07:58 +0200
[Message part 1 (text/plain, inline)]
Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:

> Am Dienstag, dem 04.04.2023 um 16:30 +0200 schrieb Rostislav Svoboda:
>> +       "Copilot.el is an Emacs plugin for GitHub Copilot. This
>> plugin is
>> +unofficial and based on binaries provided by copilot.vim. Note: You
>> need
>> +access to GitHub Copilot to use this plugin.")
>> +      (license license:gpl3+))))
> Given that this package is based on binaries,

binaries are wasm files in "dist" folder:
https://github.com/zerolfx/copilot.el/tree/main/dist

--8<---------------cut here---------------start------------->8---
tree-sitter-go.wasm
tree-sitter-javascript.wasm
tree-sitter-python.wasm
tree-sitter-ruby.wasm
tree-sitter-typescript.wasm
tree-sitter.wasm
--8<---------------cut here---------------end--------------->8---

AFAIU taken by copilot.vim repo:
https://github.com/github/copilot.vim/tree/release/copilot/dits

> and the fact that Github
> Copilot has attracted a wide variety of concerns regarding licensing,
> privacy and autocompleted vulnerabilities, I think we can do better
> without it :)

For what it matters, I totally agree with Liliana

Thanks! Gio'

-- 
Giovanni Biscuolo

Xelera IT Infrastructures
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Thu, 06 Apr 2023 19:35:02 GMT) Full text and rfc822 format available.

Message #20 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Brian Cully <bjc <at> spork.org>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>, 62664 <at> debbugs.gnu.org
Subject: Re: bug#62664: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Thu, 06 Apr 2023 15:34:30 -0400
> Given that this package is based on binaries, and the fact that Github
> Copilot has attracted a wide variety of concerns regarding licensing,
> privacy and autocompleted vulnerabilities, I think we can do better
> without it :)

I came to this ticket to make this same comment.




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Fri, 07 Apr 2023 05:01:02 GMT) Full text and rfc822 format available.

Message #23 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: "jgart" <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Date: Fri, 07 Apr 2023 05:00:09 +0000
Hi,

The author is making an effort to prepare copilot.el for submission to MELPA:

https://github.com/zerolfx/copilot.el/issues/120

This package is obviously not ready for GNU Guix at this moment given the vendored WASM, etc. but let's give the project maintainer a chance to see how they manage to prepare this Emacs package for inclusion in MELPA.




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Thu, 17 Oct 2024 22:28:01 GMT) Full text and rfc822 format available.

Message #26 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: Giovanni Biscuolo <g <at> xelera.eu>, Brian Cully <bjc <at> spork.org>,
 jgart <jgart <at> dismail.de>, Rostislav Svoboda <rostislav.svoboda <at> gmail.com>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Thu, 17 Oct 2024 17:26:55 -0500
Hi Guixers,

Here's a revised patch for emacs-copilot. Please review and let me know
if I can merge it.

This package has been cleaned of all binaries and is now being prepared
for submission to MELPA:

https://github.com/melpa/melpa/pull/9217

I believe it now complies with all requirements for inclusion in GNU
Guix.

I've added Rostislav as a co-author.

WDYT
-- 
all the best,
jgart




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Sun, 27 Oct 2024 10:41:02 GMT) Full text and rfc822 format available.

Message #29 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: jgart via Guix-patches via <guix-patches <at> gnu.org>
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 62664 <at> debbugs.gnu.org,
 jgart <jgart <at> dismail.de>, Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [bug#62664] [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Sun, 27 Oct 2024 11:39:25 +0100
Hello,

jgart via Guix-patches via <guix-patches <at> gnu.org> writes:

> Here's a revised patch for emacs-copilot. Please review and let me know
> if I can merge it.

Did you forget to add the patch to your email?

Regards,
-- 
Nicolas Goaziou






Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Sun, 27 Oct 2024 10:41:02 GMT) Full text and rfc822 format available.

Information forwarded to rostislav.svoboda <at> gmail.com, jgart <at> dismail.de, liliana.prikler <at> gmail.com, g <at> xelera.eu, bjc <at> spork.org, mail <at> nicolasgoaziou.fr, andrew <at> trop.in, cox.katherine.e+guix <at> gmail.com, guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 28 Oct 2024 02:42:02 GMT) Full text and rfc822 format available.

Message #35 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: jgart <jgart <at> dismail.de>, Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: [PATCH v2] gnu: Add emacs-copilot.
Date: Sun, 27 Oct 2024 21:40:38 -0500
* gnu/packages/emacs-xyz.scm (emacs-copilot): New variable.
Co-authored-by: Rostislav Svoboda <rostislav.svoboda <at> gmail.com>

Change-Id: If2e32f5cc4df95ce99642b41a1df818a35b4343c
---
 gnu/packages/emacs-xyz.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 488b4cb5d7..76c9ed9947 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -4422,6 +4422,43 @@ (define-public emacs-carp
 listing type errors via Flycheck, as well as REPL support for Carp.")
       (license license:asl2.0))))
 
+(define-public emacs-copilot
+  (let ((commit "b5878d6a8c741138b5efbf4fe1c594f3fd69dbdd")
+        (revision "0"))
+    (package
+      (name "emacs-copilot")
+      (version (git-version "1.27.0" revision commit))
+      (source
+       (origin
+         (uri (git-reference
+               (url "https://github.com/copilot-emacs/copilot.el")
+               (commit commit)))
+         (method git-fetch)
+         (sha256
+          (base32 "1dgi3g2n57dv1a8hjjzgccrbl1bdf4xqwn9bmr453fz4qfab0v6k"))
+         (file-name (git-file-name name version))))
+      (build-system emacs-build-system)
+      (arguments
+       (list
+        #:phases
+        #~(modify-phases %standard-phases
+            (add-after 'unpack 'substitute-copilot-node-executable
+              (lambda* (#:key inputs #:allow-other-keys)
+                (emacs-substitute-variables "copilot.el"
+                  ("copilot-node-executable"
+                   (search-input-file inputs "/bin/node"))))))))
+      (inputs (list node-lts))
+      (propagated-inputs
+       (list emacs-dash
+             emacs-editorconfig
+             emacs-f
+             emacs-s))
+      (home-page "https://github.com/copilot-emacs/copilot.el")
+      (synopsis "Copilot plugin for Emacs")
+      (description
+       "This package provides an Emacs plugin for GitHub Copilot.")
+      (license license:expat))))
+
 (define-public emacs-coterm
   (package
     (name "emacs-coterm")

base-commit: 0d15c6e46c5c3a14ae776bb98c293527ba8ad319
-- 
2.46.0





Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 28 Oct 2024 02:43:01 GMT) Full text and rfc822 format available.

Message #38 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, jgart <jgart <at> dismail.de>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Sun, 27 Oct 2024 21:41:54 -0500
> Did you forget to add the patch to your email?

Yes, I did ;()

Attached in previous email to this thread.
-- 
all the best,
jgart




Information forwarded to rostislav.svoboda <at> gmail.com, jgart <at> dismail.de, liliana.prikler <at> gmail.com, g <at> xelera.eu, bjc <at> spork.org, mail <at> nicolasgoaziou.fr, andrew <at> trop.in, cox.katherine.e+guix <at> gmail.com, guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 23 Dec 2024 17:46:02 GMT) Full text and rfc822 format available.

Message #41 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: jgart <jgart <at> dismail.de>
Subject: [PATCH v3] gnu: Add emacs-copilot.
Date: Mon, 23 Dec 2024 11:44:39 -0600
* gnu/packages/emacs-xyz.scm (emacs-copilot): New variable.

Change-Id: I3d693e5a59a28f488663421e6b2948cf8b51e031
---
 gnu/packages/emacs-xyz.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 65563bec24..f5826a87aa 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -29992,6 +29992,43 @@ (define-public emacs-dired-preview
 customisation options to control its behaviour.")
     (license license:gpl3+)))
 
+(define-public emacs-copilot
+  (let ((commit "d2126f288cebd9cd7e768c53a4cbfa6eedcb0666")
+        (revision "0"))
+    (package
+      (name "emacs-copilot")
+      (version (git-version "1.27.0" revision commit))
+      (source
+       (origin
+         (uri (git-reference
+               (url "https://github.com/copilot-emacs/copilot.el")
+               (commit commit)))
+         (method git-fetch)
+         (sha256
+          (base32 "14qnhp9pzilkrkaz2w52ylanjg44idi41dg6g7lqqc4fk0ib12kj"))
+         (file-name (git-file-name name version))))
+      (build-system emacs-build-system)
+      (arguments
+       (list
+        #:phases
+        #~(modify-phases %standard-phases
+            (add-after 'unpack 'substitute-copilot-node-executable
+              (lambda* (#:key inputs #:allow-other-keys)
+                (emacs-substitute-variables "copilot.el"
+                  ("copilot-node-executable"
+                   (search-input-file inputs "/bin/node"))))))))
+      (inputs (list node-lts))
+      (propagated-inputs
+       (list emacs-dash
+             emacs-editorconfig
+             emacs-f
+             emacs-s))
+      (home-page "https://github.com/copilot-emacs/copilot.el")
+      (synopsis "Copilot plugin for Emacs")
+      (description
+       "This package provides an Emacs plugin for GitHub Copilot.")
+      (license license:expat))))
+
 (define-public emacs-dired-launch
   (package
     (name "emacs-dired-launch")

base-commit: 64774d76869cd14712064d34f82f06c78ee796a6
-- 
2.46.0





Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 23 Dec 2024 17:48:02 GMT) Full text and rfc822 format available.

Message #44 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, jgart <jgart <at> dismail.de>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Mon, 23 Dec 2024 11:47:38 -0600
Hi Nicolas,

Here is a v3. Let me know if you still can't apply it.

I used mumi cli to send this.

emacs-copilot is now in melpa:

https://melpa.org/#/copilot

-- 
all the best,
jgart




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 23 Dec 2024 18:35:02 GMT) Full text and rfc822 format available.

Message #47 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, jgart <jgart <at> dismail.de>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Mon, 23 Dec 2024 12:34:13 -0600
Actually looking at this package more carefully, the reason it needs
node is to npm install copilot-node-server which has an unclear license:

https://github.com/jfcherng/copilot-node-server

Here's what nixpkgs did:

https://github.com/NixOS/nixpkgs/blob/master/pkgs/by-name/co/copilot-node-server/package.nix#L38

Will this be a problem for emacs-copilot since we should include
copilot-node-server as a required dependency instead of installing it
via the npm wrapper?

If yes, I can close this issue and put this in a Guix channel instead.

Would be unfortunate though, since this is becoming a popular package in
the Emacs ecosystem.
-- 
all the best,
jgart




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Mon, 23 Dec 2024 19:03:02 GMT) Full text and rfc822 format available.

Message #50 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: jgart <jgart <at> dismail.de>, 62664 <at> debbugs.gnu.org
Cc: Giovanni Biscuolo <g <at> xelera.eu>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>, Brian Cully <bjc <at> spork.org>
Subject: Re: [PATCH] [WIP] gnu: Add emacs-copilot.
Date: Mon, 23 Dec 2024 20:02:47 +0100
Am Montag, dem 23.12.2024 um 12:34 -0600 schrieb jgart:
> 
> Actually looking at this package more carefully, the reason it needs
> node is to npm install copilot-node-server which has an unclear
> license:
> 
> https://github.com/jfcherng/copilot-node-server
> 
> Here's what nixpkgs did:
> 
> https://github.com/NixOS/nixpkgs/blob/master/pkgs/by-name/co/copilot-node-server/package.nix#L38
> 
> Will this be a problem for emacs-copilot since we should include
> copilot-node-server as a required dependency instead of installing it
> via the npm wrapper?
> 
> If yes, I can close this issue and put this in a Guix channel
> instead.
Unbundling would be nicer, but if you don't need to download the
internet as part of the build or on first start to fetch all the
dependencies (i.e. if you can properly package it with available
sources + node), then you may forgo that for now.  That's a big if
however, one that rarely holds in JS ecosystems.

> Would be unfortunate though, since this is becoming a popular package
> in the Emacs ecosystem.
Selling your data to Microsoft should be less popular, then.

Cheers




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 24 Dec 2024 08:21:02 GMT) Full text and rfc822 format available.

Message #53 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: jgart <jgart <at> dismail.de>, 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>,
 Andrew Tropin <andrew <at> trop.in>, Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [bug#62664] [PATCH v3] gnu: Add emacs-copilot.
Date: Tue, 24 Dec 2024 09:20:23 +0100
Am Montag, dem 23.12.2024 um 11:44 -0600 schrieb jgart:
> * gnu/packages/emacs-xyz.scm (emacs-copilot): New variable.
> 
> Change-Id: I3d693e5a59a28f488663421e6b2948cf8b51e031
> ---
Just had a look at it and `copilot-install-server` looks quite sus to
me.

Cheers




Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 24 Dec 2024 15:04:03 GMT) Full text and rfc822 format available.

Message #56 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: "jgart" <jgart <at> dismail.de>
To: "Liliana Marie Prikler" <liliana.prikler <at> gmail.com>, 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>, Katherine 
 Cox-Buday <cox.katherine.e+guix <at> gmail.com>, Andrew Tropin <andrew <at> trop.in>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [bug#62664] [PATCH v3] gnu: Add emacs-copilot.
Date: Tue, 24 Dec 2024 15:03:48 +0000
[Message part 1 (text/plain, inline)]
> 
> Just had a look at it and `copilot-install-server` looks quite sus to
> 
Which part looks sus besides the fact that the author says that they don't own the codes and that they are obfuscated and won't fix?

https://github.com/jfcherng/copilot-node-server/issues/22#issuecomment-2560567220

🙃
[Message part 2 (text/html, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 24 Dec 2024 15:14:01 GMT) Full text and rfc822 format available.

Message #59 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: "jgart" <jgart <at> dismail.de>
To: "Liliana Marie Prikler" <liliana.prikler <at> gmail.com>, 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>, Katherine  
 Cox-Buday <cox.katherine.e+guix <at> gmail.com>, Andrew Tropin <andrew <at> trop.in>,
 Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [bug#62664] [PATCH v3] gnu: Add emacs-copilot.
Date: Tue, 24 Dec 2024 15:13:02 +0000
[Message part 1 (text/plain, inline)]
I opened an issue here with the copilot.el community:

https://github.com/copilot-emacs/copilot.el/issues/356
[Message part 2 (text/html, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#62664; Package guix-patches. (Tue, 24 Dec 2024 18:12:01 GMT) Full text and rfc822 format available.

Message #62 received at 62664 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: jgart <jgart <at> dismail.de>, 62664 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> spork.org>, Giovanni Biscuolo <g <at> xelera.eu>,
 Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>,
 Andrew Tropin <andrew <at> trop.in>, Nicolas Goaziou <mail <at> nicolasgoaziou.fr>,
 Rostislav Svoboda <rostislav.svoboda <at> gmail.com>
Subject: Re: [bug#62664] [PATCH v3] gnu: Add emacs-copilot.
Date: Tue, 24 Dec 2024 19:11:18 +0100
Am Dienstag, dem 24.12.2024 um 15:03 +0000 schrieb jgart:
> 
> > Just had a look at it and `copilot-install-server` looks quite sus
> > to me
> 
> 
> Which part looks sus besides the fact that the author says that they
> don't own the codes and that they are obfuscated and won't fix?
> 
> https://github.com/jfcherng/copilot-node-server/issues/22#issuecomment-2560567220
> 
> 🙃
Yeah, thanks for pointing that out.  I feel like atm, since IIUC this
server is required to make the package work, you can not offer this as
"free software" in good conscience (said conscience being informed by
the literal letter of the FSDG saying "no").

Cheers





This bug report was last modified 177 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.