[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1103390: RFS: quill/9.0.2-1 [ITP] -- Asynchronous Low Latency C Logging Library



Hi,

I've attached some suggested changes for your package, with explanations
for each below:

- Update d/copyright
  There are some missing copyright holders and files.

- d/watch: Use tarballs provided by GitHub
  It looks like your pristine-tar branch contains the GitHub-generated
  tarball, so debian/watch should download that as well rather than
  using git mode.

- d/control: Add <!nocheck> profile restriction to doctest-dev build-dep
  This means that if the nocheck profile is used (to skip tests during
  package build), then the doctest-dev build dependency will not be
  installed.

  TODO: -DQUILL_BUILD_TESTS should be OFF in d/rules when nocheck
  profile is enabled.

- d/rules: Remove unnecessary override_dh_auto_test
  There is no need to override this, the default will run all tests
  fine.

  TODO: With this change, there is one test that fails:

47: [doctest] doctest version is "2.4.11"
47: [doctest] run with "--help" for options
47: ===============================================================================
47: ./test/unit_tests/PatternFormatterTest.cpp:497:
47: TEST SUITE: PatternFormatter
47: TEST CASE:  pattern_formatter_source_location_prefix
47: 
47: ./test/unit_tests/PatternFormatterTest.cpp:534: FATAL ERROR: REQUIRE_EQ( formatted_string, expected_string ) is NOT correct!
47:   values: REQUIRE_EQ( cpp:504
47: , test/unit_tests/PatternFormatterTest.cpp:504
47:  )
47: 
47: ===============================================================================
47: [doctest] test cases: 1 | 0 passed | 1 failed | 15 skipped
47: [doctest] assertions: 1 | 0 passed | 1 failed |
47: [doctest] Status: FAILURE!

- Add autopkgtest suite
  For more information, see https://people.debian.org/~eriberto/README.package-tests.html
  This "provides as-installed testing of Debian packages" alongside
  tests run during package build.

If you would like your Salsa repo to be moved under the debian/ group,
add myself (@Maytha8) as the owner.

Once you've had a look and the TODOs are addressed, I will be happy to
upload this for you.

--
Maytham
From c22c0658580177a68e4be33ddd604d1db675d0f1 Mon Sep 17 00:00:00 2001
From: Maytham Alsudany <maytham@debian.org>
Date: Sat, 2 Aug 2025 12:33:08 +0800
Subject: [PATCH 5/5] Add autopkgtest suite

---
 debian/tests/CMakeLists.txt |  7 +++++++
 debian/tests/control        |  2 ++
 debian/tests/upstream-tests | 19 +++++++++++++++++++
 3 files changed, 28 insertions(+)
 create mode 100644 debian/tests/CMakeLists.txt
 create mode 100644 debian/tests/control
 create mode 100644 debian/tests/upstream-tests

diff --git a/debian/tests/CMakeLists.txt b/debian/tests/CMakeLists.txt
new file mode 100644
index 0000000..b29589a
--- /dev/null
+++ b/debian/tests/CMakeLists.txt
@@ -0,0 +1,7 @@
+cmake_minimum_required(VERSION 3.12...4.0)
+project(quill)
+
+set(CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/cmake" ${CMAKE_MODULE_PATH})
+include(QuillUtils)
+enable_testing()
+add_subdirectory(test)
diff --git a/debian/tests/control b/debian/tests/control
new file mode 100644
index 0000000..c93a8a5
--- /dev/null
+++ b/debian/tests/control
@@ -0,0 +1,2 @@
+Tests: upstream-tests
+Depends: @, cmake, doctest-dev
diff --git a/debian/tests/upstream-tests b/debian/tests/upstream-tests
new file mode 100644
index 0000000..81a6810
--- /dev/null
+++ b/debian/tests/upstream-tests
@@ -0,0 +1,19 @@
+#!/bin/sh
+
+set -eu
+
+mkdir -p "$AUTOPKGTEST_TMP"
+cp -r test "$AUTOPKGTEST_TMP"/test
+cp -r cmake "$AUTOPKGTEST_TMP"/cmake
+cp debian/tests/CMakeLists.txt "$AUTOPKGTEST_TMP"/CMakeLists.txt
+
+sed -i "s%target_link_libraries(\${TEST_NAME} quill)%%" \
+  "$AUTOPKGTEST_TMP"/test/integration_tests/CMakeLists.txt \
+  "$AUTOPKGTEST_TMP"/test/unit_tests/CMakeLists.txt
+
+mkdir -p "$AUTOPKGTEST_TMP"/build
+cd "$AUTOPKGTEST_TMP"/build
+
+cmake ..
+make
+make test
-- 
2.50.1

From d4e7a26c6251175d6bec7eebcbc313938c58bc73 Mon Sep 17 00:00:00 2001
From: Maytham Alsudany <maytham@debian.org>
Date: Sat, 2 Aug 2025 12:33:03 +0800
Subject: [PATCH 4/5] d/rules: Remove unnecessary override_dh_auto_test

---
 debian/rules | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/debian/rules b/debian/rules
index f8c8d14..3211551 100755
--- a/debian/rules
+++ b/debian/rules
@@ -5,9 +5,4 @@
 override_dh_auto_configure:
 	dh_auto_configure -- \
 	-DCMAKE_LIBRARY_PATH=$(DEB_HOST_MULTIARCH) \
-	-DQUILL_BUILD_TESTS=ON \
-
-override_dh_auto_test:
-ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
-	cd obj-*/test && ctest --build-config Release --output-on-failure -R ^multi_frontend_threads$
-endif
+	-DQUILL_BUILD_TESTS=ON
-- 
2.50.1

From 2ad4cede110f347901fa5abc9fdb076354bffe6a Mon Sep 17 00:00:00 2001
From: Maytham Alsudany <maytham@debian.org>
Date: Sat, 2 Aug 2025 12:32:46 +0800
Subject: [PATCH 3/5] d/control: Add <!nocheck> profile restriction to
 doctest-dev build-dep

---
 debian/control | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/control b/debian/control
index 926d6b1..237aafe 100644
--- a/debian/control
+++ b/debian/control
@@ -6,7 +6,7 @@ Rules-Requires-Root: no
 Build-Depends:
  debhelper-compat (= 13),
  cmake,
- doctest-dev,
+ doctest-dev <!nocheck>,
 Standards-Version: 4.7.2
 Homepage: https://github.com/odygrd/quill
 Vcs-Browser: https://salsa.debian.org/syed/quill
-- 
2.50.1

From 05379fccf547ca16640704770444f3025505d50f Mon Sep 17 00:00:00 2001
From: Maytham Alsudany <maytham@debian.org>
Date: Sat, 2 Aug 2025 12:32:27 +0800
Subject: [PATCH 2/5] d/watch: Use tarballs provided by GitHub

---
 debian/watch | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/debian/watch b/debian/watch
index 38b8be5..1af7f5f 100644
--- a/debian/watch
+++ b/debian/watch
@@ -1,4 +1,5 @@
 version=4
-opts="mode=git" https://github.com/odygrd/quill.git \
-    refs/tags/v([\d\.]+)
-
+opts="filenamemangle=s%(?:.*?)?v?(\d[\d.]*)\.tar\.gz%@PACKAGE@-$1.tar.gz%,\
+      uversionmangle=s/(\d)[_\.\-\+]?(RC|rc|pre|dev|beta|alpha)[.]?(\d*)$/$1~$2$3/" \
+  https://github.com/odygrd/quill/tags \
+  .*/v?(\d[\d\.]*)\.tar\.gz
-- 
2.50.1

From 6942ae3141af2db7e90b2f8a6d64f7c3a8f4a4f3 Mon Sep 17 00:00:00 2001
From: Maytham Alsudany <maytham@debian.org>
Date: Sat, 2 Aug 2025 12:32:05 +0800
Subject: [PATCH 1/5] Update d/copyright

---
 debian/copyright | 49 +++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 46 insertions(+), 3 deletions(-)

diff --git a/debian/copyright b/debian/copyright
index 2a0ead0..00dbdac 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -6,19 +6,62 @@ Upstream-Contact: https://github.com/odygrd/quill/issues
 Files:
  *
 Copyright:
- 2020-2025 Odysseas Georgoudis
+ 2020-2025 Odysseas Georgoudis and quill contributors
 License: Expat
 
 Files:
- debian/*
+ include/quill/bundled/fmt/*
 Copyright:
- 2025 Syed Shahrukh Hussain <syed.shahrukh@ossrevival.org>
+ 2012-2025 Victor Zverovich and fmt contributors
+License: Expat
+
+Files:
+ include/quill/bundled/fmt/chrono.h
+Copyright:
+ 2012-2025 Victor Zverovich and fmt contributors
+ 2019 Paul Dreik
 License: Expat
 
 Files: test/bundled/doctest/*
 Copyright: 2016-2019 Viktor Kirilov
 License: Expat
 
+Files:
+ include/quill/bundled/fmt/format.h
+Copyright:
+ 2012-2025 Victor Zverovich and fmt contributors
+License: Expat with fmt exception
+ Permission is hereby granted, free of charge, to any person obtaining a
+ copy of this software and associated documentation files (the "Software"),
+ to deal in the Software without restriction, including without limitation
+ the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ and/or sell copies of the Software, and to permit persons to whom the
+ Software is furnished to do so, subject to the following conditions:
+ .
+ The above copyright notice and this permission notice shall be included
+ in all copies or substantial portions of the Software.
+ .
+ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+ OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+ IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+ CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+ TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ .
+ --- Optional exception to the license ---
+ .
+ As an exception, if, as a result of your compiling your source code, portions
+ of this Software are embedded into a machine-executable object form of such
+ source code, you may redistribute such embedded portions in such object form
+ without including the above copyright and permission notices.
+
+Files:
+ debian/*
+Copyright:
+ 2025 Syed Shahrukh Hussain <syed.shahrukh@ossrevival.org>
+License: Expat
+
 License: Expat
  Permission is hereby granted, free of charge, to any person obtaining a
  copy of this software and associated documentation files (the "Software"),
-- 
2.50.1

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: