notify: catch RuntimeError when importing pynotify

Work around a problem in some distros (e.g. Fedora) where "import
pynotify" crashes in RuntimeError in some cases, e.g. when DISPLAY env
variable is not set.

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

gbp-dch: fix handling of the '--meta' option

Make it effective again - previously it was totally ignored. Also,
change it's default value to True to match the current behavior.

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

Document changes and release 0.6.22

Conflicts:
debian/changelog

git.vfs: fix close method

Preventing a infinite recursion which can be triggered by gbp pq export
--commit.

Document changes and release 0.6.21

Conflicts:
debian/changelog

Update README a bit

Git-Dch: Ignore

Conflicts:
README

Complete setup.py for pypi

Improve change reporting a bit

man: fix option argument for --git-pbuilder-options

pq: Don't fail commit if the series file is empty on the source branch

Document changes and release 0.6.20

Conflicts:
debian/changelog

pq: Add "pq export --commit" option

This commits the changes in the pq right away. This is currently
experimental and subject to change.

Conflicts:
gbp/scripts/pq.py

meta-closes: Move help text to GbpOptionParser.help

pq: document --drop

Allow to always drop pq branch after export

Closes: #761160

Conflicts:
gbp/config.py

Document changes and release 0.6.19

Conflicts:
debian/changelog

Update to git-pbuilder 1.33

Document changes and release 0.6.18

Conflicts:
debian/changelog

Mention --no-merge

Closes: #760091

Don't delete *_source.changes on source only builds

Closes: #758726

Document changes and release 0.6.17

Conflicts:
debian/changelog

bash completion: make command list dynamic

Use "gbp --list-cmds" so we don't have to hardcode the available
commands and get support for the RPM ones as they show up.

Unify doc strings a bit

since they now show up with --list-cmds

Allow to list all available commands

gbp: add --version option

Closes: #758909

Improve error messages on formatting errors

Make it easier for the user to detect misformated replacement strings in
config files and command line options.

Conflicts:
debian/git-buildpackage.install
gbp/scripts/import_orig.py

buildpackage: Make debian-tag message configurable

New config option --git-debian-tag-msg allows for the specification of
the message format string for signed debian-tags.

When left unset, the default debian-tag-msg format is still:

    %(pkg)s Debian release %(version)s

Signed-off-by: Kamal Mostafa <kamal@whence.com>
Conflicts:
gbp/scripts/buildpackage.py

dch: Only modify the mainttrailer when --git-author is in use

Use better variable name

It's a boolean indicating wether we fetch the author information from
git not the author information itself.

Git-Dch: Ignore

Brown paper bag, please

Gbp-Dch: Ignore

Make sure we fixup the changelog trailer with newer devscripts

We don't change any mainttrailer options if already given.

Thanks: James McCoy for the detailed explanation
Closes: #740566

Document changes and release 0.6.16

Conflicts:
debian/changelog

Remove all other log handlers when capturing logs

This avoids spurious error messages during test runs.

linkeList: turn the bug printouts to debug level

We don't take any action anyway so no need to confuse users.

Introduce git-buildpackage-rpm

Currently only including "gbp import-srpm"

Conflicts:
debian/control
debian/git-buildpackage-rpm.install
debian/git-buildpackage.install

import-srpm: add 'vendor' config option

Intended to represent the distribution vendor (e.g. 'Debian').
Currently, this can be used in tag format strings.

NOTE: the vendor name is converted to lowercase when used in tag names.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Conflicts:
gbp/config.py

Pass --no-pristine-tar to SRPM compnent tests

to avoid pristine-tar showing up in the matched branches

Conflicts:
tests/component/rpm/test_import_srpm.py

Introduce git-import-srpm tool

Initial version of the git-import-srpm: a tool for importing source
rpms.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Conflicts:
.gitmodules
gbp/config.py
gbp/scripts/import_srpm.py
tests/01_test_help.py
tests/component/rpm/__init__.py
tests/component/rpm/data
tests/component/rpm/test_import_srpm.py

Merge commit '1a8e6d12cc94d2b40324fe8f99de012cf2cf5913' into release-201501

Merge from devel dbfc6276c4aa50b79f1cf27cfc24badc0b18da8f

Merge from devel e374ee5a2381ba30056c1fa33bdb515d99ec704e

Merge to 7f6b72e039efa5e2c3517b9336f9b29b4cb65c71

Merge to 36c8e5cd00fd600be63160b0ef9427a70ab45870

merge to 0b1fc0d602d75da30748bbc979fed48aab805503

merger to 655a6f0528afcebe1b4493abe865ee568c505bbf

modify .gitmodule
Signed-off-by: shuai.fu <shuai01.fu@samsung.com>
Change-Id: I93c6a4eccb8d62b0a164c13902726cf3c92ac039

Upgrade version to 0.6.25

Change-Id: I1d62032a98e91130c35ceccff576262709d806b3
Signed-off-by: Rong Jinhui <jinhui.rong@samsung.com>

Tizen release 0.6.15-20140828

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

rpm packaging: use macro for python binary

For fedora distro, when building git-buildpackage rpm packages in the buildroot,
/bin/python is called to do python setup.py actions during rpmbuild,which leads
to /bin/python is taken as requirement of git-buildpackage-rpm-0.6.15-0.noarch.rpm
automatically, resulting in:

Install gbs with dnf command in fedora distro will raise error:
nothing provides /bin/python needed by git-buildpackage-rpm-0.6.12-59.1.noarch

Fixes: DEVT-197

Change-Id: I30169a506d7f92aed759124c40beb2803c196218
Signed-off-by: Li Jinjing <jinjingx.li@intel.com>
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

Add PKGBUILD to support arch linux

Three split packages: git-buildpackage-{common,rpm,doc}

Signed-off-by: Junchun Guan <junchunx.guan@intel.com>

rpm tests: add unit tests for buildpackage-rpm

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

Make pychecker happy

We don't use the class variable anywhere

UpstreamSource: move version guessing logic to PkgPolicy

PkgPolicy is more logical context, for example if guessing version from
a plain filename and not a real file.

Also, changes guess_version() to always return a tuple to simplify
checking its return value.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Conflicts:
gbp/pkg/__init__.py

Change UpstreamSource to have PkgPolicy

The UpstreamSource class now gets a PkgPolicy in it's initialization.
Also, introduces new DebiaUpstreamSource class which is taken in use in
the scripts.

The PkgPolicy is not yet used for anything in UpstreamSource.

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

UpstreamSource.guess_version: don't check for directories

Directories are not recognized anyway, if guess_version() is called for
non-existent sources. And, parse_archive_filename() works fine for
directory names, too.

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

UpstreamSource/guess_version: detect more compression formats

By utilizing the parse_archive_filename() function that detects e.g.
"alias suffixes" such as 'tgz'.

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

Fix typos

Move get_compression() out of pkg.PkgPolicy class

Renames the function to parse_archive_filename() and changes it's
return values. Filename parsing is merely generic functionality, not
tied to any packaging policy.

The function now returns the base name of the file (that is, filename
without, archive and compression extensions), archive format and
compression method. Adds supported archive formats 'tar' and 'zip' and
file extension aliases, e.g. 'tgz'.

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

rpm tests: disable one anomalous pylint error

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

packaging: depend/recommend rpmbuild

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

buildpackage-rpm: fix one error message

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

buildpackage-rpm: drop unneeded exception

We will never get here as parse_spec() always returns GbpError.

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

buildpackage-rpm: handle invalid keys in orig prefix

Gracefully handle invalid keys in the --git-orig-prefix format string.

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

buildpackage-rpm: minor refactor

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

buildpackage-rpm: get rid of prepare_export_dir()

No need to mangle the export directory.

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

buildpackage-rpm: remove some outdated/unnecessary exception handling

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

buildpackage-rpm: add '--git-native' option

Can be used to explicitly configure a package as native or non-native.
Default behavior is still to guess based on the existence of upstream
branch.

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

buildpackage: refactor tarball compression

Introduce a new function for running the compressor commands. Unifies
exception/error handling - GbpError will be returned if the compressor
fails.

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

buildpackage-rpm: remove some unused variables

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

buildpackage-rpm: add missing docstrings

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

buildpackage-rpm: remove unused imports

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

packaging: recommend/require zipmerge

Needed by CatenateZipArchive().

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

rpm-packaging: recommend/require unzip

Unzip is needed for handling zipped upstream sources.

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

rpm tests: update component test data

Adds pq/devel branch and .gitignore file to gbp-test package.

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

rpm tests: remove RepoManifest.add_project()

This functionality is moved to the rpm testdata submodule.

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

ComponentTestBase: add dirs argument to _check_repo_state()

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

log: fix auto colorizing for custom streams

Check for existence of isatty() method in the stream object. Some custom
streams (e.g. in nose) do not necessarily have this.

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

log: don't automatically setup gbp-specific logger

This makes it nicer to use gbp.* modules (e.g. gbp.git) in other
software projects which may have their own logging setup. Now, importing
gbp.* modules won't force-setup logging.

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

buildpackage-rpm: remove too general exception catching

There should be no reason to catch general Exception here.

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

buildpackage-rpm: don't crash on invalid vcs format strings

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

buildpackage-rpm: add --git-no-build option

Replaces and deprecates --git-export-only. The new option name is way
better in line with the other option names.

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

docs: document common options in one place

Move the documentation of command line options common to all tools into
one file. This unifies the manpage documentation and should make its
maintenance easier. Also, this change adds missing options to several
manpages.

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

docs: cosmetic reformat of git-dch manpage sgml source

Makes the sgml source more readable. Fix indentation, tab to space, wrap
lines.

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

docs: cosmetic reformat of pq manpage sgml source

Makes the sgml source more readable. Fix indentation, tab to space, wrap
lines.

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

docs: cosmetic reformat of import-dsc manpage sgml source

Makes the sgml source more readable. Fix indentation, tab to space, wrap
lines.

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

docs: cosmetic reformat of import-orig manpage sgml source

Makes the sgml source more readable. Fix indentation, tab to space, wrap
lines.

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

docs: change option argument names to uppercase in buildpackage man

For the sake of consistency.

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

docs: cosmetic reformat of buildpackage manpage sgml source

In order to make the sgml source more readable. Fix indentation, tab to
space, wrap lines.

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

buildpackage_rpm: always run cleaner hook

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

buildpackage_rpm: fix crash when running prebuild hook

GitRepository.git_dir is not callable.

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

tests/testutils: add directories arg to ls_* functions

With this you can exclude directories from the listing returned by the
ls_tar(), ls_zip() and ls_dir() functions.

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

buildpackage_rpm: check submodules from the exported treeish

And not from the current working copy.

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

GitRepository/has_submodules: add treeish argument

For defining a Git treeish which to look into, instead of the current
working copy.

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

ComponentTestBase: ignore system and user config

User might have a system and/or user specific gbp config files. Disable
these config files so that they don't affect the component tests.

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

ComponentTestBase: add ls_tree() method

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

ComponentTestBase: add check_files() method

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

import-orig-rpm: download tarball into parent directory

Prevents dirtying of the current Git working directory.

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

buildpackage-rpm: always create tarball for native packages

If the package is native, always (re-)create the source tarball, even if
--git-no-create-orig is used. Not creating the tarball does not make
much sense (and most probably just causes problems and confusion): for
native packages there is no place for external tarball and no patches so
we need to always have the correct sources (i.e. the exported revision)
in the tarball.

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

buildpackage-rpm: fix exit code in case of GitRepositoryError

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

buildpackage: Also print tag name when tagging the Debian release

Based on a patch by Kamal Mostafa