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

git-buildpackage: allow to build packages from detached HEAD state

with --git-ignore-branch.

Closes: #661598

git-buildpackage overlay-mode: don't fail on files named like tarball

Instead of moving individual files around simply rename the directories.
This is faster and independent of the tarballs content.

Closes: #675412

git-buildpackage: add missing import

so move_old_export works again

gbp.git.repository: add GitRepository.make_tree

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

gbp.git.repository: Add GitRepository.write_file

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

gbp.git.repository: Add GitRepository.list_tree

Refactor deb helpers: move PristineTar class

Based on a patch by Markus Lehtonen

This refactor is preparation to the upcoming rpm support.

import_dsc: add --allow-unauthenticated

Closes: #670623

Document changes and release 0.6.0~git20120524

Provide minimalist debian/control object

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

gbp.deb.ChangeLog: Add filename property

gbp.deb.changelog: Split parsing into a separate function

this will allow us to reparse the changelog after manipulation with dch.

Don't explicitly refer to lenny-backports

Thanks: Salvatore Bonaccorso

Document --bpo, --nmu and --qa in git-dch's manpage

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

git-dch: Add support for --team switch for Team uploads

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

Fix typo in doctest that made epydoc fail.

Git-Dch: Ignore

Add gbp.deb.ChangeLogSection

to parse package and version out of a changelog section

gbp.git.repository: Add a "git merge-base" wrapper

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

Config help fix

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