From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.lisp.guile.devel Subject: Re: [PATCH] Implement open-process and related functions on MinGW Date: Sat, 22 Feb 2014 17:43:33 +0200 Message-ID: <83ob1z2lwa.fsf@gnu.org> References: <834n3x8o7m.fsf@gnu.org> <83y519788a.fsf@gnu.org> <871tz0d5vc.fsf@gnu.org> <83iosc76kz.fsf@gnu.org> <87vbwc72dp.fsf_-_@gnu.org> <8361o74e0k.fsf@gnu.org> <87eh2vkxb8.fsf@yeeloong.lan> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE X-Trace: ger.gmane.org 1393083826 13476 80.91.229.3 (22 Feb 2014 15:43:46 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 22 Feb 2014 15:43:46 +0000 (UTC) Cc: ludo@gnu.org, guile-devel@gnu.org To: Mark H Weaver Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Sat Feb 22 16:43:51 2014 Return-path: Envelope-to: guile-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1WHEk3-0002dJ-BJ for guile-devel@m.gmane.org; Sat, 22 Feb 2014 16:43:51 +0100 Original-Received: from localhost ([::1]:49949 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WHEk2-0004js-T6 for guile-devel@m.gmane.org; Sat, 22 Feb 2014 10:43:50 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:53083) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WHEjv-0004jj-1T for guile-devel@gnu.org; Sat, 22 Feb 2014 10:43:48 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WHEjp-0001g3-Mz for guile-devel@gnu.org; Sat, 22 Feb 2014 10:43:42 -0500 Original-Received: from mtaout27.012.net.il ([80.179.55.183]:45772) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WHEjp-0001fj-Ey; Sat, 22 Feb 2014 10:43:37 -0500 Original-Received: from conversion-daemon.mtaout27.012.net.il by mtaout27.012.net.il (HyperSendmail v2007.08) id <0N1E00800LVEA800@mtaout27.012.net.il>; Sat, 22 Feb 2014 17:41:49 +0200 (IST) Original-Received: from HOME-C4E4A596F7 ([87.69.4.28]) by mtaout27.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0N1E00080M9PVE80@mtaout27.012.net.il>; Sat, 22 Feb 2014 17:41:49 +0200 (IST) In-reply-to: <87eh2vkxb8.fsf@yeeloong.lan> X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 80.179.55.183 X-BeenThere: guile-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Developers list for Guile, the GNU extensibility library" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Original-Sender: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.lisp.guile.devel:16913 Archived-At: > From: Mark H Weaver > Cc: ludo@gnu.org (Ludovic Court=E8s), guile-devel@gnu.org > Date: Sat, 22 Feb 2014 09:59:39 -0500 >=20 > Eli Zaretskii writes: >=20 > > This patch allows the MinGW build of Guile to have the process re= lated > > functions (open-process, kill, waitpid, status:exit-val, etc.). > > > > Implement open-process and related functions on MinGW > > > > * libguile/filesys.c (fsync, link) [__MINGW32__]: Redirect Posix > > functions to their Windows equivalents. >=20 > Gnulib has an 'fsync' module that apparently implements it on MinGW= . > I think we should use that instead, no? Yes, see my other message. > > (scm_chown, scm_open_fdes, scm_open, scm_close, scm_close_fdes) > > (scm_link, scm_chdir, set_element, fill_select_type) > > (get_element, retrieve_select_type, scm_select, scm_fcntl) > > (scm_fsync, scm_symlink, scm_readlink, scm_copy_file, scm_sendf= ile) > > (scm_dir_print, scm_dir_free): These functions are no longer > > wholesale ifdef'ed away if HAVE_POSIX is not defined. >=20 > As I wrote in my other recent message, I think we should simply > recommend that MinGW builds be done without "--disable-posix", sinc= e we > have at least one report that it works now. OK.