Disable self-check by default.

I'm going to disable self-test as a whole for the time being. It was a good idea in theory, but it's a bad user experience in the current state to get constant false positives.

It was a mistake that I left it on even a tenth this long. I think I had the vague idea that it would get me/others to fix zbarimg's reliability, but that logic was clearly wrong (and was not user-centric enough)

Author: za3k

docs/CHANGELOG

@@ -1,4 +1,5 @@
 (unreleased)
+- Disabled self-check by default, because there are too many false positives from zbarimg's flaky QR scanning from PDF
 - PDF output is more reproducible across Pillow versions
 - Added continuous integration
 - Fixed bugs

docs/MAN.txt

@@ -92,8 +92,7 @@ BACKUP OPTIONS
               Default: yes if erasure coding is enabled (default), no otherwise
 
        --skip-checks, --no-skip-checks
-              Skip erasure code  checks,  self-restore  checks.   Default:  not
-              skipped
+              Skip erasure code checks, self-restore checks.  Default: skipped
 
 RESTORE OPTIONS
        --code-count-erasure COUNT
@@ -106,11 +105,11 @@ RESTORE OPTIONS
               Force decompression (on/off).  Default: automatic
 
        --display, --no-display
-              For  webcam  scanning,  (display/don't display) a webcam preview.
+              For webcam scanning, (display/don't display)  a  webcam  preview.
               Default: display
 
        --encrypt PASSPHRASE, --no-encrypt
-              Force decryption (on/off) and give the passphrase if  decrypting.
+              Force  decryption (on/off) and give the passphrase if decrypting.
               Default: automatic
 
        --image-restore
@@ -120,15 +119,15 @@ RESTORE OPTIONS
               Set the restore file path.  Default: stdout
 
        --sha256 SHA256
-              Include  a  sha256sum to check the file.  Giving the initial part
+              Include a sha256sum to check the file.  Giving the  initial  part
               of the sha256sum does a partial check.  Default: no check, prints
               checksum to stderr
 
        --webcam-restore
               Force webcam-based restore.  Default: automatic
 
 EXIT VALUES
-       0      Success.  This includes certain self-test  failures  which  some-
+       0      Success.   This  includes  certain self-test failures which some-
               times fail by chance.
 
        1      Invalid command line option
@@ -148,30 +147,30 @@ EXIT VALUES
 BUGS
        Please report any bugs at https://github.com/za3k/qr-backup/issues .
 
-       Sometimes  scanned  images,  including  the original PDF, cannot be read
+       Sometimes scanned images, including the original  PDF,  cannot  be  read
        successfully and the restore self-check fails.  This appears to be a bug
        in zbar.
 
-       On Debian, Ubuntu and certain other systems, the default  system  image-
+       On  Debian,  Ubuntu and certain other systems, the default system image-
        magick policy prevents reading or writing PDFs.  Since qr-backup depends
-       on  both,  it will fail.  qr-backup does its best to report this problem
+       on both, it will fail.  qr-backup does its best to report  this  problem
        clearly.
 
-       It is possible to print barcodes so large or small your webcam or  scan-
+       It  is possible to print barcodes so large or small your webcam or scan-
        ner cannot read them.  Make sure to test your restore process when using
        non-default settings.
 
 COPYRIGHT
-       qr-backup  was written by Zachary Vance, and is released into the public
-       domain and under  CC0  <https://creativecommons.org/share-your-work/pub-
+       qr-backup was written by Zachary Vance, and is released into the  public
+       domain  and  under CC0 <https://creativecommons.org/share-your-work/pub-
        lic-domain/cc0/>.  The generated PDF includes the DejaVu Sans Mono font,
-       which   has   a   separate  license  <https://dejavu-fonts.github.io/Li-
+       which  has  a   separate   license   <https://dejavu-fonts.github.io/Li-
        cense.html>.
 
-       The qr-backup source code is available  at  <https://github.com/za3k/qr-
+       The  qr-backup  source code is available at <https://github.com/za3k/qr-
        backup>.
 
 AUTHORS
        Zachary Vance.
 
-qr-backup 1.1.3                    March 2025                      QR-BACKUP(1)
+qr-backup 1.1.4                   August 2025                      QR-BACKUP(1)

docs/USAGE.md

@@ -77,7 +77,7 @@ Backup options:
         Default: yes if erasure coding is enabled (default), no otherwise
     --skip-checks, --no-skip-checks
         Skip erasure code checks, self-restore checks.
-        Default: not skipped
+        Default: skipped
 
 
 Restore options:

docs/qr-backup.1.man

@@ -1,6 +1,6 @@
-.\" Automatically generated by Pandoc 3.1.12.1
+.\" Automatically generated by Pandoc 3.1.12.3
 .\"
-.TH "QR\-BACKUP" "1" "March 2025" "qr\-backup 1.1.3" ""
+.TH "QR\-BACKUP" "1" "August 2025" "qr\-backup 1.1.4" ""
 .SH NAME
 qr\-backup \- back up file(s) to a series of QR codes on paper
 .SH SYNOPSIS
@@ -114,7 +114,7 @@ Default: yes if erasure coding is enabled (default), no otherwise
 .TP
 \f[B]\-\-skip\-checks\f[R], \f[B]\-\-no\-skip\-checks\f[R]
 Skip erasure code checks, self\-restore checks.
-Default: not skipped
+Default: skipped
 .SH RESTORE OPTIONS
 .TP
 \f[B]\-\-code\-count\-erasure \f[BI]COUNT\f[B]\f[R]

qr-backup

@@ -58,7 +58,7 @@ BACKUP_OPTIONS = [
     (("--qr-version VERSION",), "10", "Uses QR codes, version VERSION. Versions range from 1-40. The bigger the version, the harder to scan but the more data per code."),
     (("--scale SCALE",), "5px", "Scale QR codes so that each small square in the QR code is SCALE x SCALE pixels."),
     (("--shuffle", "--no-shuffle"), "yes if erasure coding is enabled (default), no otherwise", "Spread QR codes across pages. This can help prevent data loss."),
-    (("--skip-checks", "--no-skip-checks"), "not skipped", "Skip erasure code checks, self-restore checks."),
+    (("--skip-checks", "--no-skip-checks"), "skipped", "Skip erasure code checks, self-restore checks."),
 ]
 RESTORE_OPTIONS = [
     (("--code-count-erasure COUNT",), "automatic", "Specify the number of erasure QR codes."),
@@ -955,7 +955,7 @@ def main_backup(args):
     use_shuffle = None
     use_tar = False
     num_copies = 1
-    do_checks = True
+    do_checks = False # Disabled until zbar is less flaky (aka never)
     verbose = False
 
     # parse arguments