all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages.
@ 2022-09-22  1:21 Hilton Chain via Guix-patches via
  2022-09-23 16:29 ` [bug#57992] Emit != omit Liliana Marie Prikler
  2022-09-24 17:42 ` [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Nicolas Goaziou
  0 siblings, 2 replies; 6+ messages in thread
From: Hilton Chain via Guix-patches via @ 2022-09-22  1:21 UTC (permalink / raw)
  To: 57992

* gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch: New
file.
* gnu/local.mk (dist_patch_DATA): Add it.
* gnu/packages/emacs-xyz.scm (emacs-use-package)[patches]: Add
emacs-use-package-avoid-positional-arguments.patch.
---
 gnu/local.mk                                  |  1 +
 gnu/packages/emacs-xyz.scm                    |  6 ++++-
 ...e-package-avoid-positional-arguments.patch | 26 +++++++++++++++++++
 3 files changed, 32 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index ef1bae5f3c..731f64b053 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1054,6 +1054,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/emacs-source-date-epoch.patch		\
   %D%/packages/patches/emacs-telega-path-placeholder.patch	\
   %D%/packages/patches/emacs-telega-test-env.patch		\
+  %D%/packages/patches/emacs-use-package-avoid-positional-arguments.patch \
   %D%/packages/patches/emacs-wordnut-require-adaptive-wrap.patch	\
   %D%/packages/patches/emacs-yasnippet-fix-tests.patch	\
   %D%/packages/patches/emacs-kv-fix-tests.patch	\
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 32e4c33f16..762d11a776 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -14994,7 +14994,11 @@ (define-public emacs-use-package
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "088kl3bml0rs5bkfymgzr15ram9qvy66h1kaisrbkynh0yxvf8g9"))))
+                "088kl3bml0rs5bkfymgzr15ram9qvy66h1kaisrbkynh0yxvf8g9"))
+              (patches
+               (search-patches
+                ;; NOTE: Cherry-picked from upstream commit.
+                "emacs-use-package-avoid-positional-arguments.patch"))))
     (build-system emacs-build-system)
     (native-inputs
      (list texinfo))
diff --git a/gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch b/gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch
new file mode 100644
index 0000000000..82b52b0400
--- /dev/null
+++ b/gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch
@@ -0,0 +1,26 @@
+From cb90d3fa4168d7e1b2f035f4fbb0d5d08db1fbec Mon Sep 17 00:00:00 2001
+From: Jonas Bernoulli <jonas@bernoul.li>
+Date: Tue, 18 May 2021 15:44:34 +0200
+Subject: [PATCH] Avoid positional arguments to define-minor-mode
+
+Back in Emacs-21.1, `define-minor-mode' grew keyword arguments to
+replace its old positional arguments.  Starting with Emacs-28.1
+a warning will be omitted if positional arguments are still used.
+---
+ bind-key.el | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/bind-key.el b/bind-key.el
+index 1d611c2..9a2ddcd 100644
+--- a/bind-key.el
++++ b/bind-key.el
+@@ -131,7 +131,8 @@
+ 
+ (define-minor-mode override-global-mode
+   "A minor mode so that keymap settings override other modes."
+-  t "")
++  :global t
++  :lighter "")
+ 
+ ;; the keymaps in `emulation-mode-map-alists' take precedence over
+ ;; `minor-mode-map-alist'

base-commit: 1a1ff7ed8fdaae58aff0e8625399cd7354038ca3
-- 
2.37.3





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

* [bug#57992] Emit != omit
  2022-09-22  1:21 [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Hilton Chain via Guix-patches via
@ 2022-09-23 16:29 ` Liliana Marie Prikler
  2022-09-24 23:41   ` [bug#57993] " Hilton Chain via Guix-patches via
  2022-09-24 17:42 ` [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Nicolas Goaziou
  1 sibling, 1 reply; 6+ messages in thread
From: Liliana Marie Prikler @ 2022-09-23 16:29 UTC (permalink / raw)
  To: Hilton Chain; +Cc: 57993, 57992, 57997

Emit: to throw or give out; "emit a warning" means "print a warning"
Omit: to leave out; "omit a warning" means "not printing a warning even
though you should".

In these cases, you are proactively working against the warnings, i.e.
preventing them.

Cheers




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

* [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages.
  2022-09-22  1:21 [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Hilton Chain via Guix-patches via
  2022-09-23 16:29 ` [bug#57992] Emit != omit Liliana Marie Prikler
@ 2022-09-24 17:42 ` Nicolas Goaziou
  2022-09-24 23:45   ` Hilton Chain via Guix-patches via
  1 sibling, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2022-09-24 17:42 UTC (permalink / raw)
  To: 57992; +Cc: hako

Hello,

Hilton Chain via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/patches/emacs-use-package-avoid-positional-arguments.patch: New
> file.
> * gnu/local.mk (dist_patch_DATA): Add it.
> * gnu/packages/emacs-xyz.scm (emacs-use-package)[patches]: Add
> emacs-use-package-avoid-positional-arguments.patch.

Thank you.

However, unless I'm missing something, I'm not sure we should bother
fixing compilation warnings. This patch can wait next version bump,
can't it?

Regards,
-- 
Nicolas Goaziou




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

* [bug#57993] Emit != omit
  2022-09-23 16:29 ` [bug#57992] Emit != omit Liliana Marie Prikler
@ 2022-09-24 23:41   ` Hilton Chain via Guix-patches via
  0 siblings, 0 replies; 6+ messages in thread
From: Hilton Chain via Guix-patches via @ 2022-09-24 23:41 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: 57993, 57992, 57997

On Sat, 24 Sep 2022 00:29:16 +0800,
Liliana Marie Prikler wrote:
> 
> Emit: to throw or give out; "emit a warning" means "print a warning"
> Omit: to leave out; "omit a warning" means "not printing a warning even
> though you should".
> 
> In these cases, you are proactively working against the warnings, i.e.
> preventing them.
> 
> Cheers

Oops :P
I'll take care in the future, thank you!




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

* [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages.
  2022-09-24 17:42 ` [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Nicolas Goaziou
@ 2022-09-24 23:45   ` Hilton Chain via Guix-patches via
  2022-09-25 12:55     ` Nicolas Goaziou
  0 siblings, 1 reply; 6+ messages in thread
From: Hilton Chain via Guix-patches via @ 2022-09-24 23:45 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: 57992

On Sun, 25 Sep 2022 01:42:38 +0800,
Nicolas Goaziou wrote:
> However, unless I'm missing something, I'm not sure we should bother
> fixing compilation warnings. This patch can wait next version bump,
> can't it?

No problem :)




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

* [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages.
  2022-09-24 23:45   ` Hilton Chain via Guix-patches via
@ 2022-09-25 12:55     ` Nicolas Goaziou
  0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Goaziou @ 2022-09-25 12:55 UTC (permalink / raw)
  To: 57992; +Cc: Hilton Chain, 57992-close

Hello,

Hilton Chain via Guix-patches via <guix-patches@gnu.org> writes:

> On Sun, 25 Sep 2022 01:42:38 +0800,
> Nicolas Goaziou wrote:
>> However, unless I'm missing something, I'm not sure we should bother
>> fixing compilation warnings. This patch can wait next version bump,
>> can't it?
>
> No problem :)

Fair enough. Closing then. Feel free to re-open the issue if you change
your mind.

Regards,
-- 
Nicolas Goaziou




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

end of thread, other threads:[~2022-09-25 13:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-22  1:21 [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Hilton Chain via Guix-patches via
2022-09-23 16:29 ` [bug#57992] Emit != omit Liliana Marie Prikler
2022-09-24 23:41   ` [bug#57993] " Hilton Chain via Guix-patches via
2022-09-24 17:42 ` [bug#57992] [PATCH] gnu: emacs-use-package: Emit deprecation messages Nicolas Goaziou
2022-09-24 23:45   ` Hilton Chain via Guix-patches via
2022-09-25 12:55     ` Nicolas Goaziou

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.