Closed tillea closed 2 years ago
I have no idea what this may be - nor do I have a way to repro. @onqtam - can you take a look? Any suggestions?
I'm no longer working on doctest and have been superseded by a different maintainer, but from a quick glance I see that argh is still using doctest 1.2.2 which is almost 5 years old - I'd suggest trying with the latest version and reporting back if that fixes the problem (because this error doesn't ring a bell).
doctest has been changed quite a lot in the last couple of years but the interface is 100% the same so upgrading should be painless. doctest 2.x requires C++11 though...
Hi,
Am Fri, Mar 18, 2022 at 01:13:02AM -0700 schrieb Viktor Kirilov:
I'm no longer working on doctest and have been superseded by a different maintainer, but from a quick glance I see that argh is still using doctest 1.2.2 which is almost 5 years ago - I'd suggest trying with the latest version and reporting back if that fixes the problem (because this error doesn't ring a bell).
doctest has been changed quite a lot in the last couple of years but the interface is 100% the same so upgrading should be painless.
Hmmm, since you are mentioning doctest: I'm not using the bundled doctest inside the Debian package. doctest.h is excluded and I'm rather using the Debian packaged version (2.4.8) to build the package.
Kind regards, Andreas.
Can you provide a minimal reproducible example that includes both frameworks and has this problem? From what I can tell this is not the argh_tests.cpp
source file in this repository because it seems to be including the doctest.h
version that is in this repository with #include "doctest.h"
instead of with <>
. Also compiler version would be very helpful. Thanks!
Can you provide a minimal reproducible example that includes both frameworks and has this problem? From what I can tell this is not the
argh_tests.cpp
source file in this repository because it seems to be including thedoctest.h
version that is in this repository with#include "doctest.h"
instead of with<>
.
Yes, you are right. That's been patched to use doctest.h from our respositories and the version for doctest in debian is 2.4.8 while the embedded one in the argh repository is still sitting at an outdated version. This is why it fails.
Easy way to reproduce this would be if you copy the doctest.h
file for 2.4.8 or any later version in the argh repository and then trigger the test.
Also compiler version would be very helpful.
We are on gcc-12 hope that helps.
Please try dropping in the latest version 2.4.8 instead.
If all runs well - make a PR.
Let me know if you have trouble with it and I'll do it. Just a bit busy lately.
Please try dropping in the latest version 2.4.8 instead. If all runs well - make a PR.
Not sure if you mean somthing else, but does not run well with the latest version, and that's what this bug report/issue is about. The failing logs are here as posted initially as well.
This is indeed a problem with doctest and it was introduced in version 2.4.7 (clang has no problems but all versions of gcc give an error) - this sample reproduces it: https://godbolt.org/z/5Y87aMW8W
I'll take a look
As per @onqtam 's suggestion, I pushed version 2.4.6. @tillea LMK if the problem is fixed.
@adishavit they are not using the version of doctest that's in this repository - instead, they are using the latest published doctest version (not sure why) which has introduced this bug. Still not fixed on my end.
Logged it on my end - watch this issue if anyone is interested: https://github.com/doctest/doctest/issues/630
Thanks @onqtam. I understand. I a closing this issue for now, as it is no longer an argh issue. Thanks everyone for investigating.
On Sun, 20 Mar, 2022, 9:23 pm Adi Shavit, @.***> wrote:
Please try dropping in the latest version 2.4.8 https://github.com/doctest/doctest/blob/master/doctest/doctest.h instead. If all runs well - make a PR.
Not sure if you mean somthing else, but does not run well with the latest version, and that's what this bug report/issue is about. The failing logs are here as posted initially as well.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1005500
@nileshpatra I don't have much time to look at this again but it seems that the issue was fixed not in version 2.4.8 of doctest but in version 2.4.9 - use this godbolt link to test different version (can confirm that with 2.4.9 or the latest master it works properly):
according to the changelog this issue (also logged in doctest as issue introduced in 2.4.7
: https://github.com/doctest/doctest/issues/630) was fixed in v2.4.9
and not in v2.4.8
:
https://github.com/doctest/doctest/blob/master/CHANGELOG.md#v249-2022-06-18
Hi, the Debian packaged version of argh received a bug report which says:
Any idea how to fix this? Kind regards, Andreas.