summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* changelog: Write and finalise for 9.6debian/9.6archive/debian/9.6Ian Jackson2019-07-25
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* git-playtree-create: Drop an unwanted set -xIan Jackson2019-07-25
| | | | | | Closes: #932959 Reported-by: Sean Whitton <spwhitton@spwhitton.name> Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* debian/control: Drop now unused dependency on libwww-perl.Ian Jackson2019-07-23
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit(1): Drop mention of LWP::UserAgent since we don't use it nowIan Jackson2019-07-23
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Abolish url_get in favour of url_fetchIan Jackson2019-07-23
| | | | | | This removes use of LWP from dgit. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Use libcurl, not @curl, for git checkIan Jackson2019-07-23
| | | | | | | | | | This means that when we sort out our TLS problems this will be fixed too. The remaining use of command-line curl is just to download additional .dsc compoments. I intend to leave that as-is. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: url_fetch: Honour new CurlOptsIan Jackson2019-07-23
| | | | | | No functional change with the existing callers. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: url_fetch: Honour new AccessBaseIan Jackson2019-07-23
| | | | | | No functional change with the existing callers. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Reorganise url_fetchIan Jackson2019-07-23
| | | | | | | | | This was archive_api_query_curl. We're working on making it good for other things too. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: libcurl usage: Install SIGPIPE handler ourselvIan Jackson2019-07-23
| | | | | | | | | | We don't want a global SIGPIPE setting. In particular we do not want to run any of subprocesses with SIGPIPE ignored. Just in case libcurl gets this wrong, tell it CURLOPT_NOSIGNAL, and do it ourselves. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Set $SIG{INT} to DEFAULTIan Jackson2019-07-23
| | | | | | | | | | | | | | There is a bug in libdpkg-perl, where Dpkg::Source::Package installs a handler on loading. This has the effect of making in-process C calls (eg to libcurl) uninterruptible by ^C. Work around it by unconditionally resetting SIGINT. (Anyone who deliberately invoked us with SIGINT ignored deserves to keep all the resulting pieces.) See #932841. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: archive_api_query_curl: Add a bit of debugIan Jackson2019-07-23
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Use WWW::Curl rather than invoking curl(1) for archive apiIan Jackson2019-07-23
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Drop support for old attempts at cert/key pinningIan Jackson2019-07-23
| | | | | | | | | | | I don't think anyone is using these. We are going to do something different to try fo fix #932570. When #790093 is fixed we may need something like archive-query-tls-key (or even exactly that) but we would have to reimplement it for WWW::Curl anyway. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* tag2upload: Use the non-readonly method for dgit fetchIan Jackson2019-07-23
| | | | | | | The principal effect in the Debian configuration is to access the master dgit git server via ssh, rather than the mirror via https. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Provide --for-push optionIan Jackson2019-07-23
| | | | | | | This is much more conveient and in some cases sensible than -cdgit-distro.DISTRO.readonly=0 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Make cmd_archive_api_query call api_query_rawIan Jackson2019-07-23
| | | | | | | | | | | | In support of #932570. Now there is only one caller of archive_api_query_cmd. It is OK to pass `undef' because - archive_ No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Move $data check into api_queryIan Jackson2019-07-23
| | | | | | | In support of #932570. Our intended other caller doesn't have a $data. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Break out api_query_rawIan Jackson2019-07-23
| | | | | | In support of #932570. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Add missing Closes of #932476 to changelog entry for 9.5Ian Jackson2019-07-23
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: start 9.6Sean Whitton2019-07-22
| | | | Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* changelog: finalise 9.5debian/9.5archive/debian/9.5Sean Whitton2019-07-22
|
* changelog: Bug close syntax fixSean Whitton2019-07-22
| | | | Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* changelog: Minor fixesIan Jackson2019-07-22
| | | | | | Mention two omitted bug Closes. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Mention Vcs-Git changeIan Jackson2019-07-22
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: UpdateSean Whitton2019-07-22
| | | | Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: Check for a detached head when pushing HEADSean Whitton2019-07-22
| | | | | | | See discussion in #932612. Suggested-by: Ian Jackson <ijackson@chiark.greenend.org.uk> Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: Check for unstitched git-debrebase branchSean Whitton2019-07-22
| | | | | | Closes: #932612 Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* debian/control: Name salsa as our Vcs-*Ian Jackson2019-07-22
| | | | | | That is mostly what we are using now. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Wrap a line and mention HACKING.Ian Jackson2019-07-22
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: UpdateSean Whitton2019-07-22
| | | | Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: Check that patches are (un)applicableSean Whitton2019-07-22
| | | | | | Closes: #932477 Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: check_treesame: Show diffstat when there is a diffSean Whitton2019-07-22
| | | | Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: check_treesame: Also pass --quiet to 'git diff'Sean Whitton2019-07-22
| | | | | | | Otherwise, when using git-debpush in a terminal, the user will get a pile of unwanted diff output each time check_treesame is called. Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* HACKING: First cutIan Jackson2019-07-22
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* git-debpush: Do not spuriously print -sn/-skIan Jackson2019-07-22
| | | | | | | | When looking for -sn or -sk in debian/source/options, we need to not print it to stdout even if grep finds it. So use -q. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk> Acked-by: Sean Whitton <spwhitton@spwhitton.name>
* git-debpush: use -m rather than -F- <<EOFIan Jackson2019-07-22
| | | | | | | | | | | | | | This means that a run with set -x shows the tag data that is being signed. The user can even then try it again. It will make it easier to do some kind of dry run, too, if we want to. We are really not in danger of hitting a command line length limit. (It also involves fewer syscalls, less use of /tmp etc., although that is really irrelevant here.) Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk> Acked-by: Sean Whitton <spwhitton@spwhitton.name>
* dgit clone: No longer create an "origin" remoteIan Jackson2019-07-22
| | | | | | | | | | | | | | | | | | | | | This was a normal git remote pointing to the dgit git server. This is not really sensible because 1. it can't simply be pushed to 2. sometimes (if the package was never pushed with dgit) it doesn't even exist 3. it may be out of date with respect to the archive The effect of 2 is that it can break `git remote update'. This "git remote add" was introduced in 2013 in 39c6c123ab730d42ec7c9ed01c30e0175c7691e7 fix branch usage and was first included in 0.2. There doesn't seem to be any evidence of me having deeply considered the merits of this at the time. Closes: #932694 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Rename debian/3-1 tag to test-dummy/ in a test gitIan Jackson2019-07-22
| | | | | | | | | | Our distro is test-dummy. We are going to stop randomly fetching tags from the dgit git server, so the debian/ one would be omitted. Replace the tag with a fresh equivalent one and change the one use site. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* git-playtree-setup: Rename from git-playtree-createIan Jackson2019-07-21
| | | | | | | The name is from when I thought it would actually mkdir. But it doesn't. No overall functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* git-playtree-create: Fix with tests/using-intreeIan Jackson2019-07-21
| | | | | Reported-by: Sean Whitton <spwhitton@spwhitton.name> Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* copyright: credit Genome Research LimitedMatthew Vernon2019-07-21
| | | | | | | | | I work for the Wellcome Sanger Institute, but am employed by Genome Research Limited, so it is they who should be credited in Copyright notices. Signed-off-by: Matthew Vernon <mv3@sanger.ac.uk> Closes: #932630
* git-debrebase(1): Make this manpage a less intimidating entry pointSean Whitton2019-07-21
| | | | | | | | | | | | | | | | | | | | | - Nicer short description right in the first section. - Don't use the term 'delta queue' until it's needed. - Don't talk about other manpages so early. - Cater for people who type `man git-debrebase` looking for a barebones tutorial, not (just) a command line reference. - That tutorial should minimise references to dgit, especially since users might want to use git-debpush to upload. Thanks to Sam Hartman for the user feedback which prompted this and Ian Jackson for feedback on the content of "QUICK REFERENCE". Closes: #926656 Signed-off-by: Sean Whitton <spwhitton@spwhitton.name> Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* Dgit.pm: Abolish now-unused $t_local_git_cfg playtree_setup argIan Jackson2019-07-21
| | | | | | | | | | | This was there so that dgit's -c options would be honoured here. But now, we get this information via @git, which dgit adds -c options to, which we pass to git-playtree-create by invoking it via git. So this is no longer used and can be abolished. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* git-playtree-setup: Provide to git-debpushIan Jackson2019-07-21
| | | | | | | | Not used yet, so no overall functional change to the script. The .deb has a new file. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk> Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* git-playtree-setup: Rewrite in shell and call it from PerlIan Jackson2019-07-21
| | | | | | | | | | We want this because git-debpush is going to want this functionality but doesn't want to include (or have a copy of) Dgit.pm. The installation arrangements are rather ugly. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk> Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* Dgit.pm: Provide @gitIan Jackson2019-07-21
| | | | | | | | | This clears the way for some parts of Dgit.pm to honour dgit's git invocation conventions. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n-commit - autogeneratedIan Jackson2019-07-21
|
* git-debpush: Check upstream source is identical in the upstream tagSean Whitton2019-07-20
| | | | | | Closes: #932476 Signed-off-by: Sean Whitton <spwhitton@spwhitton.name> Acked-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit-maint-pbo(7): Add to .gitignoreIan Jackson2019-07-20
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>