Temporary test fix.

Using openerp-client is a bad choice, it's still active. Updating to
last change for now.
SysV test replaced with systemd, closes #65
Remove useless --grab option, closing issue #64
Mock root bugfix

Old code used root as defined in /etc/mock/default.cfg. Of course.
Mock should respect the root set by mock_config, and use the root
defined in /etc/mock/whatever.cfg

Testcases involves rpmfusion code, avoiding to include these in
fedora package.
Adding overall "Complies to Packaging Guidelines" item

Most (all?) manual review templates have this item, which effectively
covers everything not stated elsewhere. I think also f-r needs this
item to avoid giving the impressiont f-r does more than it actually does.
- added python-BeautifulSoup to Requirements in README
Get filename from url itself instead using urlparse

Original code didn't work for cases like #834582. This could perhaps
have other issues, but should be generally more OK.
Fixing some user feedback

For each source without upstream, f-r outputs a message about the
missing upstream, and this being "a corner case not yet supported".
This is plain wrong, f-r will will never be able to create upstreams
for sources missing it.

Removing this output completely. There's already another message "No
upstream for whatever" displayed which gives the correct user feedback.

Fixing messages while downloading, correctly state when cache is used.
Fix so-files in non-devel package check.

Method comment is OK, stating that so-files must be in a -devel
package if they are symlinks to versioned so-files. However,
the implemenatation just tests if there are so-files in non-devel
packages(s), returning False if so. This is plain wrong, so-files
in non-devel packages are perfectly OK in many situations.

New test returns True if all so-files in devel-packages. Else, gives
a 'fix or explain' message for the so-files and return
'inconclusive'.
Don't check debuginfo for non-devel packages (#39)

Apply and fix patch from issue #39.
Fixing automatic test for Conflicts: tag (#40)

Adds automatic test for Conflicts tag according to patch and
discussion in issue #40.
Don't look for %buildroot and $RPM_BUILD_ROOT in changelog (#28)

Add optional parameter skip_changelog to SpecFile.find_all, blocking
false positives as described in #28.

The testcase in bug 770152 don't build on f16.
Only show "Using local rpms:..." message once.

Without this fix, the message above is repeated a lot of times adding
nothing but troubles.
Updating README and manpage

New contacts, python-requests not used, .... Manpage updated with
review directory layout section.
Add links to result and BUILD in review directory.

Users needs easy access to mock results and the sources (after %prep)
in order to make their own checks and reports.
Remove unused urlparse from helpers

Also cleanup imports a bit
Fixing make_release

Make it work even if invoked from another directory. Simplify
version.py parsing. Add -b  batch option, avoiding the upload
to fedorahosted query (testability). Issue a warning if there
are uncommitted changes. Fix warnings about unused metadata
description_long and missing metadata maintainer and
maintainer_email. Fix bug(s) blocking import from src/FedoraReview.
Adding test. Adding dist directory to repo
Add missing test "devel files in devel package".

For some reason this (and only this) test is missing in the template.
Adding. Here is a potential for support by looking for headers and
so-files, but for now this is just a manual test.
Move relocatable package test from cccp to generic.

Making a relocatable package could be done in any tool, not just a
C or C++. So the test should be generic.
Reviewing source URL.

Defining url limitations in manpage. Avoid using os.path.basename
on data which potentially isn't any path at all.