fix: method call in console_scripts

Makefile

@@ -8,7 +8,7 @@ install:
 uninstall:
 	python3 -m pip uninstall abcddb2vcard
 	rm -rf ./*.egg-info/
-	-rm -i "$$(which abcddb2vcard)" "$$(which vcard2img)"
+	@-rm -i "$$(which abcddb2vcard)" "$$(which vcard2img)"
 
 dist: setup.py abcddb2vcard/*
 	[ -z "$${VIRTUAL_ENV}" ]  # you can not do this inside a virtual environment.

abcddb2vcard/__init__.py

@@ -2,6 +2,6 @@
 '''
 Convert AddressBook database (.abcddb) to Contacts VCards file (.vcf)
 '''
-__version__ = '1.0.0'
+__version__ = '1.0.1'
 
 from .ABCDDB import ABCDDB

abcddb2vcard/__main__.py

@@ -1,2 +1,4 @@
 #!/usr/bin/env python3
-from . import abcddb2vcard  # execute sub-module code
+from .abcddb2vcard import main
+
+main()

abcddb2vcard/abcddb2vcard.py

@@ -11,38 +11,41 @@ try:
 except ImportError:  # fallback if not run as module
     from ABCDDB import ABCDDB  # type: ignore[import, no-redef]
 
-
 DB_FILE = str(Path.home().joinpath(
     'Library', 'Application Support', 'AddressBook', 'AddressBook-v22.abcddb'))
 
-cli = ArgumentParser(description=__doc__)
+
-cli.add_argument('output', type=str, metavar='outfile.vcf',
+def main() -> None:
+    cli = ArgumentParser(description=__doc__)
+    cli.add_argument('output', type=str, metavar='outfile.vcf',
                      help='VCard output file.')
-cli.add_argument('-f', '--force', action='store_true',
+    cli.add_argument('-f', '--force', action='store_true',
                      help='Overwrite existing output file.')
-cli.add_argument('-i', '--input', type=str, metavar='AddressBook.abcddb',
+    cli.add_argument('-i', '--input', type=str, metavar='AddressBook.abcddb',
                      default=DB_FILE, help='Specify another abcddb input file.'
                      ' Default: ' + DB_FILE)
-args = cli.parse_args()
+    args = cli.parse_args()
 
-# check input args
+    # check input args
-if not os.path.isfile(args.input):
+    if not os.path.isfile(args.input):
         print('AddressBook "{}" does not exist.'.format(args.input),
               file=sys.stderr)
         exit(1)
-elif not os.path.isdir(os.path.dirname(args.output) or os.curdir):
+    elif not os.path.isdir(os.path.dirname(args.output) or os.curdir):
         print('Output parent directory does not exist.', file=sys.stderr)
         exit(1)
-elif os.path.isfile(args.output) and not args.force:
+    elif os.path.isfile(args.output) and not args.force:
         print('Output file already exist. Use -f to force overwrite.',
               file=sys.stderr)
         exit(1)
 
-# perform export
+    # perform export
-contacts = ABCDDB.load(args.input)
+    contacts = ABCDDB.load(args.input)
-with open(args.output, 'w') as f:
+    with open(args.output, 'w') as f:
         for rec in contacts:
             f.write(rec.makeVCard())
         print(len(contacts), 'contacts.')
 
-exit()
+
+if __name__ == '__main__':
+    main()

abcddb2vcard/vcard2img.py

@@ -7,26 +7,28 @@ import sys
 from base64 import b64decode
 from argparse import ArgumentParser, FileType
 
-cli = ArgumentParser(description=__doc__)
-cli.add_argument('input', type=FileType('r'), metavar='infile.vcf',
-                 help='VCard input file.')
-cli.add_argument('outdir', type=str, help='Output directory.')
-args = cli.parse_args()
 
-# check input args
+def main() -> None:
-if not os.path.isdir(os.path.dirname(args.outdir) or os.curdir):
+    cli = ArgumentParser(description=__doc__)
+    cli.add_argument('input', type=FileType('r'), metavar='infile.vcf',
+                     help='VCard input file.')
+    cli.add_argument('outdir', type=str, help='Output directory.')
+    args = cli.parse_args()
+
+    # check input args
+    if not os.path.isdir(os.path.dirname(args.outdir) or os.curdir):
         print('Output parent directory does not exist.', file=sys.stderr)
         exit(1)
 
-os.makedirs(args.outdir, exist_ok=True)
+    os.makedirs(args.outdir, exist_ok=True)
 
-# perform export
+    # perform export
-c1 = 0
+    c1 = 0
-c2 = 0
+    c2 = 0
-name = ''
+    name = ''
-img = ''
+    img = ''
-collect = False
+    collect = False
-for line in args.input.readlines():
+    for line in args.input.readlines():
         line = line.rstrip()
         if line == 'BEGIN:VCARD':
             c1 += 1
@@ -50,5 +52,8 @@ for line in args.input.readlines():
             with open(os.path.join(args.outdir, name + '.jpg'), 'wb') as fw:
                 fw.write(b64decode(img))
 
-print(c1, 'contacts.', c2, 'images.')
+    print(c1, 'contacts.', c2, 'images.')
-exit()
+
+
+if __name__ == '__main__':
+    main()

setup.py

@@ -15,8 +15,8 @@ setup(
     packages=['abcddb2vcard'],
     entry_points={
         'console_scripts': [
-            'abcddb2vcard=abcddb2vcard.abcddb2vcard',
+            'abcddb2vcard=abcddb2vcard.abcddb2vcard:main',
-            'vcard2img=abcddb2vcard.vcard2img',
+            'vcard2img=abcddb2vcard.vcard2img:main',
         ]
     },
     long_description_content_type="text/markdown",