Re: [PATCH 0/3] fenv support round 2

[Corinna Vinschen <vinschen at redhat dot com>]

On Aug 28 10:06, joel@rtems.org wrote:
> From: Joel Sherrill <joel@rtems.org>
> 
> There are three patches in this set and hopefully two will be uncontroversial.
> 
> (1) The common include/fenv.h was copied from riscv, so delete the 
> copy in the libc/machine/riscv directory.
> 
> (2) Add glibc extension FE_NOMASK_ENV to template sys/fenv.h.
> Code was copied from Cygwin fenv.h.
> 
> (3) Add x86_64 and i386 fenv support by copying from Cygwin. No
> functional changes were made. This needs some help.
> 
>    (a) x86_64 isn't picking up fenv.c. Not sure what isn't bootstrapping right.
>        Help needed to get it integrated into libm build.

Not sure yet, still looking into the code.

>    (b) I added a copyright to fenv.c since it just said "part of Cygwin".
>        I hope it is right.

No, that's not how it should be licensed since that implies GPL3+.
We're moving the file into newlib so we should loosen the copyright
so that the code is usable for non-GPLed code.  And as the sole
copyright holder, Red Hat can move the file to another copyright.

So what you can do here is to remove the entire Cygwin header from
the file and just keep a BSD-3-Clause SPDX header.

> After (3) is merged, a Cygwin maintainer will need to clean up winsup
> and make sure all is good.

Will check.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: signature.asc
Description: PGP signature