summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* changelog: finalise 9.7debian/9.7archive/debian/9.7Ian Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Document chages since 9.6Ian Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n-commit - autogeneratedIan Jackson2019-08-11
|
* dgit: vcs-git handling: Strip [...] a la Haskell TeamIan Jackson2019-08-11
| | | | | | | | | | The final syntax of this is not yet agreed but this regexp ought to do the right thing in all reasonable cases. (Ie, no [ ] inside the [ ].) See also #932696 against policy, which is discussing the spec. Closes: #932699 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: update-vcs-git: Use vcs_git_url_of_ctrlIan Jackson2019-08-11
| | | | | | So we strip -b fragments as we ought. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: update-vcs-git: Print a less confusing message if unchangedIan Jackson2019-08-11
| | | | | | | Previously, we would say it was "already configured" which is unspecific and sounds a bit like an error. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: update-vcs-git: Do not crash if url is unchangedIan Jackson2019-08-11
| | | | | | | Previously, we would pass an empty command array to runcmd, causing a mysterious-looking failure. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Break out vcs_git_url_of_ctrlIan Jackson2019-08-11
| | | | | | This should have been done before. No functional change so far. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Fix on NFS (avoid leaking an open file in a directory to rm)St├ęphane Glondu2019-08-11
| | | | | | | | | The file debian/source/format was open by "dgit fetch" but never closed, causing failure to remove the extracted tree on NFS. Closes: #933827 Signed-off-by: St├ęphane Glondu <glondu@debian.org> Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: alternating-dgit: New testIan Jackson2019-08-11
| | | | | | | | | | This tests alternating dgit and non-dgit uploads. These were broken by the bug #934126 (check_for_git always returns false), in the http rework, and not detected by the test suite (!) This test *does* break if check_for_git always returns false. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Use http git check for three existing testsIan Jackson2019-08-11
| | | | | | | | | | | | | | At the very least we want at least one test which fails when check_for_git is nobbled ad-hoc [1] to return 1. "clone-nogit" fits that bill (verified experimentally). Pick two other tests to enable this for, roughly arbitrarily. (Two tests which chain to clone-nogit are affected too.) Of course we also want a test which fails when check_for_git falsely returns 0. We don't have one right now. Coming up... Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: WWW::Curl: Double check that $response_body was setIan Jackson2019-08-11
| | | | | | | We are making some assumptions about how WWW::Curl behaves. This confess, with appropriate tests, should catch any future problems. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: WWW::Curl Set $response_body to '' explicitlyIan Jackson2019-08-11
| | | | | | | | | | | | | | | check_for_git calls url_fetch with a WWW::Curl parameter CURLOPT_NOBODY which suppresses saving the body. The result is that CURLOPT_WRITEDATA does not update $response_body. Then, even on success, url_fetch returns undef (which ought to mean 404). Strictly this is a bit sneaky of check_for_git. But, rather than trying to make this interface more formal, just make url_fetch set $response_body to ''. It does return this except in the success case, so the error cases are still handled right. Closes: #934126 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: examplegit: Call t-select-packageIan Jackson2019-08-11
| | | | | | | | | | This can save our importers from havig to t-select-package themselves. Drop the now-redundant call from the one importer that had it. Various of the importers call t-archive-* or t-git-* that also call t-select-package, but this is not a problem. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Break out t-non-dgit-uploadIan Jackson2019-08-11
| | | | | | No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: http-git-check: New provisionIan Jackson2019-08-11
| | | | | | | | | Importing this makes us use a local http server for the git check. It's a setup so that it can imply the http-git-check dependency. Right now nothing imports this so no functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Honour new DGIT_TEST_KEEP_MUSTCLEANIan Jackson2019-08-11
| | | | | | | | | | | FTR, using this typically results in runes like DGIT_TEST_KEEP_MUSTCLEAN=y tests/using-intree tests/tests/ftpmasterapi-http 2>&1 | tee tests/tmp/log hanging instead of exiting, because the http server is still running and its stderr is still going to tee. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Rename http-static-server from ftpmasterapi-Ian Jackson2019-08-11
| | | | | | | | We are going to use this for other things too. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* http[s] tests: Add http test caseIan Jackson2019-08-11
| | | | | | Maybe we will turn this into a TLS test case in due course. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* http[s] tests: Fix stdout handling in ftpmaster test serverIan Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* http[s] tests: More-or-less functional ftpmaster http serverIan Jackson2019-08-11
| | | | | | Not used anywhere yet. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* http[s] tests: Import and slightly hack example scriptIan Jackson2019-08-11
| | | | | | | | | | This is basically a copy of some of the example code from the HTTP::Server::Simple docs, hacked up with my own (so far) nonsense. Add copyright info to file and d/copyright. Upstream licence is Artistic or GPL-1+. We choose GPL-1+ and upgrade to GPL-3+. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: lib: Provide $tmp/must-clean directory to testsIan Jackson2019-08-11
| | | | | | | This is a directory for things which should be removed on exit. Currently there are none. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Slightly more debugging for check_for_git with http[s]Ian Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Fix a typo in the entry for 9.6Ian Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Document changes to enumerate-testsIan Jackson2019-08-11
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: enumerate-tests: Spot t-alt-testIan Jackson2019-08-10
| | | | | | | | And find the dependencies of the referred-to test. No functional change since currently no chained-to test has any nonstandard dependencies. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: enumerate-tests: gencontrol-add-things: restrictionsIan Jackson2019-08-10
| | | | | | No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: enumerate-tests: gencontrol-add-things: delim featureIan Jackson2019-08-10
| | | | | | No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: enumerate-tests: Break out gencontrol-add-thingsIan Jackson2019-08-10
| | | | | | No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Start 9.7Ian Jackson2019-07-25
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* 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>