guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 08/10] gnu: readline: support mingw.


From: Jan Nieuwenhuizen
Subject: Re: [PATCH 08/10] gnu: readline: support mingw.
Date: Mon, 02 May 2016 21:25:36 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Andy Wingo 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.

Oops, didn't mean to snip this.  I mailed Alexey Pavlov and didn't get a
response yet.  I found an earlier patch that this seems to add a bit
too.

As you can see, it's #ifdef'ing out macros that are missing on mingw:
S_ISUID, S_ISGID, S_ISLNK, S_ISSOCK.

And now that I look into it, also some that were missing but are now
present: SIGHUP, SIGQUIT.  And defining S_ISDIR which is a straight copy
from stat.h, but is now present in mingw too.

Another possibility would be to just patch those macros into mingw's
stat.h and drop this patch.  WDYT?

Jan

-- 
Jan Nieuwenhuizen <address@hidden> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | AvatarĀ®  http://AvatarAcademy.nl  



reply via email to

[Prev in Thread] Current Thread [Next in Thread]