push: Make --debian-tag='' match the documentation

An empty Debian tag indicates "don't care" about the packaging branch.
Don't fail in that case but rather push out up to the branch tip:

  Currently we'd fail like

  $ gbp push --debian-tag='' guido
  Traceback (most recent call last):
    File "/usr/bin/gbp", line 149, in <module>
      sys.exit(supercommand())
    File "/usr/bin/gbp", line 145, in supercommand
      return module.main(args)
    File "/usr/lib/python3/dist-packages/gbp/scripts/push.py", line 153, in main
      to_push['refs'].append((ref, get_push_src(repo, ref, dtag)))
  UnboundLocalError: local variable 'dtag' referenced before assignment

this make it simple to push the current development work via

  gbp  push --debian-tag='' <remote>

tests: Fix "test_push_failure" with newer git

The test created a broken tag on the remote side to avoid that being
pushed in the test however git 2.34 and newer fail every push to that
repo so rather create a non-ff branch instead to make the push fail.

HACKING: Document how to keep test repos around

HACKING: mention how to rerun a single test

uscan: Remove pre-devscripts 2.10.70 compat code

We can rely on the <target> element being present in dehs output.

The output it looks for from uscan no longer matches anyway.

config: Add missing comma

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

export-orig: Consistently refer to the pristine-tar branch

Since we’re checking pristine_tar_branch, not pristine_tar.branch,
use the same expression in the log even though it’s not supposed
to be different.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

import-orig: Don’t hardcode the pristine-tar branch

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

rpm: Avoid devscript dependency

The package is currently not available everywhere.

Document changes and release 0.9.25

tests: Make sure DEBEMAIL is set

Otherwise dch based tests fail

Closes: #998573

buildpackage: Map 'debian/latest' and 'debian/main' to sid too

This avoids having to specify a suite with more modern common branch
names.

Document changes and release 0.9.24

Use collections.abc.Iterable instead of collections.Iterable

since Python 3.10 dropped the alias.

d/control: Depend on dh-sequence-python3

This makes the requirement more obvious since dh-python does
multiple things

d/control: Add RulesRequiresRoot

d/control: Drop versioned python3 dep

Even oldoldstable is recent enough

d/rules: Drop trailing whitespace

Gbp-Dch: Ignore

d/changelog: Drop trailing whitespace

Gbp-Dch: Ignore

Add translation infrastructure

Make it simple to create/update the pot file and provide an initial
po file for de.

Fix import-dsc on empty repo if defaultBranch!=master

When importing a dsc on a empty repository when `ìnit.defaultBranch` is
set to something like `main` in `~/.gitconfig` we failed with:

    gbp:debug: ['git', 'update-ref', '-m', 'gbp: Import Upstream version 0.5.10.2', 'refs/heads/master', '95172aee8d1a4e6c69494e4f158f904111710168']
    gbp:debug: ['git', 'symbolic-ref', 'HEAD']
    gbp:debug: ['git', 'show-ref', 'refs/heads/main']
    gbp:debug: ['git', 'symbolic-ref', 'HEAD']
    gbp:debug: ['git', 'show-ref', 'refs/heads/main']
    gbp:debug: ['git', 'branch', '-m', 'None', 'debian/buster']
    gbp:error: Git command failed: Error running git branch: error: refname refs/heads/None not found
    fatal: Branch rename failed
    Traceback (most recent call last):
      File "/usr/local/lib/python3.9/dist-packages/gbp-0.9.22-py3.9.egg/gbp/scripts/import_dsc.py", line 519, in main
        commit = import_upstream(repo, sources[0], dsc, options)
      File "/usr/local/lib/python3.9/dist-packages/gbp-0.9.22-py3.9.egg/gbp/scripts/import_dsc.py", line 269, in import_upstream
        repo.rename_branch(repo.branch, options.debian_branch)
      File "/usr/local/lib/python3.9/dist-packages/gbp-0.9.22-py3.9.egg/gbp/git/repository.py", line 333, in rename_branch
        self._git_command("branch", args.args)
      File "/usr/local/lib/python3.9/dist-packages/gbp-0.9.22-py3.9.egg/gbp/git/repository.py", line 245, in _git_command
        raise GitRepositoryError("Error running git %s: %s" % (command, detail.decode().strip()))
    gbp.git.repository.GitRepositoryError: Error running git branch: error: refname refs/heads/None not found
    fatal: Branch rename failed

That's because we created the commit on `refs/heads/master` and then
tried to rename it from `refs/heads/main`.

To avoid that, if the repository is empty look at the name of the
current branch as reported by the symbolic ref pointed by `HEAD`.

Closes: #906600
Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

git: Actually include the missing key name in the KeyError message

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

dch: Require EMAIL or DEBEMAIL to be set

The alternative would be to set a dummy address in case of
--git-author but that again can use $EMAIL so just fail if
both are unset.

Closes: #987611

Document changes and release 0.9.23

spec: Fix rpm build on centos7 and centos8

spec: Ignore byte-compiled python files

docs: Check for docbook2man too

docs: Use either python3 or python

Some distros (such as Debian) don't ship a /usr/bin/python anymore

setup.py: Explicitly open the changelog file as utf-8

Fix gbp-builder-mock to honor configured options for mock

gbp-builder-mock: Update to work with current mock releases

spec: Swtich requires to python3 too

Thanks: Quanah Gibson-Mount

spec: Install setup-git-attributes command

This unbreaks the autopkgtest

setup.py: Allow for more liberal version numbers

Otherwise this causes trouble for backports, downstreams, etc.

setup.py: Avoid dpkg-parsechangelog

This helps on non-debian systems

repo_setup: fix documentation generation

docstring format for bullet lists requires a leading dash. This fix
prevents build failures when running `make docs`.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>

manpages: fix gbp-setup-gitattributes

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>

tests/11_test_dch_main.py: Fix OS release check for Ubuntu.

Closes: #983204

Add tests for setup-gitattributes functionality

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

Install the setup_gitattributes script

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

scripts: Make setup_gitattributes future-proof

* Add --all (default) and --[no-]dgit-defuse-attrs to enable
  adding more options in future.
* Rewrite the manual page to reflect the change in approach.
* Error out when running with no presets enabled

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

d/control: Capitalise Git throughout the package descriptions

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

d/control: Add the new subcommand to the package description

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

docs: Add a manpage for gbp setup-gitattributes

Describle how and why the attributes are being disabled, refer
to dgit, git-deborig and Git documentation; add a reference to
the gbp-clone manpage.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

docs: gbp-clone: Document --defuse-gitattributes

Describe concisely what the option does; a longer description
will go into a separate manpage.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

scripts: Add setup-gitattributes subcommand

The setup-gitattributes subcommands overrides Git attributes
unconditionally, without verifying first the package ships any
.gitattributes files.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

clone: Add --defuse-gitattributes

On clone, disable Git attributes that may interfere with building
packages shipping .gitattributes *iff* they ship those files.
Set to auto by default, can also be turned off completely or
can be performed unconditionally.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

repo_setup: Add check/setup_gitattributes

setup_gitattributes() sets up .git/info/attributes in a way to prevent
transformations from interfering with packaging, because the working tree
files can differ from the Git revision history (and from the source packages).

Similar functionality has been implemented by dgit and git-deborig, so we try
to stay compatible and re-use the name of the attribute macro. Since dgit doesn’t
disable export-subst and export-ignore, which may interfere with export-orig, we
add this on top the same way git-deborig does.

dgit used to have an older preset missing some of the attributes, so if
we find those, we disable them.

The user, should they choose so, still can re-enable the attributes
disabled in this way by adding more specific rules further in the
.git/info/attributes file.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

gbp.git: Add support for long listing format (with object sizes)

This is useful to be able to scan a tree for a specific file
but only act if it’s non-empty.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

gbp.git: Change list_tree to return an iterator

When working with huge trees, we want to avoid consuming large amounts
of memory just to throw away most of it almost immediately when we only
need one entry or a few of them.

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

tests: Match the correct files in filter tests

Verify we match at all by adding a positive test as well

clone: Don’t check out the Debian branch if it doesn’t exist

Signed-off-by: Andrej Shadura <andrew.shadura@collabora.co.uk>

dch: Add support for the --local=suffix option

Closes: #857370
Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Add info about --git-ignore-branch when not on branch

Closes: #956491

import-dsc: Delete NULs from dpkg-parsechangelog output before parsing

Commit messages may not contain NUL characters; in practice,
Debian changelogs sometimes do, usually as the result of
incorrectly used encoding for non-ASCII personal names.

As a safety net, delete all NULs from the output of dpkg-parsechangelog
before parsing it, so that they don’t get fed into Git or anything else
further in the processing.

Closes: #981340

zsh completion: update completion for "pull" and further new commands

Provide zsh completion for new commands:

* config
* export-orig
* import-ref
* pristine-tar
* push
* tag

While at it, also extend completion for `gbp pull`.

Signed-off-by: Michael Prokop <mika@debian.org>

Document changes and release 0.9.22

tests: Don't be fooled by 32bit userspace on amd64

This unbreaks salsa-ci

manpages: Fix typo in gbp-buildpackage-rpm.1

Drop maintainer scripts

They were needed for the oldstable -> stable upgrade

d/control: Bump supported python3 version

We specify current stable as a lower bound.

dh_missing: Ignore __pycache_

d/control: Switch to dh-compat 13

Update rpm component test submodule

rpm: Fix tags in rpm spec files

d/control: Require newer rpm

The version changed API and parsing strictness in multiple places,
don't try to cater for older versions.

tests: Honor GBP_TESTS_NOCLEAN in rpm tests too

rpm: Fix tags in rpm spec files

Patches are named similar to sources, counting from 0.

https://docs.fedoraproject.org/en-US/Fedora_Draft_Documentation/0.1/html/RPM_Guide/ch09s03s04.html

rpm: Open tempfile in text mode

rpm: Stop decoding strings

Fails with newer rpmlib

autopkgtest: Set %_arch

Seems to be needed on armhf.

Closes: #980256

Define %python_sitelib for autopkgtest

The rpm package no longer defines this, instead there is python3_sitelib
in /usr/lib/rpm/macros.d/macros.python3 provided by the
python3-rpm-macros package (not in Debian).

Document changes and release 0.9.21

tests: Fixup changelog test on Ubuntu

The keys in os-release differ from lsb-release

gitlab-ci: Use a more recent Ubuntu LTS

doctests/test_Changelog: Don't expect /etc/lsb-release

Closes: #976914

tests/11_test_dch_main.py: Don't expect /etc/lsb-release on Debian

import-ref: Fix help

We want an upstream version, not  path

docs: Correct "option" typo in gbp-buildpackage manpage

Closes: #972081

buildpackage: Fix tests and test arch override

buildpackage: Honor --git-arch for GBP_CHANGES_FILE

Closes: #941345

docs: Add example for exporting the source tree

See #969952

import-orig: Refactor vcs_tag_parent

This makes the "tag" variable just the tag.  This improves the
human-readability of the error message if the tag cannot be found.

import-orig: Refactor vcs_tag_parent

This eliminates an indentation level.

import-orig: Fix a comment typo

salsa-ci: Skip reprotest

It triggers a tess failure since a test can write to a
destinatino it should not be able to write too.

test_import_orig: Drop unused variable

This makes flake8 happy

test_import_orig: Fix typo

import-orig: Allow the post-unpack hook to filter tarball files

With post-unpack scripts is currently possible to filter out files that are
not needed from the upstream branch, however it is not possible to use it to
filter files that will end up in the orig file, and in some scenarios this
is not easily doable just using a filter list.

So, run the post-unpack hook just after unpacking and before repacking the
tar.

Add tests to verify this both when using with regular --filter option
and by itself.

Closes: #812721

import-orig: expose unpacked sources dir to the postunpack hook

Use an $GBP_SOURCES_DIR to expose the post-unpack sources dir to the hook.
Include a test that checks that all the expected env variables are set.

autopkgtests: Depend on @builddep@

Otherwise gbp is not able to build itself.

Closes: #968982
Thanks: Lukas Märdian for the detailed report

smoketest: Avoid writing to ..

Use a temporary directory to avoid 'allow-root'.

autopkgtest: Pass RPM macros on command line

This avoids the need to modify files in the home directory.

spec: Don't fail on empty cache dir

spec: Include the correct file

export_ref.py does not exist.

policy: Fix typos

Thanks: Andrius Merkys
Closes: #966130

docs: Clarify that only tarball creation is affected.

Thanks: Pierre Equoy

docs: Drop --git- from dch invocation

Thanks Pierre Equoy

Document changes and release 0.9.20

Make flake8 E741 happy

Closes: #964040