dput/TODO.code-style

To Do list for improving code idioms
####################################

To Do
=====

* Remove all use of global variables.

* Reduce ‘if __name__ == '__main__'’ block to minimum.

* Use idiomatic ‘configparser’ processing.

  * Remove direct query to ‘DEFAULT’ config section.

* Use ‘str.format’ for all string formatting and interpolation.

  * Remove usage of ‘%’ formatting operator.

* Use ‘argparse’ for command-line parsing.

  * Remove usage of ‘getopt’.

* Use ‘logging’ module throughout for all messages.

  * Remove usage of ‘print’ for debug-level messages.
  * Remove usage of ‘print’ for Informational messages.
  * Remove usage of ‘print’ for warning message.
  * Remove usage of ‘print’ for error messages.
  * Remove usage of ‘print’ for critical error messages.

* Use ‘sys.stdout.write’ for all normal output.

  * Remove usage of ‘print’ for normal output.

* Use ‘subprocess’ API for all subprocess interaction.

  * Remove usage of ‘os.system’.
  * Remove usage of ‘os.spawnv’.
  * Remove usage of ‘os.popen’.
  * Remove usage of ‘os.waitpid’.

* Use a Python native SSH library (e.g. Paramiko or Spur).

  * Remove usage of subprocess for SSH.

* Use a Python native GnuPG library (e.g. ‘python-gnupg’).

  * Remove usage of subprocess for GnuPG.


Done
====

* Remove all use of global variables.

  * dput.dcut.progname
  * dput.dcut.version
  * dput.dcut.USAGE
  * dput.dput.dput_version
  * dput.dput.files_to_remove
  * dput.dput.USAGE

* Use ‘subprocess’ API for all subprocess interaction.

  * Remove usage of ‘os.popen3’.


..
    Local variables:
    coding: utf-8
    mode: text
    mode: rst
    End:
    vim: fileencoding=utf-8 filetype=rst :