#1028912 Please build with C++20 to enable multi-symbol detection support for DataMatrix

#1028912#5
Date:
2023-01-14 20:53:15 UTC
From:
To:
In d6dc409ef15ff7171c7f9130dff7266a7c65f053[1], upstream implemented
detection for multiple DataMatrix symbols in a single input. As
mentioned in the commit message, this needs C++20.

Patch attached.

[1] https://github.com/zxing-cpp/zxing-cpp/commit/d6dc409ef15ff7171c7f9130dff7266a7c65f053

#1028912#10
Date:
2023-01-16 00:37:40 UTC
From:
To:
在 2023-01-14星期六的 21:53 +0100,Jakob Haufe写道:

Looks okay. I am not sure about whether it will bring ABI breakage; given
the current transition freeze, I plan to enable it with zxing-cpp 2.0 upload
after Debian 12 release.

Thanks,
Boyuan Yang

#1028912#15
Date:
2023-01-20 13:52:13 UTC
From:
To:
Yes, you are right, it does break the ABI.
abi-compliance-checker report attached.

Cheers,
sur5r

#1028912#20
Date:
2023-09-14 09:35:17 UTC
From:
To:
Given the release is done, it would be great to have a newer zxing along
with a C++20 build. Are there any blockers? Do you need help/testers/...?

Cheers,
sur5r

#1028912#25
Date:
2024-02-23 07:44:00 UTC
From:
To:
With the arrival of 2.2.1-3 in testing, can this be considered fixed
or will the issue need to stay open until that reaches the next
'release'?

 - Axel

#1028912#30
Date:
2024-02-23 08:39:53 UTC
From:
To:
Control: fixed -1 2.1.0-3

It's fixed, yes.

Commit:

https://salsa.debian.org/debian/zxing-cpp/-/commit/5c51fce504e6a72a3356f589fd0294ca8990806b

Changelog entry:

zxing-cpp (2.1.0-3) experimental; urgency=medium

  * debian/rules:
    - Disable BUILD_UNIT_TESTS as suggested upstream. This flag should
      only be used in debug builds.
    - Use C++20 build to enable DataMatrix multi-symbol detection.
      (See #1028912)