unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Add oauth2 customize group (resolve FIXME)
@ 2021-02-19  1:16 Naoya Yamashita
  2021-02-19 10:17 ` Julien Danjou
  0 siblings, 1 reply; 5+ messages in thread
From: Naoya Yamashita @ 2021-02-19  1:16 UTC (permalink / raw)
  To: emacs-devel, julien

[-- Attachment #1: Type: Text/Plain, Size: 214 bytes --]


Hi, thanks for maintaining such a great project!

I find oauth2 ELPA package and also find a easy FIXME comment.
This patch add a new group and changed all customization
variables to belong to it.

Regards,
Naoya

[-- Attachment #2: 0001-packages-oauth2-oauth2.el-Add-oauth2-group-and-use-i.patch --]
[-- Type: Text/X-Patch, Size: 1469 bytes --]

From b6619724abc1f268eb3b68bc9077f899f9d70bea Mon Sep 17 00:00:00 2001
From: Naoya Yamashita <conao3@gmail.com>
Date: Fri, 19 Feb 2021 09:56:48 +0900
Subject: [PATCH] * packages/oauth2/oauth2.el: Add oauth2 group and use it

(oauth2): New group.
(oauth2-token-file): Belongs to it.
---
 oauth2.el | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/oauth2.el b/oauth2.el
index 43c4e9e4d4..43e48c1dbe 100644
--- a/oauth2.el
+++ b/oauth2.el
@@ -47,6 +47,12 @@
 (defvar url-callback-arguments)
 (defvar url-callback-function)
 
+(defgroup oauth2 nil
+  "OAuth 2.0 Authorization Protocol."
+  :group 'comm
+  :link '(url-link :tag "Savannah" "http://git.savannah.gnu.org/cgit/emacs/elpa.git/tree/?h=externals/oauth2")
+  :link '(url-link :tag "ELPA" "https://elpa.gnu.org/packages/oauth2.html"))
+
 (defun oauth2-request-authorization (auth-url client-id &optional scope state redirect-uri)
   "Request OAuth authorization at AUTH-URL by launching `browse-url'.
 CLIENT-ID is the client id provided by the provider.
@@ -149,8 +155,7 @@ TOKEN should be obtained with `oauth2-request-access'."
 
 (defcustom oauth2-token-file (concat user-emacs-directory "oauth2.plstore")
   "File path where store OAuth tokens."
-  ;; FIXME: This var doesn't belong to any group.  Either add it to some
-  ;; pre-existing group or create an `oauth2' group for it.
+  :group 'oauth2
   :type 'file)
 
 (defun oauth2-compute-id (auth-url token-url scope)
-- 
2.30.0


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

* Re: Add oauth2 customize group (resolve FIXME)
  2021-02-19  1:16 Add oauth2 customize group (resolve FIXME) Naoya Yamashita
@ 2021-02-19 10:17 ` Julien Danjou
  2021-02-19 11:13   ` Naoya Yamashita
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Danjou @ 2021-02-19 10:17 UTC (permalink / raw)
  To: Naoya Yamashita; +Cc: emacs-devel

On Fri, Feb 19 2021, Naoya Yamashita wrote:

Thanks Naoya.

I pushed your change.

> Hi, thanks for maintaining such a great project!
>
> I find oauth2 ELPA package and also find a easy FIXME comment.
> This patch add a new group and changed all customization
> variables to belong to it.
>
> Regards,
> Naoya
>
> From b6619724abc1f268eb3b68bc9077f899f9d70bea Mon Sep 17 00:00:00 2001
> From: Naoya Yamashita <conao3@gmail.com>
> Date: Fri, 19 Feb 2021 09:56:48 +0900
> Subject: [PATCH] * packages/oauth2/oauth2.el: Add oauth2 group and use it
>
> (oauth2): New group.
> (oauth2-token-file): Belongs to it.
> ---
>  oauth2.el | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/oauth2.el b/oauth2.el
> index 43c4e9e4d4..43e48c1dbe 100644
> --- a/oauth2.el
> +++ b/oauth2.el
> @@ -47,6 +47,12 @@
>  (defvar url-callback-arguments)
>  (defvar url-callback-function)
>  
> +(defgroup oauth2 nil
> +  "OAuth 2.0 Authorization Protocol."
> +  :group 'comm
> +  :link '(url-link :tag "Savannah" "http://git.savannah.gnu.org/cgit/emacs/elpa.git/tree/?h=externals/oauth2")
> +  :link '(url-link :tag "ELPA" "https://elpa.gnu.org/packages/oauth2.html"))
> +
>  (defun oauth2-request-authorization (auth-url client-id &optional scope state redirect-uri)
>    "Request OAuth authorization at AUTH-URL by launching `browse-url'.
>  CLIENT-ID is the client id provided by the provider.
> @@ -149,8 +155,7 @@ TOKEN should be obtained with `oauth2-request-access'."
>  
>  (defcustom oauth2-token-file (concat user-emacs-directory "oauth2.plstore")
>    "File path where store OAuth tokens."
> -  ;; FIXME: This var doesn't belong to any group.  Either add it to some
> -  ;; pre-existing group or create an `oauth2' group for it.
> +  :group 'oauth2
>    :type 'file)
>  
>  (defun oauth2-compute-id (auth-url token-url scope)

-- 
Julien Danjou
/* Free Software hacker
   https://julien.danjou.info */



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

* Re: Add oauth2 customize group (resolve FIXME)
  2021-02-19 10:17 ` Julien Danjou
@ 2021-02-19 11:13   ` Naoya Yamashita
  2021-02-19 15:14     ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Naoya Yamashita @ 2021-02-19 11:13 UTC (permalink / raw)
  To: julien; +Cc: emacs-devel


> Thanks Naoya.
> 
> I pushed your change.
> 

Thanks, but I'm bit sad when see the commit.
http://git.savannah.gnu.org/cgit/emacs/elpa.git/commit/?h=externals/oauth2&id=dc069550616fb0a72507489ea796d0e1bd8b48c9

The commit `author` should be `Naoya Yamashita <conao3@gmail.com>`.

You may think it's a small thing, but OSS contributors are
motivated by commits being properly marked as my contribution, so
pushing inappropriate commits may demotivate them.

This time is okay, but please pay attention to the Author field next time!



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

* Re: Add oauth2 customize group (resolve FIXME)
  2021-02-19 11:13   ` Naoya Yamashita
@ 2021-02-19 15:14     ` Stefan Monnier
  2021-02-19 16:57       ` Naoya Yamashita
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Monnier @ 2021-02-19 15:14 UTC (permalink / raw)
  To: Naoya Yamashita; +Cc: julien, emacs-devel

Hi,

> The commit `author` should be `Naoya Yamashita <conao3@gmail.com>`.
>
> You may think it's a small thing, but OSS contributors are
> motivated by commits being properly marked as my contribution, so
> pushing inappropriate commits may demotivate them.
>
> This time is okay, but please pay attention to the Author field next time!

Very well said, thank you for reminding us (copyright attribution is
also another reason why we want to be careful with it).


        Stefan




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

* Re: Add oauth2 customize group (resolve FIXME)
  2021-02-19 15:14     ` Stefan Monnier
@ 2021-02-19 16:57       ` Naoya Yamashita
  0 siblings, 0 replies; 5+ messages in thread
From: Naoya Yamashita @ 2021-02-19 16:57 UTC (permalink / raw)
  To: monnier; +Cc: julien, emacs-devel

> Very well said, thank you for reminding us (copyright attribution is
> also another reason why we want to be careful with it).

Thanks for your comment, Stefan!

Again, thanks to all of you maintainers and committers, I can
continue to use Emacs. Thank you!



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

end of thread, other threads:[~2021-02-19 16:57 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-19  1:16 Add oauth2 customize group (resolve FIXME) Naoya Yamashita
2021-02-19 10:17 ` Julien Danjou
2021-02-19 11:13   ` Naoya Yamashita
2021-02-19 15:14     ` Stefan Monnier
2021-02-19 16:57       ` Naoya Yamashita

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).