commit 27ec6d31edba7790a2cd0dd68cbf6be4709d3630
Merge: 172aeaa ca720b0
Author: Syed Shahrukh Hussain <syed.shahrukh@ossrevival.org>
Date: Sun Sep 6 07:04:24 2026 +0500
Merge branch 'tai/pr-resolve-autopkgtest-rtn4' into 'master'
d/patches: fix template-id constructor names for C++20
See merge request science-team/palabos!4
commit ca720b0fd58f81f27d450ea1c80e44ac94983ac3
Author: Tai Ho (tai271828) <e@tai271828.me>
Date: Sat Sep 5 17:21:31 2026 +0200
d/patches: fix template-id constructor names for C++20
GCC 16 became the default compiler in forky and sid with gcc-defaults
1.233, and its default language mode is -std=gnu++20. That enables
-Wtemplate-id-cdtor, which fires three times on the installed header
gridRefinement/gridRefinementFunctional3D.hh:
gridRefinementFunctional3D.hh:72:24: warning: template-id not allowed
for constructor in C++20 [-Wtemplate-id-cdtor]
72 | Xinterpolation2D<T>() : DirectedInterpolation2D<T>()
and identically at :306 (Yinterpolation2D) and :537 (Zinterpolation2D).
Naming a constructor with a template-id is deprecated since C++17 and
ill-formed in C++20.
The header is reached from palabos3D.hh, so every consumer of
libplb-dev sees the warnings. It also made buildTest1-3 fail on
ci.debian.net: the tests compiled, linked and ran to completion, but
debian/tests/control does not permit output on stderr.
Only the constructor name changes; the base-class initialiser keeps its
template argument. The result is valid in every C++ standard, so the
library's own -std=c++11 build is unaffected. Verified in a forky
chroot with g++ 16.2.0: stderr from all three tests drops to zero bytes.
Among the 6 debian patches available in version 2.3.0+repack1-2 of the package, we noticed the following issues: