Re: parse-datetime license

[Giuseppe Scrivano]

Paul Eggert  writes:

> I think I'd rather leave that one GPL, as it's pretty complicated and
> doesn't redo anything in any library.  It was originally intended to
> share code among GPLed apps.

unfortunately this will will prevent OSTree[1] which is distributed
under GPLv2+ to use parse-datetime which is using GPLv3+.

My request is to change its license to GPLv2+ so to allow programs
distributed under GPLv2+, such as OSTree, to use it.

It has already happened in the past that the license for some modules
was changed to accomodate integrations with other programs.  Why
wouldn't this be the case as well?

Thanks,
Giuseppe

1) https://wiki.gnome.org/Projects/OSTree

Re: parse-datetime license

[Paul Eggert]

Giuseppe Scrivano wrote:

My request is to change its license to GPLv2+ so to allow programs
distributed under GPLv2+


Ah, sorry, I thought you meant LGPL.  Still, the GNU project prefers GPLv3 these 
days, and changing this particular module to be v2 would probably require 
consultation with RMS to know the particulars.  Please feel free to bug him 
about it (you'll need to bring him up to speed).

Silence obstack.c -Wc++compat warning

[Alan Modra]

Fixes
warning: request for implicit conversion from ‘void *’ to ‘struct 
_obstack_chunk *’ not permitted in C++ [-Wc++-compat]

I moved the assignment to h->chunk to fix an overlong line, then
decided it would be better after the alloc failure check just to do
things the same way as in _obstack_newchunk.

* obstack.c (_obstack_newchunk): Silence -Wc++compat warning.
(_obstack_begin_worker): Likewise.  Move assignment to h->chunk
after alloc failure check.

diff --git a/lib/obstack.c b/lib/obstack.c
index 3b99dfa..5ad3169 100644
--- a/lib/obstack.c
+++ b/lib/obstack.c
@@ -133,9 +133,10 @@ _obstack_begin_worker (struct obstack *h,
   h->chunk_size = size;
   h->alignment_mask = alignment - 1;
 
-  chunk = h->chunk = call_chunkfun (h, h->chunk_size);
+  chunk = (struct _obstack_chunk *) call_chunkfun (h, h->chunk_size);
   if (!chunk)
 (*obstack_alloc_failed_handler) ();
+  h->chunk = chunk;
   h->next_free = h->object_base = __PTR_ALIGN ((char *) chunk, chunk->contents,
alignment - 1);
   h->chunk_limit = chunk->limit = (char *) chunk + h->chunk_size;
@@ -197,7 +198,7 @@ _obstack_newchunk (struct obstack *h, _OBSTACK_SIZE_T 
length)
 
   /* Allocate and initialize the new chunk.  */
   if (obj_size <= sum1 && sum1 <= sum2)
-new_chunk = call_chunkfun (h, new_size);
+new_chunk = (struct _obstack_chunk *) call_chunkfun (h, new_size);
   if (!new_chunk)
 (*obstack_alloc_failed_handler)();
   h->chunk = new_chunk;

-- 
Alan Modra
Australia Development Lab, IBM