Documnt that the patches must apply without fuzz

Fix comitter vs committer typos

Thanks: Sandro Tosi
Closes: #748339

Handle version format errors more gracefully

So far if a package claimed to be non native but the version number
didn't contain a '-' we failed like:

  Traceback (most recent call last):
    File "/usr/bin/gbp", line 9, in <module>
      load_entry_point('gbp==0.6.13', 'console_scripts', 'gbp')()
    File "/usr/lib/python2.7/dist-packages/gbp/scripts/supercommand.py", line 82, in supercommand
      return module.main(args)
    File "/usr/lib/python2.7/dist-packages/gbp/scripts/buildpackage.py", line 541, in main
      output_dir)
    File "/usr/lib/python2.7/dist-packages/gbp/scripts/buildpackage.py", line 96, in prepare_upstream_tarball
      upstream_tree = git_archive_build_orig(repo, cp, output_dir, options)
    File "/usr/lib/python2.7/dist-packages/gbp/scripts/buildpackage.py", line 270, in git_archive_build_orig
      upstream_tree = get_upstream_tree(repo, cp, options)
    File "/usr/lib/python2.7/dist-packages/gbp/scripts/buildpackage.py", line 245, in get_upstream_tree
      cp['Upstream-Version'])
    File "/usr/lib/python2.7/dist-packages/gbp/deb/git.py", line 107, in version_to_tag
      return format % dict(version=DebianGitRepository._sanitize_version(version))
    File "/usr/lib/python2.7/dist-packages/gbp/deb/git.py", line 122, in _sanitize_version
      return version.replace('~', '_').replace(':', '%')

It shouldn't be like that.

gbp pq: document --force

Slightly improve usage output

Document changes and release 0.6.13

config: add decorator to add_option_* functions

This allows us to build an internal list of valid options and print
these.

Import command not module

This matches the function name

docs: re-enable missing manpage

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

Remove tests now covered by 18_test_Config

Revert "GitRepository/add_remote_repo: fix the 'tags' argument"

Which caused an unwanted behavior of not fetching refs/heads at all --
only tags were fetched.

This reverts commit 5dd598fb8f2a9109189fad6cf631811f12ce2130

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

Add minimal 'config' command

This only allows to print single config values so far.

Closes: #733470

.gitignore: ignore editor backup files

Test option parser fallbacks more thoroughly

revealing another bug where we overwrote parsed values with defaults

Closes: #733759

Document changes and release 0.6.12

Use a much simpler version to fix the command name in --help

mostly reverting e1780f0.

Closes: #742907

Document changes and release 0.6.11

Fix command output

The first line lacked the subcommand like:

$ gbp pull --help
Usage: gbp [options] - safely update a repository from remote

instead of

$ gbp pull --help
Usage: gbp pull [options] - safely update a repository from remote
           ^^^^

ComponentTestBase: use regex matching in log checking

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

pq: minor log message typo fix

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

Use a temporary directory

This avoids file name collisions and weired files in the working copy.

tests: more robust check for the default urgency level of dch

The dch tool might not report it's version if it is incorrectly
compiled. Create a dummy changelog and parse it "manually" to determine
the default urgency level.

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

GitRepository/add_remote_repo: fix the 'tags' argument

Explicitly use --tags in git arguments - otherwise tags might not be
fetched.

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

GitRepository/get_commit_info: correctly handle file renames/copies

Use the '--no-renames' git command line option in order to prevent two
filenames per entry.

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

docs: add some missing sgml closing tags

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

Fix config file name

Remove incorrect docs about multiple filter options

Closes: #718536

Request verbose output on bug reports

Document changes and release 0.6.10

git-import-dsc: Document --create-missing-branches

import_dsc: Create missing debian branch with --create-missing-branches

Closes: #739888

Remove cover-min-percentage

It makes running single tests harder than necessary since it requires
the percentage also when using "nose <testname>".

config: Don't pull in config defaults twice

This would otherwise overwrite values set in the legacy config sections.

Closes: #733759

config: Don't pull in config defaults twice

This would otherwise overwrite values set in the legacy config sections.

Closes: #733759

Update copyright notice

Git-Dch: Ignore

Document changes and release 0.6.9

docs: improve docs on using upstream's git repo

The documentation on using upstream's git directly instead of tarballs
was very terse. Add a step by step guide on how to get started and
how to update to new upstream versions.

import_orig: test error paths of find_source

tests: adapt dch tests for older devscripts

Determine the default urgency level by determining the version number of
the dch tool from command line.

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

Change the default of cleaner to /bin/true

Running a clean command within version control by default nowadays
causes more trouble than it helps. It's unnecessary with
pbuilder/cowbuilder/sbuild and with export-dir. So change the default
from 'debuild clean' to a noop ('/bin/true').

Closes: #670624

Make parsing config file sections symmetric

Always read the legacy command's config file section prior to the
subcommand's config file section.

Until now 'gbp <subcommand>' would read '[subcommand]' as well as
'[gbp-<subcommand>]' sections while 'gbp-<subcommand>' would only read
'[gbp-<subcommand>]' sections.

Closes: #733759

Avoid backtrace on config file parse errors

"gbp buildpackage" did this already but others didn't

Closes: #733759

Depend on newer devscripts

These contain a uscan that handles repackaging at the right level and
won't let us pick up the wrong tarball name.

Closes: #635920
Thanks: gregor herrmann and all others involved

Document changes and release 0.6.8

Determine changes file name based on dpkg-buildpackage options

Closes: #732678

Run nosetest on executables too

They're all well behaved and this allows us to use doctests in them

Adjust test cases to newer devscripts

Closes: #732384

No need to document --verbose

Properly document --help instead.

Thanks: Olivier Berger for pointing this out
Closes: #731149

Document changes and release 0.6.7

Use --distribution instead of --dist

Closes: #730788

Fiz description of backports branches

log: add error and warning aliases

since I tend to use them instead of err and warn.

Closes: #728896

import-dsc; Merge upstream version by tag

instead of simply using the upstream branch name. THis makes sure
we also merge upstream versions that were imported previously with
e.g. "gbp import-orig".

Closes: #698222

Document changes and release 0.6.6

Disable html coverage generation

until python-nose is fixed.

Update to git-pbuilder 1.30

that resyncs with what we shipped in gbp.

import-dsc: allow to skip Debian tag creation

Closes: #636368

Add zsh completion

Closes; #717002

docs: update pq manpages regarding pq command meta tags

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

pq: support 'Topic' patch-export command

Topic can be defined with either 'Gbp: Topic <topic>' or 'Gbp-Pq: Topic
<topic>' in the commit message.

This is to replace the "gbp-pq-topic: <topic>" command.

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

pq: listen to 'Gbp-Pq:' commands, too

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

pq: add format_diff() function

For generating a patch file from a diff between two arbitrary commits.

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

pq: support patch-export commands

Support giving commands to pq as a meta tag in commit message. The
format is "Gbp: <command> [args]".

Currently, only one command is supported. namely 'ignore'. That is, one
can use 'Gbp: Ignore' in the commit message for ignoring the commit in
patch-generation.

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

pq: properly generate non-ascii patch files

Encode non-ascii email headers properly. Also, set MIME headers
correctly for the message body if the commit message body contains
non-ascii characters.

The reason for constructing the message in a little bit "clumsy" way is
the intention is to match the output of git-format-patch as closely as
possible.

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

pq.format_patch: support file path filtering

Implements a filter option that allows filtering out changes to certain
files/paths in the patch-generation. A commit is totally ignored if all
files would be filtered out. The path filter is given as a Python
regexp.

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

pq: rewrite patch export functionality

Use our own function for constructing the patch files instead of using
the format-patch command of git. This way, we get the desired output
format directly, without the need for the error-prone "format-patch,
parse patch files, mangle and re-write patch files" cycle.

Also, fix patch naming in patch generation when '--no-patch-numbers' is
used.  Previously, multiple commits with the same subject resulted in
multiple patches having the same filename. This lead into broken series
with missing patches as patch files were overwritten by the topmost
commit.

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

Add missing bash completion for gbp import-dscs

Closes: #727749

gbp-clone: support cloning to a specific directory

Closes: #725666

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

gbp-clone: support repo URLs like "host:repo.git"

Previously, gbp didn't correctly parse this kind of URL and clone
failed.

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

Document changes and release 0.6.5

Bump standards version

Don't export DEB_VENDOR="Debian"

Tests now take Ubuntu versions into consideration.

Closes: #723756

Be more robust about git status output changes

by using --porcelain. Heavily based on a patch by rian m. carlson

Closes: #726260

doc: Fix git-pbuilder example

Closes: #725875

Document changes and release 0.6.4

Backports got integrated into the main archive

From wheezy on we can use the main archive

Update to git-pbuilder 1.29

pq: Only print number or tries if we try more than once

pq: print which patch failed to apply

typo s/bei/be/

Closes: #722359

tests: remove unused import in submodule tests

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

tests: upper case global variables in submodule tests

Prevent some confusing name clashes.

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

tests: add some docstrings to git submodule tests

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

tests: use eq_ and ok_ from nose tools in submodule tests

For consistency and better assert error messages.

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

tests: add test for archiving without git submodules

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

GitRepository: deprecate the get_remote_repos method

Superceded byt the get_remotes() method.

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

GitRepository.has_remote_repo: use get_remotes method

Use the new get_remotes() method

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

git: new class and method for remote repositories

Add a new GitRemote class for representing git remote repositories.
The initial, very limited, version only contains information about the
fetch and push URLs of the remote repository.

Also, add a new GitRepository.get_remotes() method for getting remote
repositories as instances of the new GitRemote class.

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

GitRepository.pull: Add 'all_remotes' option

Also changes the method to utilize the GitArgs class.

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

GitRepository.push: Add 'tags' option

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

GitRepository.push: Add 'force' option

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

GitRepository.list_tree: add 'paths' option

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

GitRepository.diff: add 'ignore_submodules' option

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

common/buildpackage: fix handling of empty tarball prefix

Previously empty prefix in tarball generation introduced a leading slash
in all filenames in the archive.

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

Add link to online docs

Remove unused import

ComponentTestBase: use eq_() ok_() from nose.tools

For better assert messages.

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

GitRepository.diff: prevent usage of external diff

External diff might break patch generation, for example.

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

GitRepository.diff: add 'text' option

For generating textual diffs. Useful for Pq - for example, the 'patch'
utility does not support git binary diffs.

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

GitRepository: Add clean() method

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