pq: do author guessing outside the apply_patch functions

Call the author parsing/guessing function outside the apply patch
functions. This way, the caller can decide when to do the guessing, and
with which parameters. Now the apply_patch functions do what their name
suggests.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

log: make color setting a tristate on/off/auto

Makes it possible to force color=on e.g. when piping output. Also, moves
all 'auto' logic to one single place, i.e. the streamhandler.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Document --git-pristine-tar-commit

Tristate: drop broken and unused is_valid_state() method

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Tristate: allow to init from another Tristate object

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Check the sha1 doesn't exceed 40 bytes

GitRepository/strip_sha1: fix length checking

Accept longer sha1 than what was asked for. The length option given to
git is merely a "wish to get a sha1 of this length". Git may also return
longer sha1 if truncating to given length would give
ambiguous/non-unique sha1.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

git-dch: Allow to create changelogs when not on a branch

This makes snapshot changelog generation e.g. within jenkins builds
simpler since jenkins by defaults checkouts out the commit without
creating a branch.

gbp-create-remote-repo: Set HEAD in remote repo

to debian branch

Closes: #692006

gbp.log: Improve error message on broken color lists

New configuration option for setting the output color scheme

Adds a new command line and config file option 'color-scheme' for
selecting the colors used in log output.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

log: add support for setting the color scheme

Implements changeable color schemes in the gbp.log module. Color scheme
is given as a colon-separated list, with one color for each log level:
'<debug_color>:<info_color>:<warning_colro>:<error_color>'. Colors can
be given as an integer number (ANSI terminal color code) or color name
(.e.g 'red').  Missing or empty fields are interpreted as using the
default color for that log level.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

gbp.log: lowercase the levelname

too match the output before the switch to Python's logging module

log: rewrite the module to use Python logging module

Replaces the Logger with a new one, derived from the Logger class of the
Python standard library. Colorized output is handled by a separate
handler class.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

gbp-pq: Improve patch header write out

Avoid trailing new lines in the patch header when reexporing patches.
to avoid diffs when nothing changed.

Ignore rope project files

as generated by ropemacs

Document changes and release 0.6.0~git20121124

git-import-dscs: Properly catch import errors

Closes: #694113

Remove unused imports and adjust formatting

Git-Dch: Ignore

gbp-pq: don't use plural form when there's only one try left

GitRepository/has_submodules: use correct .gitmodules file

Examine .gitmodules from the "root directory" of the repo, not the
current working directory. Fixes has_submodules() method, when called
from arbitrary cwd.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commit_info: support tags

Dereference the given revision to a commit. Fixes get_commit_info() when
called for a tag.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Test gbp.pq.common.write_patch

Output test results in xunit format too

GitRepository/get_submodules: use correct path

By default, run git in the repo path, not current cwd. Also, now returns
submodule paths without leading './'.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commits: more flexible revision ranges

Support getting the complete history of an arbitrary commit-ish
(since=None, until=COMMIT-ISH). Formerly this was only possible for the
current git HEAD. Now, get_commits(since=None, until='COMMIT')
translates to 'git log COMMIT'.

Also, for consistency, add support for getting the history from an
arbitrary commit until the current HEAD (since=COMMIT-ISH, until=None).

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

import-orig: remove dead GbpNothingImported exception

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

import-orig: remove dead if statement

GitRepository.commit_dir() never returns empty objects. It raises an
exception, instead.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Add sha1 stripping in a single place

so we can easily sanity check the result.

Test if setting a topic works

pq: Allow to pass in custom fucntion to fetch authorship information

so the RPM based tools don't need to rely on a control file but
can e.g. look at the spec file.

Test apply_and_commit_patch

Use an absolute  path when parsing the control file

so we con't accidentally parse a file from another dir.

testutils: create missing directories when adding a file

pq: fix "no authorship" warning messages

Add one missing string field. Show the patch filename instead of the
pull path - the user doesn't need to know if the patch was applied from
a temporary directory, for example.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

tests: make tests a Python module

Makes it possible to reuse code between various tests.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

notifications.py: remove unused import

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

command_wrappers: suppress some pylint warnings

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Add issing git- prefix to --color and --notify options

Thanks: Filippo Rusconi
Closes: #693978

Remove unused import

Minor git-import-orig manpage improvements

GbpError accepts an error message

so no need to print it separately

Allow to remove the orig tarball symlink

that's used make pristine-tar see the correct orig tarball name.

Closes: #692401

GitRepository/diff: catch git error

Raise an exception if the git command fails. Also, utilize _git_inout()
instead of the deprecated _git_getoutput().

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/diff: add 'paths' argument

Makes the diff function more versatile. I.e. makes diffing only certain
paths possible. Also, add basic unittests for the diff() method.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository.find_tag: change deprecated _git_getoutput to _git_inout

Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository: fix merge() for older git versions

Utilize _cmd_has_feature() in GitRepository.merge() to not give
edit/no-edit option for older versions of git-merge, that don't support
it. Fixes a regression (with git-version < 1.7.8) caused by commit
f3aa87fa0361a.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository: implement _cmd_has_feature() method

This method is intended for checking if the local git (client) command
supports a certain feature. The "feature" is considered to be a
command line option. E.g. does "merge" command have the "edit" feature
translates to does git-merge support the '--edit' command line option.

To figure this out, _cmd_has_feature() parses through the "OPTIONS"
section of the man page of the git command.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Fix  testsuite failure if DEB_VENDOR != Debian

The package fails to build from source if the DEB_VENDOR is not Debian,
as the test output from git-dch expects Debian distribution specific
behaviour.

You can see the failed build log with DEB_VENDOR=Ubuntu here:
https://launchpadlibrarian.net/121690052/buildlog_ubuntu-raring-i386.git-buildpackage_0.6.0~git20120822_FAILEDTOBUILD.txt.gz

Closes: #692289

Signed-off-by: Guido Günther <agx@sigxcpu.org>

Print number of tries left

Add support dch's --security option

PEP-8 and pyflakes cleanups

Add debian/source/format parser

Document changes and release 0.6.0~git20120822

GitArgs/add: support iterable and non-string args

Support giving iterables (other than basestring, e.g. list(s)) as an
argument to GitArgs.add(). Also, add support non-iterable arguments that
support the str() conversion.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository: Implement set_upstream_branch and get_upstream_branch methods

set_upstream_branch sets upstream branch for the local branch using
git branch --set-upstream
get_upstream_branch returns info about upstream branches

Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

manpages: correct manual section and remove duplicate entry

docs: make refentry ids of manpages more consistent

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

docs: fix cross-referencing in manpages

Docbook2man has to be run twice to make sure the cross-references are
correct. First pass is just to update the cross-reference file
(manpage.refs). The second the builds the actual manpages. Otherwise
forward-references might be broken.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/branch_contains: remove prefix '*' in branch name

The git output always prefixes the current branch name with '*'. Without
this fix branch_contains() always fails for the currently active branch.

Signed-off-by: Zhang Qiang <qiang.z.zhang@intel.com>

manpage: Fix typo in git-dch's multimain-merge option

Closes: #684322

dch: Fix error reporting on parsing errors

Document changes and release 0.6.0~git20120803

Improve error reporting from uscan

by parsing out the warnings and error fields from the dehs output.

Move uscan to separate class

Add tests for gbp.deb.{DpkgCompareVersions,DscFile}

gbp.command_wrappers: Reformat to 80 chars line length

Test help output by importing the modules

so we get correct coverage information for the scripts

Skip tests requiring devscripts if dch is not there

so tests don't fail on rpm based systems.

Test behavior of gbp.scripts.dch.main().

* tests/11_test_dch_main.py: Test common cases with "--release" and
  "--snapshot". Try 2 consecutive snapshots to check for merged entries.

Remove multiple spaces

GitModifier: add tests for dict interface

GitModifier: use __getitem__ to fetch date

instead of accessing __dict__ directly which only has _date.

GitRepository: raise GitRepositoryError on git errors

Raise GitRepositoryError in cases where CommandExecFailed (from
GitCommand) was previously silently passed forward.

Heavily based on a patch by Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commit_info: add file status

Add file status and name to the info returned by the get_commit_info()
method.

Signed-off-by: Zhang Qiang <qiang.z.zhang@intel.com>
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commit_info: add committer info

Add committer to the info returned by get_commit_info() method. Returns
committer name, email and timestamp as a GitModifier object.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commit_info: add author timestamp

Add author timestamps to the info returned by get_commit_info() method.

Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitModifier: More flexible date handling

Allow to pass in the date as datetime object, timestamp or git raw date.
and allow to retrieve these values. This make constructing GitModifiers
from python simpler.

Fix variable renaming

GitRepository: option to ignore untracked in is_clean()

Add an option to ignore untracked files when checking if the repository
is clean.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_author_info: return user.name as name

Return users full name (user.name) instead of email (user.email) as the
'name' of the author when taking the value from git config.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

gbp.git.GitRepository: Add remove_remote

git.repository.GitRepository.add_remote_repo: use GitArgs

GitRepository: Make rev_parse's short option an int everywhere

GitRepository/rev_parse: add new argument 'short'

Adds a new argument to get abbreviated SHA1.

Also, modifies rev_parse() to use GitArgs class.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

GitRepository/get_commit_info: check return value of git command correctly

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Make exception syntax consistent

GitRepository: Add test that covers remote branches in has_branch()

GitRepository: return GitModifier object instead of separate fields

this make sure the number of return values doesn't stays sane when
also returning timestamps and committer information.

GitRepository: make get_commit_info() more robust

Now uses git-show instead of git-log. This is needed for further
enhancements (namely to get name-status for merge commits). Also, use
null-character as the field separator which makes parsing more reliable.

The method now returns 'body' of the commit message as is, without
stripping or splitting to lines.

In addition, get_commit_info() now uses GitArgs and _git_inout() instead
of the deprecated _git_getoutput().

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

tests: test GitRepository.get_commit_info()

GitRepository: fix process cwd in _git_inout()

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

PristineTar: move Debian-specific stuff to DebianPristineTar

Continuation to the PristineTar refactoring, makes the "common"
PristineTar independent of DebianPkgPolicy. This commit moves the
Debian-specific has_commit() and checkout() methods to DebianPristineTar
class and replaces them with more generic functions in the base class.
Also, drops the Debian-specific get_commit() method completely, as it
was not used outside the PristineTar class itself.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

UpstreamSource: automatically detect package name and versions from directories

of the form packagename-<version>

Make TestDir testcase use a newly created directory

neeeded for the upcoming packagename/version parsing support

deb.git: Fixed typo in method name

the double underscore prefix was dropped ages ago.

Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>

common/pq: use strip in apply_and_commit_patch()

Use the strip information of the patch when applying patches. Also,
changes GitRepository.apply_patch() to accept integer values as 'strip'
argument.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

import-orig: move is_link_target() to common module

This change makes is_link_target() re-usable in the upcoming RPM-tools.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

gbp.conf: Make config option printing more consistent

Document changes and release 0.6.0~git20120601