Re: [PATCH v5 2/9] t: import the clar unit testing framework

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

 



On Mon, Aug 19, 2024 at 02:21:44PM -0700, Junio C Hamano wrote:
> Patrick Steinhardt <ps@xxxxxx> writes:
> 
> > diff --git a/t/unit-tests/clar/clar.c b/t/unit-tests/clar/clar.c
> > new file mode 100644
> > index 0000000000..3fc2c76815
> > --- /dev/null
> > +++ b/t/unit-tests/clar/clar.c
> > @@ -0,0 +1,842 @@
> > ...
> > +
> > +#ifdef _WIN32
> > +#	include <windows.h>
> > +#	include <io.h>
> > +#	include <shellapi.h>
> > +#	include <direct.h>
> 
> This seems to break Windows build
> 
>   In file included from t/unit-tests/clar/clar.c:22:
>   D:/git-sdk-64-minimal/mingw64/include/windows.h:89:10: fatal error: shellapi.h: No such file or directory
>      89 | #include <shellapi.h>
>         |          ^~~~~~~~~~~~
>   compilation terminated.
> 
> https://github.com/git/git/actions/runs/10459342252/job/28963152995
> 
> As we have other topics that I do want to do well in 'seen' before
> merging them to 'next', I'd temporarily drop this topic from 'seen'
> and push out the rest of the topics.

Makes sense. I (or rather Dscho, thanks!) will have a deeper look at
Windows and then also rebase the branch to fix conflicts with the
updated tests in "master".

Patrick




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux