aboutsummaryrefslogtreecommitdiff
path: root/176
diff options
context:
space:
mode:
authorRodrigo Arias Mallo <rodarima@gmail.com>2025-09-28 20:26:15 +0200
committerRodrigo <rodarima@gmail.com>2025-09-28 23:10:55 +0200
commitfb510ea86be5ceb9e91573890242581fdbd77ad8 (patch)
treed819fe40683592008d136727f5a0b03e48dc1164 /176
Initial versionHEADmain
Diffstat (limited to '176')
-rw-r--r--176/index.md57
1 files changed, 57 insertions, 0 deletions
diff --git a/176/index.md b/176/index.md
new file mode 100644
index 0000000..7fd49dc
--- /dev/null
+++ b/176/index.md
@@ -0,0 +1,57 @@
+Title: tests: add HTML test items to EXTRA_DIST
+Author: Kangie
+Created: Fri, 31 May 2024 23:36:00 +0000
+State: closed
+
+This will enable tests to be easily run by downstream distributions.
+
+These files are not included in the 3.1.0 release tarball, resulting in errors like:
+
+```
+make[4]: Entering directory '/var/tmp/portage/www-client/dillo-3.1.0/work/dillo-3.1.0/test/html'
+make[4]: *** No rule to make target 'render/b-div.html', needed by 'render/b-div.html.log'. Stop
+```
+
+When distributions try to package and test the software.
+
+--%--
+From: rodarima
+Date: Sat, 01 Jun 2024 11:16:46 +0000
+
+Thanks! I was not planning on adding the rendering tests in the release, as they may increase a lot the tarball size, but as we don't have many tests so far, the size increase is not too big:
+```
+% ls -l releases/dillo-3.1.0.tar.gz git/release/dillo-3.1.0.tar.gz
+-rw-r--r-- 1 ram ram 1275663 Jun 1 11:43 git/release/dillo-3.1.0.tar.gz
+-rw-r--r-- 1 ram ram 1198855 Jun 1 11:42 releases/dillo-3.1.0.tar.gz
+```
+So I will add them for now, and if we see the tarball size gets too big we may need to create a minimal release tarball without them (still fits in a floppy disk).
+
+--%--
+From: rodarima
+Date: Sat, 01 Jun 2024 11:38:37 +0000
+
+Hmm, I tested you patch, but it seems to be still failing. The tests are included in the tarball, but they are not found by make check. I'll try to add a test to the CI so it fails there too, as `make distcheck` doesn't enable the tests by default.
+
+--%--
+From: rodarima
+Date: Sat, 01 Jun 2024 11:48:22 +0000
+
+Tried to push my CI commit into your branch but GitHub closed the PR when I accidentally pushed our master branch, so I cannot update it now. I'll create another PR using this branch: https://github.com/dillo-browser/dillo/tree/make-distcheck-html-tests.
+
+--%--
+From: Kangie
+Date: Sun, 02 Jun 2024 23:05:08 +0000
+
+> Thanks! I was not planning on adding the rendering tests in the release, as they may increase a lot the tarball size, but as we don't have many tests so far, the size increase is not too big:
+>
+> ```
+> % ls -l releases/dillo-3.1.0.tar.gz git/release/dillo-3.1.0.tar.gz
+> -rw-r--r-- 1 ram ram 1275663 Jun 1 11:43 git/release/dillo-3.1.0.tar.gz
+> -rw-r--r-- 1 ram ram 1198855 Jun 1 11:42 releases/dillo-3.1.0.tar.gz
+> ```
+>
+> So I will add them for now, and if we see the tarball size gets too big we may need to create a minimal release tarball without them (still fits in a floppy disk).
+
+Thanks for merging!
+
+Consider switching to xz for compression, it's widely available and will give you a little more breathing room with that hard limit. :) \ No newline at end of file