Respect XDG_CACHE_HOME directive

We assumed users do not override default directory, now we honour
environment setting
Create directory for log before trying to use it

We used to call basicConfig with assumption that SESSION_LOG parent
directory already existed. However this is not necessarily true so now
we create directory tree in advance
Fix confusing typo in error message (nobuild->no-build)

Make the error message output consistent with actualy commandline
option not to confuse users
Fix backtrace when a ruby package has no %check section

Testing another fix on https://bugzilla.redhat.com/show_bug.cgi?id=839064
showed the following backtrace :

Exception down the road...
Traceback (most recent call last):
  File "/home/misc/checkout/git/FedoraReview/src/FedoraReview/review_helper.py", line 133, in run
    self.__do_report()
  File "/home/misc/checkout/git/FedoraReview/src/FedoraReview/review_helper.py", line 79, in __do_report
    self.__run_checks(self.bug.spec_file, self.bug.srpm_file)
  File "/home/misc/checkout/git/FedoraReview/src/FedoraReview/review_helper.py", line 105, in __run_checks
    writedown=not Settings.no_report)
  File "/home/misc/checkout/git/FedoraReview/src/FedoraReview/checks_class.py", line 185, in run_checks
    test.run()
  File "/home/misc/checkout/git/FedoraReview/src/FedoraReview/checks/ruby.py", line 113, in run
    for line in self.spec.get_section('%check')['%check']:
add a function is_archive to know if a source is a tarball or not
Add a check for source tarball, to see if there is proper comments

The heuristic used is the following, if the source is in a tarball,
the chance to have been handwritten is almost null. If that's not a
tarball, then it was likely added by the packager ( initscripts,
desktop file, etc ). That's not perfect, but better than nothing.
Catch IOErrors when working with external plugins

This will prevent us from throwing backtraces all over the place when
weird things happen
Use proper way to get mock build directory

srpm.get_build_dir() should be decomissioned
Fix last section not being added to section list

This meant that last section (usually changelog) was not available and
get_section would return nothing
* use hashlib module instead of running md5sum CLI tool to compute checkum * allow user to select checksum algorithm used with new option -k/--checksum (currently allowed: md5, sha1) * some refactoring in methods names to reflect that change (_md5 => _checksum) * tests updated
add unit tests for new -k/--checksum option
update man page to reflect new command-line option --checksum
Update version string to 0.2.1
Adding koji-download-script

To use koji scratch builds, script is neede to download the results.
This one is picked up from a personal homepåage, and might need separate
packaging. For now, entered into f-r to explore this way to handle things.
Document use of koji scratch builds in manpage
CheckLicensInDoc: use rpm -qldp instead of %doc (issue #91).

Old code compares license files in source with contents in %doc
and %_docdir. This is unreliable e. g.,  when a directory is listed
there is no way to find out the files included.

New code compares license files with output from rpm -qldp which gives
the complete list of files and thus works also when dirs are listed.
Improve error message when mock group test fails (issue 78).
Crash using nested mock config files (issue 89, patch by hguemar)
Improve 'not in mock group' error message

This has caused confusion for few new users that tried to use f-r
right after adding themselves to mock group

See https://fedorahosted.org/FedoraReview/ticket/78 for discussion
about handling this situation in different ways and why this simple
approach was chosen
Change 2 checks from MUST to SHOULD

Guidelines are suggesting this change, but they do not mandate it. We
should reflect this correctly