From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Michael J Gruber <git@grubix.eu>
Cc: git@vger.kernel.org, Adam Dinwoodie <adam@dinwoodie.org>,
Junio C Hamano <gitster@pobox.com>, Jeff King <peff@peff.net>,
Ramsay Jones <ramsay@ramsayjones.plus.com>
Subject: Re: [PATCH 1/2] test-lib: group system specific FIFO tests by system
Date: Fri, 15 Sep 2017 21:32:53 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.21.1.1709152128000.219280@virtualbox> (raw)
In-Reply-To: <a159230a-d406-c772-13f0-f82a36faf763@grubix.eu>
Hi Michael,
On Fri, 15 Sep 2017, Michael J Gruber wrote:
> Johannes Schindelin venit, vidit, dixit 15.09.2017 00:21:
> >
> > On Thu, 14 Sep 2017, Michael J Gruber wrote:
> >
> >> test-lib determines whether a file-system supports FIFOs and needs to do
> >> special casing for CYGWIN and MINGW. This separates those system
> >> specific settings from those at more central place.
> >>
> >> Set mkfifo() to false in the central system specific place so that the
> >> same test works everywhere.
> >
> > The mkfifo() emulation of Cygwin seems to work, no? I think it works even
> > in MSYS2, but not in MINGW.
> >
> > So maybe this patch should affect only the MINGW arm?
>
> I only reorganised the code, so in that sense the patch does not affect
> any system ;)
>
> If indeed mkfifo works on CYGWIN than a separate patch should remove the
> exclusion of CYGWIN; alas, I can't confirm (I wish MS still had the old
> academic alliance programme).
You don't need an academic license, as there are free VMs available (they
expire after a while, though):
https://developer.microsoft.com/en-us/windows/downloads/virtual-machines
It seems that the current free VM will expire in 4 days. After that,
probably a new one will be available that expires some time later (not
sure about the exact time frame of those VMs).
Alternatively, you can download VMs meant for web testing at
https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ which
expire after 90 days.
Ciao,
Dscho
prev parent reply other threads:[~2017-09-15 19:33 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-13 19:00 [PATCH] test-lib: don't use ulimit in test prerequisites on cygwin Ramsay Jones
2017-09-13 19:20 ` Jonathan Nieder
2017-09-14 8:13 ` Michael J Gruber
2017-09-14 12:54 ` Johannes Schindelin
2017-09-14 14:52 ` [PATCH 1/2] test-lib: group system specific FIFO tests by system Michael J Gruber
2017-09-14 14:52 ` [PATCH 2/2] test-lib: ulimit does not limit on CYGWIN and MINGW Michael J Gruber
2017-09-14 17:31 ` Ramsay Jones
2017-09-14 22:21 ` [PATCH 1/2] test-lib: group system specific FIFO tests by system Johannes Schindelin
2017-09-15 10:31 ` Michael J Gruber
2017-09-15 16:38 ` Ramsay Jones
2017-09-15 19:32 ` Johannes Schindelin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.21.1.1709152128000.219280@virtualbox \
--to=johannes.schindelin@gmx.de \
--cc=adam@dinwoodie.org \
--cc=git@grubix.eu \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=peff@peff.net \
--cc=ramsay@ramsayjones.plus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).