summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* documentation: typo suggestions from Paul HardyPaul Hardy2019-03-01
| | | | Signed-off-by: Paul Hardy <unifoundry@gmail.com>
* dgit: import-dsc: New test for abs/rel dsc component linksIan Jackson2019-03-01
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: import-dsc: Handle relative symlinks correctlyIan Jackson2019-03-01
| | | | | | | | | The logic here was correct only with bpd = `..'. Closes: #913259 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk> squash!! dgit: import-dsc: Handle relative symlinks correctly
* dgit: cmd_import_dsc: comment on lack of is_orig_file checkIan Jackson2019-01-13
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n: nl.po: Remove a spurious spaceIan Jackson2019-01-13
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: start 6.4~Ian Jackson2019-01-07
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: finalise 8.3debian/8.3archive/debian/8.3Ian Jackson2019-01-07
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: preparing 8.3Ian Jackson2019-01-07
| | | | | | gbp dch and significant manual editing. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: Mention manually that we close #917194.Ian Jackson2019-01-06
| | | | | | We'll dedupe the entry when we tidy up after running gbp-dch. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n-commit - autogeneratedIan Jackson2019-01-06
|
* i18n: Dutch message translationsFrans Spiesschaert2019-01-06
| | | | | | dgit: [INTL:nl] Dutch po file for the dgit package Signed-off-by: Frans Spiesschaert <Frans.Spiesschaert@yucom.be>
* i18n: Unfuzz the dgit-user(7).nl manpageIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit-user(7): Fix formatting error in comment about multi-archIan Jackson2019-01-06
| | | | | | These lines should not be indented. That makes them format oddly. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n: dgit-user_7.nl.poFrans Spiesschaert2019-01-06
| | | | | | | | | dgit: [INTL:nl] Dutch po file for the dgit-user_7 manpage [ file copied into right place and po4a.cfg updated -iwj ] Closes: #918253 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* i18n: Fix pairwise-pocheckIan Jackson2019-01-06
| | | | | | | | The condition to spot only bare < did not work. Fix it. Tcl does not have look-behind assertions, so use . and a look-ahead assertion. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: When reporting no such package, say `source package'Ian Jackson2019-01-06
| | | | | | This may help a bit pending a fuller fix to #844206/#870496. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: dotdot_bpd_transfer_origs: Change if to early returnIan Jackson2019-01-06
| | | | | | This drops an indentation level. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Test that we can copy an orig from bpdIan Jackson2019-01-06
| | | | | | | Manually move just the .orig from the bpd to `..'. Then check that when we are done they are (links to) the same file. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Transfer origs from .. to bpd when fetchingIan Jackson2019-01-06
| | | | | | | | | | | | | | | | | | | | | | | | This is the second half of #904878. When fetching we want to avoid downloading once again origs that are already in `..', but not in bpd. So call the facility we now have for making clones of things from `..' to bpd. `..' might contain various junk. We try to treat it with a bit of suspicion. In particular, when we have a dsc we know exactly which orig files we are looking for. So only try to link those. We don't do this transfer for debian tarballs or diffs, or for origs that would be related according to their filenames, but aren't in the dsc. This maximises the probability that fetch will succeed even if there is garbage in the `..'. (Garbage in the `..' will sometimes unavoidably cause source builds to fail, because when doing a source build we don't know what origs to expect.) Closes: #904878 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Transfer origs from .. to bpd when buildingIan Jackson2019-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | This is apropos of #904878. When building we need to consider anything in .. that would be (accordingg to its filename) a possible orig for the package we are building, since we do not know what origs there are supposed to be, and missing ones generate unhelpful error messages. We `copy' things from .. to bpd, rather than just trying to use them directly from .., because otherwise lots of other things won't work right. For example, we might generate .dsc's in the bpd which expect the files alongside, but without the files. But we don't actually copy any files. We prefer to: * If the thing in .. is a symlink, copy the link text, adjusting it into an absolute link. * If the thing in .. is a file, try to hardlink it, or failing that make a symlink to it. The result is that we never duplicate the file contents, but still, where possible, we prefer to make the thing in bpd stand alone so that if the `copy' in .. is deleted everything is still fine. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Provide dotdot_bpd_transfer_origs for using origs from ..Ian Jackson2019-01-06
| | | | | | No callers yet so no functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Remove foolish use of $b in quilt_fixup_multipatchIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Remove foolish uses of $b in unpack_playtree_linkorigs etc.Ian Jackson2019-01-06
| | | | | | | Evidently I had forgotten that $b is weird in Perl and should not by used for things other than sorting. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: generate_commits_from_dsc: Move up $upstreamvIan Jackson2019-01-06
| | | | | | We will want this in a moment. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: generate_commits_from_dsc: Factor out $bpd_absIan Jackson2019-01-06
| | | | | | | | And add a newline. We will want this in a moment. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: Test --clean-git[-ff],alwaysIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Document new --clean=git behavioursIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean: Provide -wg[f]a short aliasesIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean: Provide new --git[-ff],always clean modeIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: cleaning: Improve handling of note about ignoresIan Jackson2019-01-06
| | | | | | | | | | | | | | * Provide a new $ignmessage parameter to clean_tree_check_git and use that. This makes it a bit easier to: * Only print the additional message if the clean mode actually honoured ignores. If it didn't then the problem is not `due to missing .gitignore entries' because they would not have helped. * Change all callers other than clean_tree_check_git_wd to pass the empty string, so no other overall functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean: Do check the tree even when building in playtreeIan Jackson2019-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we might be doing quilt fixup, then the quilt fixup's attempt to merge back into master might fail if there are uncommitted debian/patches. So it is sometimes wrong to just not clean at all. We don't want the behaviour to depend on the source package format, and ideally not on the clean mode. Also, the user may have forgotten to `git add', in which case they will thank us for spotting their mistake. In the original design table in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=910705#56 says this for this case /: disregarded would be fine to delete but ** better to trip ? ** ** want opt to disregard ** but this was not implemented at the time. So implement that now. I think I have concluded that with -wg and -wgf the right way to turn these new failures into successes is to have a way to have the clean actually done. If that is not wanted, one can say -wn instead. This is particularly true given that ignoring the problem can produce confusing failure, as seen in #914317. So there will be a new --clean=git[-ff],always shortly. Closes: #914317 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: exmaplegit: Do not leave the tree dirtyIan Jackson2019-01-06
| | | | | | | | This test setup facility passed a dirty working tree to all of its consumers. That was not really intended or useful, and is about to start causing trouble. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean mode: Refactor -wg parsing to prep for suffixesIan Jackson2019-01-06
| | | | | | | | This will make it convenient to add new suffix letter(s) to -wg. No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean: Prepare cleanmode_re for suffixes on git and git-ffIan Jackson2019-01-06
| | | | | | No functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clean: Change matcbing of git to regexpsIan Jackson2019-01-06
| | | | | | | | | We are going to have some ,... suffixes on this, so we need more general matching. No functional change with the current set of clean modes. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit(1): Fix a formatting typoIan Jackson2019-01-06
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* Dgit.pm: Allow uppercase (ascii) letters in multi-orig componentsIan Jackson2019-01-06
| | | | | | | | | | | | | | We didn't recognise files like firefox-esr_60.4.0esr.orig-l10n-sv-SE.tar.bz2 as orig files. As a result, dgit would see all the things in the corresponding subdirectory as having been added in Debian changes, leading it making inaccurate complaints and bombing out. See also #918438 against policy, where I (effectively) argue that this is a design error in the `3.0 (quilt)' format. Closes: #916926 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit-repos-policy-debian: Remove duplicated text from force messageIan Jackson2019-01-06
| | | | | | | | | $ovwhy is from SELECT deliberately ... WHERE ... OR deliberately = ? where ? is a key in %deliberately which came from the command line where it already contains --deliberately=. Closes: #913676 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Check that the build-products-dir is accessibleIan Jackson2019-01-05
| | | | | | | | In fetch/clone, and build. Others are less likely to be an irritating problem. Closes: #913648. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: clone: Create destination directory before using networkIan Jackson2019-01-05
| | | | | | | | | We are going to want to check the bpd (apropos of #913648), which might be a relative path. We should fail quickly in this case. No significant functional change. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit-maint-debrebase(7): handle DFSG-filtering for a new packageSean Whitton2019-01-05
| | | | | Closes: #915973 Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
* dgit: Pass --no-source-only-changes to pbuilder and cowbuilderIan Jackson2019-01-05
| | | | | | | | | | | This will break with older versions of pbuilder and cowbuilder, unfortunately. That breakage can be worked around by passing dgit one or both of the new options: --pbuilder!:--no-source-only-changes --cowbuilder!:--no-source-only-changes Closes: #904862. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit: Provide --program!:optionIan Jackson2019-01-05
| | | | | | Apropos of #904862. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* test suite: pretend-pbuilder: Reject unknown optionsIan Jackson2019-01-05
| | | | | | The failure to do this was simply a bug. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit(1): Fix reference to -wdn/-wddn in a message.Ian Jackson2018-11-28
| | | | | | It's not -wdu/-wddu. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit(1): Fix documentation of .quilt-mode configIan Jackson2018-11-20
| | | | | | | | | In 0457c2d771ef6fe6c0883344add9dd4da0c60917 dgit: Honour new .clean-mode-newer access config option this was accidentally deleted. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* po/README: Fix reference to dgit-user_7.potIan Jackson2018-11-12
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: start 8.2~Ian Jackson2018-11-08
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* changelog: fianlise 8.1debian/8.1archive/debian/8.1Ian Jackson2018-11-08
| | | | Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
* dgit-maint-debrebase(7): Minor corrections to #903377 wordingIan Jackson2018-11-08
| | | | | | | | | | | Do not suggest trying without --overwrite first, since if you were going to pass it anyway you may as well do so right away. And mention that git-debrebase will normally prompt you to say dgit --overwrite. Closes: #903377 Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>