bug-gnulib@gnu.org mirror (unofficial)
 help / color / mirror / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Bruno Haible <bruno@clisp.org>
Cc: Gnulib bugs <bug-gnulib@gnu.org>
Subject: Re: [PATCH 1/2] parse-datetime2: new module
Date: Sat, 6 Mar 2021 08:26:09 -0800	[thread overview]
Message-ID: <57fb5ae5-3b23-ae65-6eb8-2d400571628d@cs.ucla.edu> (raw)
In-Reply-To: <7310414.fi537E5Mvj@omega>

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

On 3/2/21 7:25 PM, Bruno Haible wrote:

> $ ./gnulib-tool --create-testdir --dir=../testdir-all --single-configure
> gnulib-tool: warning: module parse-datetime2 depends on a module with an incompatible license: gettime
...

Thanks for the heads-up; I had forgotten about the parse-datetime 
special case for licensing. I installed the attached 1-byte patch to 
work around the issue.

[-- Attachment #2: 0001-parse-datetime2-fix-licensing.patch --]
[-- Type: text/x-patch, Size: 1538 bytes --]

From 487b9551b63ef936a6be6df38d1c9484cd97810c Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Sat, 6 Mar 2021 08:23:48 -0800
Subject: [PATCH] parse-datetime2: fix licensing

Problem reported by Bruno Haible in:
https://lists.gnu.org/r/bug-gnulib/2021-03/msg00017.html
* gnulib-tool (func_get_license): Treat parse-datetime2
(actually, anything starting with "parse-datetime")
like parse-datetime, as far as licenses go.
---
 ChangeLog   | 9 +++++++++
 gnulib-tool | 2 +-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 86c3e04e8..9706d7826 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2021-03-06  Paul Eggert  <eggert@cs.ucla.edu>
+
+	parse-datetime2: fix licensing
+	Problem reported by Bruno Haible in:
+	https://lists.gnu.org/r/bug-gnulib/2021-03/msg00017.html
+	* gnulib-tool (func_get_license): Treat parse-datetime2
+	(actually, anything starting with "parse-datetime")
+	like parse-datetime, as far as licenses go.
+
 2021-03-06  Bruno Haible  <bruno@clisp.org>
 
 	free-posix: Update documentation.
diff --git a/gnulib-tool b/gnulib-tool
index 2ba5dec65..23b2c4456 100755
--- a/gnulib-tool
+++ b/gnulib-tool
@@ -2683,7 +2683,7 @@ func_get_license ()
       ;;
   esac
   case "$1" in
-    parse-datetime )
+    parse-datetime* )
       # These modules are under a weaker license only for the purpose of some
       # users who hand-edit it and don't use gnulib-tool. For the regular
       # gnulib users they are under a stricter license.
-- 
2.27.0


  reply	other threads:[~2021-03-06 16:26 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-01  8:29 [PATCH 1/2] parse-datetime2: new module Paul Eggert
2021-03-01  8:29 ` [PATCH 2/2] unlocked-io: do not redefine getc_unlocked etc Paul Eggert
2021-03-03  3:25 ` [PATCH 1/2] parse-datetime2: new module Bruno Haible
2021-03-06 16:26   ` Paul Eggert [this message]
2021-03-06 17:54     ` Bruno Haible

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.gnu.org/mailman/listinfo/bug-gnulib

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57fb5ae5-3b23-ae65-6eb8-2d400571628d@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=bruno@clisp.org \
    --cc=bug-gnulib@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).