all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0
@ 2023-08-14 10:52 Piotr Kwiecinski
  2023-08-15  7:27 ` Hilton Chain via Guix-patches via
  2023-08-15  9:19 ` [bug#65285] [PATCH] gnu: emacs-orderless: Update " Piotr Kwiecinski
  0 siblings, 2 replies; 5+ messages in thread
From: Piotr Kwiecinski @ 2023-08-14 10:52 UTC (permalink / raw)
  To: 65285; +Cc: Piotr Kwiecinski

previously used commit was pointing to 0.8 instead 1.0
---
 gnu/packages/emacs-xyz.scm | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 0f0751e39a..e62c71752e 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -11290,8 +11290,7 @@ (define-public emacs-selectrum
     (license license:expat)))
 
 (define-public emacs-orderless
-  (let ((commit "004cee6b8e01f8eb0cb1c683d0a637b14890600f"))
-    (package
+  (package
       (name "emacs-orderless")
       (version "1.0")
       (source
@@ -11299,9 +11298,9 @@ (define-public emacs-orderless
          (method git-fetch)
          (uri (git-reference
                (url "https://github.com/oantolin/orderless")
-               (commit commit)))
+               (commit version)))
          (sha256
-          (base32 "115bwqi2yc44bgvcl7lha8p2s6jgh5hksn4wa9s0kpfxhi14jwmy"))
+          (base32 "1jmwyici90v82gqjq44n0scrwhmg05pi9yplkn4bpss80c83dxp4"))
          (file-name (git-file-name name version))))
       (build-system emacs-build-system)
       (arguments
@@ -11322,7 +11321,7 @@ (define-public emacs-orderless
 that match all of the components in any order.  Each component can match in
 any one of several ways: literally, as a regexp, as an initialism, in the flex
 style, or as multiple word prefixes.")
-      (license license:gpl3+))))
+      (license license:gpl3+)))
 
 (define-public emacs-consult
   (package

base-commit: b15381460ed74e72792ff182dd2ca7a06ba59b0c
-- 
2.39.2





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0
  2023-08-14 10:52 [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0 Piotr Kwiecinski
@ 2023-08-15  7:27 ` Hilton Chain via Guix-patches via
  2023-08-15  9:32   ` Piotr Kwieciński
  2023-08-15  9:19 ` [bug#65285] [PATCH] gnu: emacs-orderless: Update " Piotr Kwiecinski
  1 sibling, 1 reply; 5+ messages in thread
From: Hilton Chain via Guix-patches via @ 2023-08-15  7:27 UTC (permalink / raw)
  To: Piotr Kwiecinski; +Cc: 65285

Hi Piotr,

Thank you for the patch!  There're some minor changes to make.

On Mon, 14 Aug 2023 18:52:38 +0800,
Piotr Kwiecinski wrote:
>
> previously used commit was pointing to 0.8 instead 1.0
> ---


Please write commit logs in ChangeLog format [1].  You can also check
the commit history for examples.

For this commit, the following can be used:
--8<---------------cut here---------------start------------->8---
gnu: emacs-orderless: Update to 1.0.

* gnu/packages/emacs-xyz.scm (emacs-orderless): Update to 1.0, previously used
commit was pointing to 0.8.
--8<---------------cut here---------------end--------------->8---


>  gnu/packages/emacs-xyz.scm | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index 0f0751e39a..e62c71752e 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -11290,8 +11290,7 @@ (define-public emacs-selectrum
>      (license license:expat)))
>
>  (define-public emacs-orderless
> -  (let ((commit "004cee6b8e01f8eb0cb1c683d0a637b14890600f"))
> -    (package
> +  (package
>        (name "emacs-orderless")
>        (version "1.0")
>        (source
> @@ -11299,9 +11298,9 @@ (define-public emacs-orderless
>           (method git-fetch)
>           (uri (git-reference
>                 (url "https://github.com/oantolin/orderless")
> -               (commit commit)))
> +               (commit version)))
>           (sha256
> -          (base32 "115bwqi2yc44bgvcl7lha8p2s6jgh5hksn4wa9s0kpfxhi14jwmy"))
> +          (base32 "1jmwyici90v82gqjq44n0scrwhmg05pi9yplkn4bpss80c83dxp4"))
>           (file-name (git-file-name name version))))
>        (build-system emacs-build-system)
>        (arguments
> @@ -11322,7 +11321,7 @@ (define-public emacs-orderless
>  that match all of the components in any order.  Each component can match in
>  any one of several ways: literally, as a regexp, as an initialism, in the flex
>  style, or as multiple word prefixes.")
> -      (license license:gpl3+))))
> +      (license license:gpl3+)))
>
>  (define-public emacs-consult
>    (package
>
> base-commit: b15381460ed74e72792ff182dd2ca7a06ba59b0c
> --
> 2.39.2


I think it's fine to also change the indentation here, remove two
leading spaces in every line after the "(package" one, your editor may
do this for you. :)


Thanks

[1]: <https://www.gnu.org/prep/standards/standards.html#Change-Logs>




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#65285] [PATCH] gnu: emacs-orderless: Update to 1.0.
  2023-08-14 10:52 [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0 Piotr Kwiecinski
  2023-08-15  7:27 ` Hilton Chain via Guix-patches via
@ 2023-08-15  9:19 ` Piotr Kwiecinski
  2023-08-27  3:54   ` bug#65285: " Hilton Chain via Guix-patches via
  1 sibling, 1 reply; 5+ messages in thread
From: Piotr Kwiecinski @ 2023-08-15  9:19 UTC (permalink / raw)
  To: 65285; +Cc: Piotr Kwiecinski

* gnu/packages/emacs-xyz.scm (emacs-orderless): Update to 1.0, previously used
commit was pointing to 0.8.
---
 gnu/packages/emacs-xyz.scm | 57 +++++++++++++++++++-------------------
 1 file changed, 28 insertions(+), 29 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 0f0751e39a..877895f011 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -11290,39 +11290,38 @@ (define-public emacs-selectrum
     (license license:expat)))
 
 (define-public emacs-orderless
-  (let ((commit "004cee6b8e01f8eb0cb1c683d0a637b14890600f"))
-    (package
-      (name "emacs-orderless")
-      (version "1.0")
-      (source
-       (origin
-         (method git-fetch)
-         (uri (git-reference
-               (url "https://github.com/oantolin/orderless")
-               (commit commit)))
-         (sha256
-          (base32 "115bwqi2yc44bgvcl7lha8p2s6jgh5hksn4wa9s0kpfxhi14jwmy"))
-         (file-name (git-file-name name version))))
-      (build-system emacs-build-system)
-      (arguments
-       (list
-        #:phases
-        #~(modify-phases %standard-phases
-            (add-after 'install 'makeinfo
-              (lambda _
-                (invoke "makeinfo" "orderless.texi")
-                (install-file "orderless.info"
-                              (string-append #$output "/share/info")))))))
-      (native-inputs
-       (list texinfo))
-      (home-page "https://github.com/oantolin/orderless")
-      (synopsis "Emacs completion style that matches multiple regexps in any order")
-      (description "This package provides an orderless completion style that
+  (package
+    (name "emacs-orderless")
+    (version "1.0")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/oantolin/orderless")
+             (commit version)))
+       (sha256
+        (base32 "1jmwyici90v82gqjq44n0scrwhmg05pi9yplkn4bpss80c83dxp4"))
+       (file-name (git-file-name name version))))
+    (build-system emacs-build-system)
+    (arguments
+     (list
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-after 'install 'makeinfo
+            (lambda _
+              (invoke "makeinfo" "orderless.texi")
+              (install-file "orderless.info"
+                            (string-append #$output "/share/info")))))))
+    (native-inputs
+     (list texinfo))
+    (home-page "https://github.com/oantolin/orderless")
+    (synopsis "Emacs completion style that matches multiple regexps in any order")
+    (description "This package provides an orderless completion style that
 divides the pattern into space-separated components, and matches candidates
 that match all of the components in any order.  Each component can match in
 any one of several ways: literally, as a regexp, as an initialism, in the flex
 style, or as multiple word prefixes.")
-      (license license:gpl3+))))
+    (license license:gpl3+)))
 
 (define-public emacs-consult
   (package

base-commit: 642ae73ec0c38ee4758ad9d39f16232c8945c6b6
-- 
2.39.2





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0
  2023-08-15  7:27 ` Hilton Chain via Guix-patches via
@ 2023-08-15  9:32   ` Piotr Kwieciński
  0 siblings, 0 replies; 5+ messages in thread
From: Piotr Kwieciński @ 2023-08-15  9:32 UTC (permalink / raw)
  To: Hilton Chain; +Cc: 65285

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

Hi Hilton,
thank you for the guidance and patience.
I hope the updated patch is correct now.

On Tue, 15 Aug 2023 at 09:29, Hilton Chain <hako@ultrarare.space> wrote:

> Hi Piotr,
>
> Thank you for the patch!  There're some minor changes to make.
>
> On Mon, 14 Aug 2023 18:52:38 +0800,
> Piotr Kwiecinski wrote:
> >
> > previously used commit was pointing to 0.8 instead 1.0
> > ---
>
>
> Please write commit logs in ChangeLog format [1].  You can also check
> the commit history for examples.
>
> For this commit, the following can be used:
> --8<---------------cut here---------------start------------->8---
> gnu: emacs-orderless: Update to 1.0.
>
> * gnu/packages/emacs-xyz.scm (emacs-orderless): Update to 1.0, previously
> used
> commit was pointing to 0.8.
> --8<---------------cut here---------------end--------------->8---
>
>
> >  gnu/packages/emacs-xyz.scm | 9 ++++-----
> >  1 file changed, 4 insertions(+), 5 deletions(-)
> >
> > diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> > index 0f0751e39a..e62c71752e 100644
> > --- a/gnu/packages/emacs-xyz.scm
> > +++ b/gnu/packages/emacs-xyz.scm
> > @@ -11290,8 +11290,7 @@ (define-public emacs-selectrum
> >      (license license:expat)))
> >
> >  (define-public emacs-orderless
> > -  (let ((commit "004cee6b8e01f8eb0cb1c683d0a637b14890600f"))
> > -    (package
> > +  (package
> >        (name "emacs-orderless")
> >        (version "1.0")
> >        (source
> > @@ -11299,9 +11298,9 @@ (define-public emacs-orderless
> >           (method git-fetch)
> >           (uri (git-reference
> >                 (url "https://github.com/oantolin/orderless")
> > -               (commit commit)))
> > +               (commit version)))
> >           (sha256
> > -          (base32
> "115bwqi2yc44bgvcl7lha8p2s6jgh5hksn4wa9s0kpfxhi14jwmy"))
> > +          (base32
> "1jmwyici90v82gqjq44n0scrwhmg05pi9yplkn4bpss80c83dxp4"))
> >           (file-name (git-file-name name version))))
> >        (build-system emacs-build-system)
> >        (arguments
> > @@ -11322,7 +11321,7 @@ (define-public emacs-orderless
> >  that match all of the components in any order.  Each component can
> match in
> >  any one of several ways: literally, as a regexp, as an initialism, in
> the flex
> >  style, or as multiple word prefixes.")
> > -      (license license:gpl3+))))
> > +      (license license:gpl3+)))
> >
> >  (define-public emacs-consult
> >    (package
> >
> > base-commit: b15381460ed74e72792ff182dd2ca7a06ba59b0c
> > --
> > 2.39.2
>
>
> I think it's fine to also change the indentation here, remove two
> leading spaces in every line after the "(package" one, your editor may
> do this for you. :)
>
>
> Thanks
>
> [1]: <https://www.gnu.org/prep/standards/standards.html#Change-Logs>
>


-- 
Regards
Piotr Kwiecinski

[-- Attachment #2: Type: text/html, Size: 4033 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#65285: [PATCH] gnu: emacs-orderless: Update to 1.0.
  2023-08-15  9:19 ` [bug#65285] [PATCH] gnu: emacs-orderless: Update " Piotr Kwiecinski
@ 2023-08-27  3:54   ` Hilton Chain via Guix-patches via
  0 siblings, 0 replies; 5+ messages in thread
From: Hilton Chain via Guix-patches via @ 2023-08-27  3:54 UTC (permalink / raw)
  To: Piotr Kwiecinski; +Cc: 65285-done

Hi Piotr,

On Tue, 15 Aug 2023 17:19:03 +0800,
Piotr Kwiecinski wrote:
>
> * gnu/packages/emacs-xyz.scm (emacs-orderless): Update to 1.0, previously used
> commit was pointing to 0.8.
> ---

Applied as e9bea73e2da021cfccfb5356d378f2feb668742c, thank you for the
contribution!




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-08-27  4:37 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-14 10:52 [bug#65285] [PATCH] gnu: update emacs-orderless to 1.0 Piotr Kwiecinski
2023-08-15  7:27 ` Hilton Chain via Guix-patches via
2023-08-15  9:32   ` Piotr Kwieciński
2023-08-15  9:19 ` [bug#65285] [PATCH] gnu: emacs-orderless: Update " Piotr Kwiecinski
2023-08-27  3:54   ` bug#65285: " Hilton Chain via Guix-patches via

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.