On Fri 29 Apr 2016 17:12, Jan Nieuwenhuizen <jann...@gnu.org> writes:

> * gnu/packages/patches/readline-6.3-mingw.patch: New file.
> * gnu-system.am (dist_patch_DATA): Add it.
> * gnu/packages/readline.scm (readline): Support mingw.

Can you clarify the licensing on the patch, and note its original
author?  Ideally we should send the patch upstream, too.

>        (arguments `(#:configure-flags
>                     (list (string-append "LDFLAGS=-Wl,-rpath -Wl,"
>                                          (assoc-ref %build-inputs "ncurses")
> -                                        "/lib")
> +                                        ,(if (mingw-target?) "/bin" "/lib"))

Is there a reason for MinGW ncurses to put its library in /bin on a
MinGW system?  Better if it put it in /lib like everything else...

Andy

Reply via email to