Arch Linux User Repository

Thanks, it looks good and clean now :-)

@patlefort @Martchus done, tell me if you need anything else.

While it is a good idea to add sodepends, please don't add versions to them. The version is automatically generated by makepkg - that's the whole point. These dependencies should also go preferably under package(). Checkout https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=syncthingtray-qt6 for an example.

I would also leave out the version in the other dependencies. Adding a specific version is only done exceptionally in Arch Linux packaging as it is normally a given that one updates all their dependencies consistently. Hence these specific versions often become out of date without anybody noticing and are then rather useless.

It's generally preferable to add them as runtime only, like:

package() {
    depends+=('libfmt.so' 'libspdlog.so')

You should omit the version, they will be pinned to the installed version automatically when installed. For example, the .so file depends on libspdlog.so.1.16, so any update to spdlog version would require kddockwidgets-qt6 be rebuilt always.

@patlefort done, is it ok for you just like this?

Can you please add libspdlog.so and libfmt.so in depends? It helps detecting needed rebuild.

My "ownstuff" repository contains a binary package of this (among many other AUR packages).

Looks like https://aur.chaotic.cx contains it, too. (They seem to have forgotten to rebuild hotspot, though.)

Can someone make a binary version of this package please?

@trosh are you building it in a clean chroot? because I do not get any error.

I'm getting the following build errors:

[  4%] Building CXX object src/CMakeFiles/kddockwidgets.dir/Config.cpp.o
In file included from /usr/include/fmt10/fmt/format.h:49,
                 from /usr/include/spdlog/fmt/fmt.h:29,
                 from /usr/include/spdlog/common.h:50,
                 from /usr/include/spdlog/spdlog.h:12,
                 from /home/trosh/.cache/yay/kddockwidgets-qt6/src/KDDockWidgets-2.2.5/src/core/spdlog_formatters_p.h:17,
                 from /home/trosh/.cache/yay/kddockwidgets-qt6/src/KDDockWidgets-2.2.5/src/core/Logging_p.h:26,
                 from /home/trosh/.cache/yay/kddockwidgets-qt6/src/KDDockWidgets-2.2.5/src/core/Utils_p.h:19,
                 from /home/trosh/.cache/yay/kddockwidgets-qt6/src/KDDockWidgets-2.2.5/src/Config.cpp:23:
/usr/include/fmt10/fmt/core.h:289:30: error: redefinition of ‘struct fmt::v11::is_contiguous<T>’
  289 | template <typename T> struct is_contiguous : std::false_type {};
      |                              ^~~~~~~~~~~~~
In file included from /usr/include/spdlog/fmt/fmt.h:28:
/usr/include/fmt/base.h:636:30: note: previous definition of ‘struct fmt::v11::is_contiguous<T>’
  636 | template <typename T> struct is_contiguous : std::false_type {};
      |                              ^~~~~~~~~~~~~
/usr/include/fmt10/fmt/core.h:293:8: error: redefinition of ‘struct fmt::v11::monostate’
  293 | struct monostate {
      |        ^~~~~~~~~
/usr/include/fmt/base.h:339:8: note: previous definition of ‘struct fmt::v11::monostate’
  339 | struct monostate {
      |        ^~~~~~~~~
/usr/include/fmt10/fmt/core.h:319:45: error: redefinition of ‘template<class ... T> constexpr void fmt::v11::detail::ignore_unused(const T& ...)’
  319 | template <typename... T> FMT_CONSTEXPR void ignore_unused(const T&...) {}
      |                                             ^~~~~~~~~~~~~
/usr/include/fmt/base.h:363:45: note: ‘template<class ... T> constexpr void fmt::v11::detail::ignore_unused(const T& ...)’ previously declared here
  363 | template <typename... T> FMT_CONSTEXPR void ignore_unused(const T&...) {}
      |                                             ^~~~~~~~~~~~~
/usr/include/fmt10/fmt/core.h:321:27: error: redefinition of ‘constexpr bool fmt::v11::detail::is_constant_evaluated(bool)’
  321 | constexpr FMT_INLINE auto is_constant_evaluated(
      |                           ^~~~~~~~~~~~~~~~~~~~~
/usr/include/fmt/base.h:365:16: note: ‘constexpr bool fmt::v11::detail::is_constant_evaluated(bool)’ previously defined here
  365 | constexpr auto is_constant_evaluated(bool default_value = false) noexcept
      |                ^~~~~~~~~~~~~~~~~~~~~
/usr/include/fmt10/fmt/core.h:340:49: error: redefinition of ‘template<class T> constexpr T fmt::v11::detail::const_check(T)’
  340 | template <typename T> constexpr FMT_INLINE auto const_check(T value) -> T {
      |                                                 ^~~~~~~~~~~
/usr/include/fmt/base.h:382:56: note: ‘template<class T> constexpr T fmt::v11::detail::const_check(T)’ previously declared here
  382 | template <typename T> FMT_ALWAYS_INLINE constexpr auto const_check(T val) -> T {
      |                                                        ^~~~~~~~~~~

… and a few thousand more lines of errors.

When I tried building straight from the current Git branch using the instructions from the manual I seem to get the same errors; but I'm not seeing anything about that on the Issues page on GitHub. Is it something about C++ standard version? Am I doing something wrong?

Thanks :-)