log: set gbp verbose logging if --debug is specified

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

log: prefix output with logger name if --debug is specified

Helps to identify gbs and gbp log output when debugging.

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

Update default build repsitory url

Change-Id: I44f6f2fb1982f65d8c1b2f542f0590004fc4ead6

Use --tmp-dir of gbp in gbs import. #574

Cherry-pick from:bc8c03c3c2f608c7b8ab257db2c9197d30a51f85

Change-Id: Ia0f6e372d0c7667e81032cc874b3d5b4ff10fb94

Use logging from gbp and delete the msger module

As gbs already depends on gbp it makes sense to use its logging facility
instead of msger, which is copy&pasted from another project. Gbp logging
is based on standard Python logging functionality. It basically only
adds color output to it. This change adds a minimal functionality to the
'gitbuildsys.log' module to act as a GBS-specific logger.

This change is also takes more than 400 lines of code from the project
and decreases amount of pylint errors on more than 30% (120->84)

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

export: update 'VCS:' tag in spec

Updates/inserts a 'VCS:' tag to the spec file, after exporting the
sources. The format of the tag is 'repopath#tagname'.

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

rewrite the waiting decorator and move it to a new module

In preparation for removing the msger module introduce a new module
'gitbuildsys.log' to contain all logging/output related functionality.

This change introduces and takes into use a rewritten 'waiting'
decorator in the new 'log' module.

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

Add supporting armv6l

depanneur will do more arch compatible checking

Change-Id: I4476fdf447cdd3438b264f94d1c8373ec3787736

check if upstream and pristine-tar are available in remote, #551

Change-Id: Ie38295a60d8ab2b58fbc1150b2bcaa12cb8e6bee

Update dependency and bump to 0.12

Change-Id: I00907f70cb8aeddece3e71ba463ea89f5e451c15

Add --clean-repos option, #548

This option is used to cleanup local repos generated by 'gbs build'

If there're some old RPMs in 'gbs build's local repo, this option
can be used to clean up them.

Change-Id: I1ccbceaa01c8c20a909102727a76e2cf314b80d9

Launch editor for long submit request message. #553

* add functions edit() and edit_files()
* change argument of TempCopy.__init__ from original_path to
    content
* change test_changelog.py to make tests pass

Change-Id: I59b10276db6d5e0fd9843009be01c218c440ece5

import: invert the --no-merge option to --merge

Not-merging should be the default as merging upstream to master very
easily breaks the patch generation. Moreover, using gbs import to import
tarballs creates the 'upstream' branch which enables patch-generation.
Thus, the gbs toolset currently supports a haphazard/broken workflow.

Merging only fits in the "one-big-tarball" maintenance model, do it only
if user really requests it.

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

Add release notes for gbs 0.11

Change-Id: I4ba5db662e1c0bf37f8233c5d3c607e867cb873a

Export to $packaging_dir. #521

Change-Id: I7899df49c2281563311415331695e15808c17035

Remove trailing slash in $packaging_dir.

Build will fail if pass --packaging-dir=xxx/ to depanneur

Change-Id: I99695c57241f576c8dab262a7ffb135858a9dd70

Ignore .gbs.conf in patch-generation

The package-specific configuration file can be seen as a packaging
files. Don't generate patches that add/modify .gbs.conf.

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

No hard code default base project, #539

* If you realy want to to use default base project, you can set
  base_prj in obs section of .gbs.conf
* If no base project, and no target project specified, the default
  target project is home:<user>:gbs
* If base_project specified, then target project will be:
  home:<user>:gbs:<target_project>
* If no base project, and target project need to be created, then
  gbs will give warning to ask user to set repos for new target
  project manually.

Change-Id: Ib65e90f076f186e3486837199c2098dd723f67d5

Merge "x86_64 support, #527" into devel

msger: sync with the new rewritten gbp logging

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

Merge "Modify changelog order to follow default order of git log. #508" into devel

cmd_import: enable importing patches

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

x86_64 support, #527

Change-Id: I9e9e1f94d60dae3f61df0a1b880250220511e203

Merge remote-tracking branch 'origin/release-0.11.1' into devel

Modify changelog order to follow default order of git log. #508

Change-Id: Id5575f47a8dad741bbca1956b279eb9857dfa0fc

Change --spec to use only base file name

Detail of discussion is in #485

* --spec accept only base file name
* only check spec file in working copy if --include-all is given,
  otherwise check in commit
* if gbs want to parse spec file, it must parse the correct
  version that in --commit(default HEAD)
* unify --spec for all subcommands
  - ch: use the spec given first
  - rb: --include-all means to use work copy, otherwise checkout
    spec in some revision to parse
  - lb: check only one project is about to build and
    pass --spec directly to depanneur
* add several helper functions
  - show_file_from_rev: cat file content from given rev
  - file_exists_in_rev: check whether file exist in given rev
  - glob_in_rev: glob file pattern in given rev

Change-Id: I0621634b50ddedf65eab3c41db4d6f537194b345

Remove pylint warnings

Change-Id: I5cafb9637fea36d9612c708fc57daba302f19648

Add --define option. #540

Change-Id: Ic924d639908a1ce731bb525cbcaba02217930377

show simple progress message for long time operations #325

Change-Id: I1df3f74459dd92cb77f1cd7d7879d965578b8d0b

Add --keep-packs to keep unused packages in buildroot, #516

Change-Id: I902d98b07eaa0596849a550f7f4b97716d828468

Bump to version gbs 0.11.1

Update dependency: depanneur(>=0.2.1)

Change-Id: I8d0de5c8c72311a1fdd932f596b05504595f4813

set TIZEN_BUILD_ROOT as abspath

Change-Id: Ie1df08b2b62ca1b9f07e6ae5ee08567e291646ed

noinit support

--noinit can't be specified together with --clean or --clean-once

Change-Id: Ib3fd84ae6b53db9478b69c9daf99fe23c5f12c4e

Remove output repo and buildroot info to depanneur, #482

Change-Id: If885f0e62a5aa1368b33e924a4dbb6ecb845ed72

pristine-tar import support and update dependency

Change-Id: I830c4ff4ff6c95d8713584c098faf77c264f6536

Support for configurable packaging directory

Add config file parameter and command line option to specify the
packaging directory. Command line option takes preference over the
config file setting.

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

Fixed empty response crash

Sometimemes OBS returns empty response without raising any exceptions.
remove_files API tries to parse this response as xml and crashing if
it's empty. It happens not very often, so trying 3 times to get
something parseable should work around this OBS bug.

Fixes: #495

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

Implemented uploading only changed files to OBS

Before uploading local files to OBS remotebuild code now checks if it's
really needed. Only new and changed files are uploaded. Old files are
deleted from OBS project.

This change also reorganizes remotebuild code a bit to avoid repeated
queries to OBS.

Fixes: #474

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

bump to version gbs 0.11

Change-Id: Ib475a847d03c709d16f957184fc6d2189932abe2

Fix typos. #497

Change-Id: Ifdacca345a449ffbd8ba503a0dc8d5b399ba50d1

Update dependency for gbs 0.11

* depanneur >= 0.2
* tizen-gbp-rpm >= 2012.10.25
* osc >= 0.136.0

Change-Id: Ia3e1d947e1c5134df3cf2780d33288e13f6ac053

Change default repo config. Fix #488

* change default repo from snapshots to releases
* comment repos of base and main

Change-Id: If1c2ba3911e6301a41883335c7f1e126ea1725e3

fix backtrace while trying to remove 'None' file, #490

Change-Id: I7d1b7c3701fb9e36006b7f88a9248feb4bcd4060

Merge "guess_spec() should return a spec belonging to given project" into devel

depend osc-0.136

Change-Id: Ia0488e4fcec45a27c3c01965cfe9ea3bccced03e

guess_spec() should return a spec belonging to given project

Fix #483

Change-Id: I29810b0ae6aa11598793cd7dc3fcf29616330a97

Add two more usage example for gbs build

Change-Id: I17f1a459e65a64b36c4bcc5addd32391e8c8ffe9

Don't show generating src rpm info if no --source-rpm

Change-Id: I0aed603c387633f2abfc7b2bf31dd95a63e35894

Revert "Revert "Implement build --spec. Fix #386""

This reverts commit e72f7f2c49239e09d98b6cea96ce9a0d84be45eb.

After more discussion, we'd prefer to keep the current implementation
just like gbs 0.9. Fix all --commit / --include-all issue in next
gbs release.

Change-Id: I0717d663975cbbc6cac3f5993c4ebda4a742946f

Merge "Remove checking of target project, fix #475" into devel

raise an ObsError if SSL verification failed

Remove checking of target project, fix #475

If no permission to create target project, gbs return:
error <obs>: Can't set meta for Tools:Devel1: HTTP Error 403:  Forbidden
Without this patch, gbs rb can only create target project with name start
with home, which does not make sense.

Change-Id: I7ba7f2f01b9b01a1beca920412fdc53c4641e292

Revert "Implement build --spec. Fix #386"

This reverts commit 9c3e0e1683a39752a86f34166aacec5a103bd662.

Implement build --spec. Fix #386

Pass the spec file to depanneur, it will only build that file.

Change-Id: I9dd3aabe719d2f022e1650493bc44a27ca3b5b67

Merge "submit: fixes to target guessing logic" into devel

Merge "Cleaned up runtime package dependencies" into devel

Cleaned up runtime package dependencies

Gbs doesn't depend on rpm directly since it started to use
git-buildpackage. It doesn't depend on build since it started to use
depanneur. It doesn't depend on createrepo since local build mode was
dropped in favor of using depanneur. Binfmt-support and qemu are not
used at all. Dependency to python-pycurl was missing in debian/control.
Dependency to python >= 2.5 was replaced with dependency to
python >= 2.7 as gbs now uses argparse.

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

submit: fixes to target guessing logic

Try to find the upstream branch of the current branch, not the target.

Also, get rid of the confusing variable name 'target_branch' as we're
not submitting to any branch - the 'target' is just one part of the tag
name.

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

submit: fix crash in guessing remote in case upstream is not set

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

Remove setup qemu env code, fix #466

this work should be done in build script.

also remove dependency of qemu-arm-static, which should depend by
build.

Change-Id: I7801938955022c3bec1e707a9bbfb9675751f794

submit: try to guess the remote server

If the current branch has upstream set, use it's remote as the server to
push to. Only uses 'origin' as default if no upstream is set.

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

Merge "Make build/remotebuild profile selectable from cmdline" into devel

Merge "Unify color of debug message in gbs, gbp and depanneur." into devel

Make build/remotebuild profile selectable from cmdline

Implements a new cmdline option to select the profile to use for
building. User can give the profile name without the "profile." prefix.

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

Make local buildroot configurable in config file

The buildroot can be set under the 'general' section as a global setting
and/or under individual profiles as a profile-wide setting.

Setting supports string substitution for ${tmpdir} and ${profile}.

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

Merge "empty commit message is not allowed for gbs submit, #468" into devel

Merge "More correct way to parse repo url" into devel

Merge "Fixed bug in command line aliases implementation" into devel

empty commit message is not allowed for gbs submit, #468

Change-Id: Ie89de3c755602ecbf0be09c4d69e9873c1b1a7ef

Unify color of debug message in gbs, gbp and depanneur.

Fix #450.

Change-Id: Ifdd796f6b4747e0a8903f41be24f1ee9af0cf3ed

More correct way to parse repo url

Only continue with PageNotFound exception, other exceptions, such as
AuthFailed, TimeoutError, UrlError, raise exception and exit directly.

Change-Id: I5f98d632f3af88223c266dba1a35e253c7767b1c

Fixed bug in command line aliases implementation

If --conf global option is used its argument considered as subcommand
name. This change fixes this.

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

Fixed getting workdir in remotebuild mode

Used the same approach as in other modules: workdir is assigned to git
topdir.

Fixes: #464

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

Merge "Used guess_spec in changelog mode" into devel

Used guess_spec in changelog mode

This change fixes confusing behaviour of changelog mode. When there are
more than one .spec file in packaging/ directory first one is picked up
for processing. Users were confused, because there was not way to
understand which spec will be used and there was not way to specify spec
to proceed with.

Fixes: #399

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

check the existance of 'source_rpm' before using it

Change-Id: I132dde430d123191b7ebe207eafbce8175ff079b

Implemented command line parsing using argparse

Fixes: #259

Change-Id: I25c6eabfedc30f089a0c5cb4cebaac3b31b97c8c

cmd_export: use gbp to build src rpm

Fixes #396

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

Don't rely on absolute path of Depanneur

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

Use os.system to ignore INT/QUIT signal while calling depanneur

subprocess.call does not ignore INT/QUIT signal, which result in
gbs exit first before depanneur finished

Change-Id: I88dccad5295f29f07c9d6206287f0f2b6210e5e0

Better help text for --upstream-tag option

See #379.

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

export: handle new gbp return value

GBP now distinguishes tarball/patch generation errors from other errors.
Fixes a problem where GBS thought that all GBP errors where related to
tarball/patch generation, and, possibly printed a misleading warning and
re-tried GBP export in vain.

See #382.

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

Repo or OBS section can inherit only user or password from profile.

- Fix #387, add test cases for this.
- inline user/password have the hight priority.

Change-Id: I6a8d8f6d81d012596e7383d65279060d9cfac0c9

Check the format of build conf

build conf must end with .conf and can't contain '-'.

Change-Id: I44dc370b40844a0c1b911c50cd4f22cfe644bd5c

Format default config template.

- Add more comments and two tizen url to default config template.
- Remove useless config item buildconf and build_cmd

Change-Id: I0faec1b20ad1a41044f6115770fb087371f70303

Check if specified build conf exists.

Change-Id: Ia2488a6e6841a7f87397c5c1f595bccc2b737e06

Use abspath for build.conf.

Change-Id: Ia739ff531f8a32906ac3f4913fd753960d77ec84

Correct setting gbs build root order

--buildroot option first, then TIZEN_BUILD_ROOT env var, and last
choice ~/GBS-ROOT/, which is the default.

Change-Id: Ieae8c7170c0f3e58a676351d3f34ad6a4ed5a87e

Merge "patch-generation: filter out changes to packaging dir" into release-0.10

patch-generation: filter out changes to packaging dir

Utilize the ignore-path functionality of git-buildpackage to ignore
changes made to the packaging files when generating patches.

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

Update gbs version from source

This affect output of gbs --version

Change-Id: Ia994406218fc41e08c544c7509ffe409aca9d2cb

Update gbs build help info use build instead of localbuild

Change-Id: I50e182dbf7d626989b51c811d891296b16210fe0

Don't use local build conf, give error msg instead

local build conf is out of date, and should be discarded.

Change-Id: Iba23ae5cfe082b8e11aac033e0128cf047506a5b

Merge "transfer correct gbs export options to depanneur" into release-0.10

Small changes to usage format

Change-Id: I1778037d94a9c5ef56be7352a9b9f9fda688d678

transfer correct gbs export options to depanneur

Change-Id: If988d075a91392434d0619662e131291d0bc2123

bump to version 0.10

This commit is for new gbs 0.10 release branch (release-0.10).

Change-Id: Ice6b60b9974a2cbaf272a3a491dab3e1b4f24a20

Update chroot module

As depanneur is used to create build, and multiple build root will
be created if --threads=<> specified, so we remove -A option.
The build root should be specified while using gbs chroot, just as
chroot command.

Change-Id: I014e441f2ff206002595bd3acf54ff749ffcf353

Use build-experimental to replace build module

We have tested new new build module 'build-experimental' for several
circles, and its compartiable with build module, no any regression
issue, so let's use this module to replace current build module.

Change-Id: I58ea77cb86f21d32a5819bb0213291e407f810e5

use 'warning' instead of 'warn'

Change-Id: I0b60aa144acffac2b2b16765a6e63a9724add6a8