Re: [devel] Обновление protobuf

[Paul Wolneykien <manowar@altlinux.org>]

В Thu, 13 Feb 2025 10:35:03 +0300
Paul Wolneykien <manowar@altlinux.org> пишет:

> В Thu, 13 Feb 2025 07:19:19 +0300
> Anton Farygin <rider@basealt.ru> пишет:
> 
> > On 13.02.2025 00:02, Paul Wolneykien wrote:  
> > >> Так пересобрал бы локально для теста.    
> > >    Или может быть правильно было бы действовать так:
> > >
> > >    1. добиться сборки нового protobuf*без других пакетов*;
> > >    2. расшарить задание и пригласить дополнять его зависимыми
> > >       пакетами, предварительно проверяя и починяя их?
> > >
> > >    Не в этом ли состояла идея shared tasks, когда их придумали?
> > >
> > >    В целом ведь не важно, кто именно добавит в задание очеред    
> > 
> > Для этого надо понять что чинить.
> > 
> > Можно локально сделать простой цикл по всем зависящим пакетам и 
> > посмотреть кто не соберётся.  
> 
>   Хорошо, сделаем. Но очевидный минус такого подхода в том, что нельзя
> вместе обсудить ошибки сборки. Именно по этой причине я сначала добавил
> в задание все зависимые пакеты. Но, конечно, это тоже не сработало ---
> сборка не пошла дальше первого сломавшегося.

  Так, вчера многие пакеты собрались, репозиторий сформировался:

    https://packages.altlinux.org/ru/tasks/372112/
    https://git.altlinux.org/repo/372112/

  Судя по всему, для сборки некоторых пакетов для начала требуется
обновить стандарт C++, по которому они собираются:

C++14: clementine, mosh, osmpbf, purple-whatsapp
C++17: mumble, osgearth

  Остальные требуют анализа:

android-tools:
/usr/include/absl/strings/internal/str_format/output.h:75:8: error: no member named '___xxx_write' in 'std::basic_ostream<char>'
   75 |   out->write(s.data(), static_cast<std::streamsize>(s.size()));
      |   ~~~  ^
packages/modules/adb/sysdeps.h:549:17: note: expanded from macro 'write'
  549 | #define  write  ___xxx_write
      |                 ^

apache-orc:
/usr/src/RPM/BUILD/apache-orc-1.9.1/c++/src/ColumnReader.cc:272:21: error: possibly dangling reference to a temporary [-Werror=dangling-reference]
  272 |     const Timezone& writerTimezone;
      |                     ^~~~~~~~~~~~~~

arrow:
from /usr/src/RPM/BUILD/arrow-12.0.0/cpp/x86_64-alt-linux/src/arrow/flight/CMakeFiles/arrow_flight_objlib.dir/Unity/unity_2_cxx.cxx:4:
/usr/include/grpcpp/impl/codegen/config_protobuf.h:93:9: error: ‘Status’ in namespace ‘google::protobuf::util’ does not name a type
   93 | typedef GRPC_CUSTOM_UTIL_STATUS Status;
      |         ^~~~~~~~~~~~~~~~~~~~~~~

bear:
/usr/include/grpcpp/impl/codegen/config_protobuf.h:93:9: error: 'Status' in namespace 'google::protobuf::util' does not name a type
   93 | typedef GRPC_CUSTOM_UTIL_STATUS Status;
      |         ^~~~~~~~~~~~~~~~~~~~~~~

clementine:
from /usr/src/RPM/BUILD/clementine-1.4.0/x86_64-alt-linux/ext/libclementine-tagreader/tagreadermessages.pb.cc:4:
/usr/include/absl/base/policy_checks.h:79:2: error: #error "C++ versions less than C++14 are not supported."
   79 | #error "C++ versions less than C++14 are not supported."
      |  ^~~~~
/usr/include/google/protobuf/port_def.inc:159:15: error: static assertion failed: Protobuf only supports C++14 and newer.

gprc:
/usr/src/RPM/BUILD/grpc-1.53.0/src/core/lib/iomgr/event_engine_shims/tcp_client.cc:74:23: error: 'StrCat' is not a member of 'absl'
   74 |                 absl::StrCat("tcp-client:", addr_uri.value()))
      |                       ^~~~~~

mosh:
from conftest.cpp:73:
/usr/include/absl/base/policy_checks.h:79:2: error: #error "C++ versions less than C++14 are not supported."
   79 | #error "C++ versions less than C++14 are not supported."
      |  ^~~~~
/usr/include/google/protobuf/port_def.inc:159:15: error: static assertion failed: Protobuf only supports C++14 and newer.

mumble:
from /usr/src/RPM/BUILD/mumble-1.4.287/x86_64-alt-linux/src/Mumble.pb.cc:4:
/usr/include/absl/strings/string_view.h:53:26: error: 'string_view' in namespace 'std' does not name a type
   53 | using string_view = std::string_view;
      |                          ^~~~~~~~~~~
/usr/include/absl/strings/string_view.h:53:21: note: 'std::string_view' is only available from C++17 onwards

osgearth:
from /usr/src/RPM/BUILD/osgearth-3.7/src/osgEarth/MapboxGLGlyphManager.cpp:28:
/usr/include/absl/strings/string_view.h:53:26: error: 'string_view' in namespace 'std' does not name a type
   53 | using string_view = std::string_view;
      |                          ^~~~~~~~~~~

osmpbf:
from /usr/src/RPM/BUILD/OSM-binary-1.5.0/x86_64-alt-linux/osmpbf/fileformat.pb.cc:4:
/usr/include/absl/base/policy_checks.h:79:2: error: #error "C++ versions less than C++14 are not supported."
   79 | #error "C++ versions less than C++14 are not supported."
      |  ^~~~~
/usr/include/google/protobuf/port_def.inc:159:15: error: static assertion failed: Protobuf only supports C++14 and newer.

purple-whatsapp:
from identitykeypair.cpp:3:
/usr/include/absl/base/policy_checks.h:79:2: error: #error "C++ versions less than C++14 are not supported."
   79 | #error "C++ versions less than C++14 are not supported."
      |  ^~~~~
/usr/include/google/protobuf/port_def.inc:159:15: error: static assertion failed: Protobuf only supports C++14 and newer.

sysdig:
In file included from /usr/src/RPM/BUILD/sysdig-0.39.0/x86_64-alt-linux/libsinsp/cri-v1alpha2.grpc.pb.h:37:
In file included from /usr/include/grpcpp/impl/proto_utils.h:28:
/usr/include/grpcpp/impl/codegen/config_protobuf.h:93:9: error: no type named 'Status' in namespace 'google::protobuf::util'; did you mean simply 'Status'?
   93 | typedef GRPC_CUSTOM_UTIL_STATUS Status;
      |         ^~~~~~~~~~~~~~~~~~~~~~~
/usr/include/grpcpp/impl/codegen/config_protobuf.h:72:33: note: expanded from macro 'GRPC_CUSTOM_UTIL_STATUS'
   72 | #define GRPC_CUSTOM_UTIL_STATUS ::google::protobuf::util::Status
      |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~