aboutsummaryrefslogtreecommitdiffstats
path: root/CLAUDE.md
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-05-18 10:49:06 +0200
committerDanilo M. <danix@danix.xyz>2026-05-18 10:49:06 +0200
commit9f1ad23a187a493ffa812064e6641061553e6386 (patch)
treea164278161830d2f168f830f81fbafbb07670f9e /CLAUDE.md
parente647c20e9f387b250469cda2b515cf767d2955a8 (diff)
downloadmkhintfile-9f1ad23a187a493ffa812064e6641061553e6386.tar.gz
mkhintfile-9f1ad23a187a493ffa812064e6641061553e6386.zip
feat: multiline DOWNLOAD support, test suite, updated docsHEADmaster
- Add parse_multiline_var, prompt_continuation_urls, build_multiline_value, _process_download_var; refactor update_checksums to handle multi-URL vars - First URL always re-downloaded; continuation URLs prompt user, skip re-download if unchanged - Add tests/mkhint_test.sh: 44 cases covering all commands and edge cases - Update README and CLAUDE.md: correct -N semantics, --new behavior, multiline DOWNLOAD flow, test suite docs
Diffstat (limited to 'CLAUDE.md')
-rw-r--r--CLAUDE.md39
1 files changed, 34 insertions, 5 deletions
diff --git a/CLAUDE.md b/CLAUDE.md
index 61f8e89..e464d97 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -26,23 +26,52 @@ bash mkhint --help
bash mkhint --list
bash mkhint --version 2.0.1 --hintfile mypackage
bash mkhint --version 1.2.3 --new mypackage
-bash mkhint --new mypackage # empty hint, no version
-bash mkhint --version 2.0.1 --hintfile mypackage --no-dl # skip downloads, add NODOWNLOAD=yes
+bash mkhint --new mypackage # keep VERSION from .info, no downloads
+bash mkhint --version 2.0.1 --hintfile mypackage --no-dl # downloads + md5 + NODOWNLOAD=yes
bash mkhint --new mypackage --no-dl # create hint with NODOWNLOAD=yes
bash mkhint --delete mypackage
bash mkhint --clean
```
-No test suite exists. Test manually against real `REPO_DIR`/`HINT_DIR` or with mock directories.
+### Automated test suite
+
+Uses mock `REPO_DIR`/`HINT_DIR` and a fake `wget` — no real downloads, no real directories needed. Requires `/var/lib/sbopkg/SBo-git/` for `.info` file fixtures (read-only).
+
+```bash
+bash tests/mkhint_test.sh
+```
+
+Test coverage:
+
+| ID | Scenario |
+|-----|----------|
+| T1 | `--new` from `.info`, no version — template copied, VERSION kept from .info |
+| T2 | `--new -v` — version updated, md5 recalculated |
+| T3 | `--new -N` no version — NODOWNLOAD=yes added, no downloads |
+| T4 | `--new -v -N` — version + md5 + NODOWNLOAD=yes |
+| T5 | `--new` when hint exists — backup + empty skeleton |
+| T6 | `--hintfile -v` single URL — version + md5 updated, backup created |
+| T7 | `--hintfile -v -N` — version + md5 + NODOWNLOAD=yes |
+| T8 | `DOWNLOAD=UNSUPPORTED`, `DOWNLOAD_x86_64` has URL — 32-bit skipped, x86_64 md5 recalculated |
+| T9 | `-N` alone — exits 1 |
+| T10 | `--hintfile` on nonexistent file — exits 2 |
+| T11 | `--delete` — removes hint and .bak |
+| T12 | `--delete` nonexistent — exits 2 |
+| T13 | `--new` multiline `.info`, no version — template copied, original md5s kept |
+| T14 | `--new` multiline `.info` with `-v` — first URL+md5 updated, continuation md5 kept |
+| T15 | `--clean` — removes all .bak files |
+
+When adding new features, add a corresponding test case to `tests/mkhint_test.sh`.
## Key Behaviors
- `--hintfile` update: backs up to `.bak`, replaces old version string globally via `sed`, re-downloads both URLs to recalculate MD5 checksums. Skips download if value is `UNSUPPORTED` or `UNTESTED`.
-- `--new` with existing `.info`: copies `.info` as template, strips `PRGNAM`, `HOMEPAGE`, `MAINTAINER`, `EMAIL`, comments out `REQUIRES`, sets `ARCH="x86_64"`.
+- `--new` with existing `.info`: copies `.info` as template, strips `PRGNAM`, `HOMEPAGE`, `MAINTAINER`, `EMAIL`, comments out `REQUIRES`, sets `ARCH="x86_64"`. Keeps `VERSION` from `.info`. If `-v` given, updates version string and recalculates checksums.
- `--new` when hint already exists: backs up old, creates empty skeleton.
-- `--no-dl` / `-N`: skips all downloads; inserts `NODOWNLOAD=yes` after `MD5SUM_x86_64=` line. Works with `--hintfile` or `--new`. Error if used alone.
+- `--no-dl` / `-N`: downloads and recalculates checksums as normal, then appends `NODOWNLOAD=yes` after `MD5SUM_x86_64=`. Works with `--hintfile` or `--new`. Error if used alone.
- `--delete` / `-d`: removes hint file and `.bak` if present. Accepts multiple package names. Exits 2 on first missing file.
- Downloads go to `/tmp/mkhint/download` (single shared temp file, deleted after md5 calculation).
+- Multiline `DOWNLOAD`/`DOWNLOAD_x86_64`: parsed via `parse_multiline_var` (awk). First URL always re-downloaded. Continuation URLs (2+) prompt user interactively — changed URLs re-downloaded, unchanged URLs keep existing md5. Written back via `perl -i` with `\` continuation format preserved. Shared logic in `update_checksums` → `_process_download_var`.
## Exit Codes