Git.Repository.__git_inout: Close all other file descriptors

No need to leak fds to the child.

Git.Repository.__git_inout: properly set stdin

We should set stdin to subprocess.PIPE when piping input according
to the subprocess docs.

Use has_key

It's shorter and get's rid of the unused dummy variable.

Document gbp's help option

Document changes and release 0.6.1

Depend on python-pkg-resources

Closes: #714238

Split {Build-,}Deps per line

and sort alphabetically

Document changes and release 0.6.0

import-dsc: print filename instead of object name

config: Don't fill in the parser with all defaults

There's no need to fill the parser with all the built in defaults since
we can simply copy them to the config from the class dict. This allows
us to first set the values from the old {git,gbp}-* commands and then
only overwrite changed values from the newer "gbp <command>" versions.

Otherwise we'd overwite all old style config with the new style one.
This also fixes the problem where the default section wouldn't be read
at all if the command doesn't even have an empty section in any gbp.conf
file.

Fix typo

Git-Dch: Ignore

gbp-import-dscs: invoke gbp import-dsc

instead of git-import-dsc

Consistently call gbp the supercommand

and robustify against invalid modules names.

gbp: support --help

Update package description

NEWS: add a note about the new gbp super command

Rename the sgml files from git- to gbp-

gbp.conf: Use command names instead of {git,gbp}-

docs: Rename the entities from &git-<command>; to &gbp-<command>;

manpages: Changes references from git- to gbp-

docs: Use gbp <command> instead of {git,gbp}-<command>

Add gbp manpage

Bash complete on "gbp <command>" too

Git-Ignore generated gbp.conf.5 too

GbpOptionParser: add test for 'filter' option handling

GbpOptionParser: Make sure we parse the old config sections

For backward compatibility between {gbp,git}-<subcommand> and "gbp
<subcommand>" make sure we parse the former sections if using the later.

GbpOptionParser: Make sure we access the GbpOptionParser*'s default dict

and not the one from the OptionParser. The instance's default dict this
gets reset to empty when invoking OptionParser.__init__.

Add wrapper for all gbp commands

So like git you can now use gbp <command> instead of git-<command> or
gbp-<command>. The manpages and docs aren't adjusted yet.

GitRepository/diff: allow single object

Allow diffing a single object, e.g. getting diff from single commit.

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

Simplify expression

Git-Dch: Ignore

GitRepository.get_subject: use get_commit_info

This kills another _git_getoutput. Also deprecate the method since we
don't need to special case the subject.

buildpackage: use get_commit_info() instead of get_subject()

Utilize the get_commit_info() method of GitRepository instead of the
deprecated get_subject().

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

Remove unused imports

import-orig: keep working copy in sync with branch HEAD

Update working copy and index (to branch HEAD) if we modify the
currently checked-out branch. Otherwise the repository is left in
unclean state when the current branch is upstream or pristine-tar and
the '--no-merge' option is used.

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

GitRepository/get_commit_info: add patchname to info

Add a new 'patchname' field to the information returned by
get_commit_info. That is, the subject in a sanitized format, similar to
what git-format-patch uses.

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

gitmodifier: make the datetime object always have timezone

To be consistent with the date property and for easier compatibility
with git dates.

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

Document changes and release 0.6.0~git20130630

Add Jenkins Scratchbuilder example

It's a simple script that can be used to build Debian packages via gbp
and Jenkins.

git-buildpackage: Unbreak overlay mode

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

GitRepository: Use LC_ALL=C when checking features

This fixes the doctest and also makes sure we use an up to date manpage.

Thanks: Daniel Dehennin for the report

Document changes and release 0.6.0~git20130506

manpages: Document gbp-pull's --ignore-branch option

gbp-pull: Fix typo in help output

GitRepository Use _git_inout in get_branch

to suppress the confusiong "fatal: ref HEAD is not a symbolic ref". We
alrady put out a more descriptive error message. This makes using
--ignore-branch look far less dangerous.

Add --ignore-branch to gbp-pull

This allows us to fetch upstream and pristine-tar even when in detached
head state. This can be usefull when e.g. building in jenkins which uses
a detached head by default.

dch: make sure branch is always defined

This fixes a problem when running git-dch form a detached HEAD

trivial: simplify option assignment

to ease readability

Silence test_debian_missing_author

and better use the output for verification

Silence dpkg --compare-versions

and add the error detail to the exception message. This silences the
testsuite and makes sure we have error message and exception output in
sync.

Use Popen.communicate instead of subprocess call

so we can capture stderr and use it in error messages intead of spewing
it on the console.

Make parse_dsc a classmethod of DscFile

so we have the object creation close to the object itself.

ChangeLog: Remove is_native

to avoid accidental usage. Use DebianSource.is_native instead.

dch: Use DebianSource.is_native

instead of ChangeLog.is_native since the former used
debian/source/format in favor of debian/changelog.

Revert "Use Popen.communicate instead of subprocess call"

This reverts commit 0c582cd827f164168a313e0bb4eabf9487bdd088.

Use Popen.communicate instead of subprocess call

so we can caputure stderr and pass it along with any errors.

GitRepository.has_feature: capture stderr

to avoid spurious output on the console and add the error message to the
exception intead.

GitRepository.commit_dir: raise GitRepositoryError instead of GbpError

and make sure the callers forward the error properly

Silence GitRepository.write_tree

but add any error output to the exception instead. All callers have been
audited to properly output the message if needed.

command_wrappers: allow to silence __call__

and use that in the doctest to get rid of the spurious output

Discard git mailinfo's error message

since we don't expect it to succeed.

Silence GitRepository.grep_log

None of the gbp script rely on the output being printed. In fact we
improve the exception by appending the stderr output.

Silence GitRepository.set_upstream_branch

It's not used within gbp itself so no command line tool expects
an error message to be printed.

Silence GitRepository.has_treeish

This also silences 10_test_get_upstream_tree

10_test_get_upstream_tree: Fix test descriptions

14_test_gbp_import_dscs: Mock gbp.log.err

to get rid of the spurious error message

buildpackage: implement --[no-]hooks option

For enabling/disabling all hooks. This option does not affect the
builder command, though.

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

tests: Fix tests for Ubuntu

On Ubuntu dch produces different version numbering. Adapt tests for
this.

Adds a jew class to parse '/etc/lsb-release' to determine the
distribution and it's codename to dynamically adapt tests accordingly.

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

DebianSource: fix is_native()

Determine from changelog if debian/source/format does not provide any
type (1.0 format).

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

tests: add missing import in test_Changelog

That was erroneously removed in
3d6b68ae541d6020ff747628b3e44196999e0a50.

and breaks without dch installed

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

GitRepository: add describe() method

Provides more output options than find_tag().
- longfmt: for getting the tag name in the long format (tag, number of
  commits and sha1)
- always: for falling back to sha1 if no (matching) tag name is found
- abbrev: for defining the length of sha1 returned

Change GitRepository.find_tag() to use the the new method.

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

tests: another fix for older dch

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

tests: fix for older dch

Prevent dch from asking user input during tests.

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

buildpackage: catch and handle DebianSourceError gracefully

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

DebianSource: raise DebianSourceError if reading changelog fails

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

bash completion: handle upstream-tree like upstream-branch

That's as close as it gets since every ref like is valid

Silence _git_command

by using _git_inout instead of GitCommand

Silence git clone

by using __git_inout instead of GitCommand

GitRepository: make __build_env a static method

since cls is unused and it silences pychecker

Silence git init

to make the test suite less verbose

Document changes and release 0.6.0~git20130414

Don't try access source before it's filled

Document changes and release 0.6.0~git20130413

Add gbp.git.source.Source.sourcepkg

to easily fetch the source package name

Git-Dch: Ignore

Honor debian/source/format

when checking if a package is a native package

Closes: #669267

Introduce Source class

so we don't have to expose all the details of Debian's different
files and conventions.

Add minimal vfs interface

so we can access blobs in git as file like objects

Use _git_inout for git.show

so we don't spew the error message on stdout for nonexistent objects

Revert "GitRepository.show: use _git_inout()"

This changes the newline behaviour. readlines() as used by
_git_getoutput correctly keeps the newlines while this change doesn't.

This reverts commit 92edb4eda14cf4b5fd1514feb7b81aa50456285c.

tests: Use tempfile.mkdtemp to create temp dirs for tests

This puts test dirs below /tmp which often is a tmpfs.

All tests include the context module which consolidates tmpdir creation
and cleanup, undoes a chdir in teardown and silences log messages.

GitRepository.show: use _git_inout()

Instead of the deprecated _git_getoutput() method. Also, capture stderr
and put the error message to the exception, instead.

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

Add .coveragerc

To get correct report if coverage is invoked directly (not through
nose/disttools) in order to get an xml report, for example.

Currently only used for restricting the coverage output to the gbp
python package. That is, same thing that 'cover-package' option in
setup.cfg does.

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

Fix typos

Git-Dch: Ignore

Generate coverage xml

for cobertura style coverage information that can be picked up
by e.g. Jenkins

Add urgency management.

* tests/11_test_dch_main.py: test the new --urgency option to
  dch.main(). Add only meaningful tests, i.e. options alone and combined
  with --release and --snapshot.

* git-dch (fixup_section): Manage "urgency" option.
  (main): Add version_group option "-U" and "--urgency" to set the
  urgency level of the entry.

* docs/manpages/git-dch.sgml: Update documentation for new option.

Add option to manage distribution fields for non snapshot mode.

The snapshot mode fails to merge two "debian/changelog" entries if the
distribution is not "UNRELEASED".

* tests/11_test_dch_main.py: test the new --distribution option to
  dch.main(). Add only meaningful tests, i.e. options alone and combined
  with --release and --snapshot.

* git-dch (fixup_section): Rename as it's used to fix header in addition
  to trailer, this require a new named parameter "options".
  (main): Add version_group option "-D" and "--distribution" to
  specify the distribution name.
  Add version_group option "--force-distribution" to force the provided
  distribution to be used, used by "dch".
  Rename fixup_trailer() to fixup_section() and add options as parameters.

* docs/manpages/git-dch.sgml: Update documentation for new options.

Closes: #646684

Move DscFile to separate module

Move DebianPkgPolicy to separate module

Move debian/changelog manipulation to gbp.deb.changelog.ChangeLog.

spawn_dch switch gbp.command.wrappers.Command.

* gbp/deb/changelog.py (ChangeLog.spawn_dch): static method adapted from
  gbp.scripts.dch and converted to gbp.command_wrappers.Command.
  (add_entry): New method adapted from
  gbp.scripts.dch.add_changelog_entry.
  (add_section): New method adapted from
  gbp.scripts.dch.add_changelog_entry. Remove DebianGitRepository and
  options, this has nothing to do with changelog management.

* tests/test_Changelog.py: Test new methods.

* gbp/scripts/dch.py: Remove useless functions: system(), spawn_dch(),
  add_changelog_section() and add_changelog_entry().
  Update calls accordingly.
  (fixup_trailer): Use spawn_dch() method of ChangeLog class.
  (process_options): dch_options became a list.
  (main): Use add_section() and add_entry() methods of ChangeLog object.
  Take care of upstream version since ChangeLog.add_section() does not
  manage it anymore.
  Update exception handling, ChangeLog.spawn_dch() can raise
  "CommandExecFailed" exception.

Closes: #672954

Document changes and release 0.6.0~git20130329

Include the name of the package being built in the debian tag message

Currently, the message in the debian tag is just:

  "Debian release %s" % cp.version

This is a bad idea, because it means that the signed message itself
contains no mention of the project that is being worked on.

Since all git repositories are conceptually the same git repository
(some just have commits that others don't have), a malicious attacker
could inject tags from project A into the repository for project B and
the original developer's signature on those tags would be intact.

This is potentially a security problem.  For example: if there are
automated build systems that pull from a repo and verify signed tags
made by a known developer (and that developer contributes to multiple
projects), this conflation could be used to make those systems build
packages from an entirely other project.

The attached patch enforces the inclusion of the name of the package
into the tag's message.

Closes: #704018