Don't break link structure if package is a link in OBS, #675

Change-Id: Iac1991d81f996e3f8f4d8368a9bafbea824af723

Add missing --no-patch-export options for remotebuild

Change-Id: I9730b05c0de08e06b27e9c1b22408a8f72230e07

Add --tag option for submit module to support group submission, #596

workflow of group submission:

gbs side:

1) one developer call 'gbs sr' to submit a patch, and he/she told
   other developer the tag name, created by gbs for this submission.
2) other developer use --tag to specified the same tag to submit
   other packages.

backend service:

Creating different OBS build project for different tag, and all submission
with the same tag would be submit to the same OBS project for building.

Release engineer:

All packages in the same OBS build project should be accept at the same
time

Change-Id: I04b838b0c9fc9cf469b814223b43f5bbcde9860e

cmd_pull: add --all cmdline option

Updates all remote-tracking branches that have identical name in the
remote.

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

Merge "Implement pull subcommand" into devel

Merge "Implement clone subcommand" into devel

Implement pull subcommand

For updating a git repository representing a package managed with gbs.

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

Implement clone subcommand

For cloning a git repository representing a package managed with gbs.

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

Only check permission if outdir exists

- raise error if make dir fails
- only check permission of outdir if outdir exists

Change-Id: Id86dee7093d38bf8861d88bcf299cb902fac6da2

fix misleading error handling while no spec file found

Change-Id: I7be2d57cd5c9e1c2756686cb9f43d4eb1a998fd0

Arch clean up, #654

Remove unsupportted arches, and only leave x86_64, i586, armv7l,
armv7hl, armv6l. ia32,i686,i386 are deprecated, but can be supportted
in gbs 0.13, and remove totally start from gbs 0.14

Update docs/GBS.rst, and use i586 for all examples and logs.

Change-Id: Ib7e936e5169cf725d40570da3e125a412eea3fde

Re-orgnize option help info of build module, #595

- Clean up examples
- Only five group left:
  * build configuration options
  * build env options
  * speed up building options
  * git-tree options
  * package selection options

Change-Id: Iba7ba18a990f188a7d58b8258d7bddb87b3d7658

Check write permission of output dir to avoid traceback, #617

Change-Id: I8c343279b1d7791016c12aff2310e4c2d28c5443

Add --no-patch-export options for export/build module, #594

with --no-patch-export, gbs does not create patches between
upstream and export-treeish, and and create tar ball from the
export-treeish instead of upstream branch

Change-Id: Iabad13eb75c425a104f20f507ae265801ab6aa3b

add more import options, #594

--filters for filtering out e.g. scm directories
only for srpms and spec file options:
 --allow-same-version for re-importing the same version
 --native option for telling that "we're" the upstream
 --no-patch-import to prevent trying of importing patches
only for upstream tar ball options:
 --upstream-vcs-tag: setting a upstream git tag as the parent.

Change-Id: Ib90a0520e027f3e7df777e870e1172beb9e6e9d7

remove checking packaging dir in export module, fix #635

packaging dir will be checked while guessing spec file.

Change-Id: I31409b41372018637bbf1bba608dfff84805ebe9

Clean up build options and group them, #595

Change-Id: I39494c869d1e9fcaa1fa7b89b1ab4e8b86e59eee

merge docs from release-0.12 to devel branch

Change-Id: I69de1536457f4d39a2a23d3cecaf0421aca851b0

use lstrp() to strip prefix '/'

Change-Id: I4106d71e3a602daee8f09545b709ebb0c09f4e8a

Removed duplicated code

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

Removed unused code

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

Configure coverage to cover only gbs modules

Previous attempt to configure coverage with setup.cfg was not working
properly. It can be seen in OTC Tools Tester Jenkins job as coverage
report includes gbp and even nose modules. This change is the second
attempt to fix the issue. Expected result is to see only gbs modules in
Jenkins coverage report.

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

Code cleanup

Fixed pylint findings, restructured code, removed unused APIs.

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

cmd_export: fix a variable name clash

Which causes the wrong upstream branch name to be used.

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

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