summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* test: Handle alternative QUILT_PATCHES valuesJean Delvare2017-05-093-8/+8
| | | | | | | | | | When using an alternative QUILT_PATCHES value, we can't use %{P} in the test case, because P is set in the Makefile before the test case has a chance to redefine QUILT_PATCHES. Instead we have to make the prefix optional in the output of the quilt commands, so that the test will succeed regardless of QUILT_PATCHES_PREFIX being set or not. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* test: Consistently use %{P} and %{_P}Jean Delvare2017-05-0933-352/+352
| | | | | | | | | | | There is no point in defining P and _P in the Makefile to allow for QUILT_PATCHES_PREFIX to be set or unset if we do not use these variables consistently in the test suite. Replace hardcoded "patches/" with %{P} or %{_P} throughout the test suite wherever relevant so that unsetting QUILT_PATCHES_PREFIX in test.quiltrc actually works. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* test: Drop incorrect uses of %{P}Jean Delvare2017-05-096-11/+11
| | | | | | | | | %{P} only makes sense when checking the output of a quilt command. The value of QUILT_PATCHES_PREFIX can obviously not affect what the user is writing on the command line, nor the output of other commands. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* Fix Makefile when QUILT_PATCHES_PREFIX isn't setJean Delvare2017-05-091-2/+2
| | | | | | | | | | | | Unsetting QUILT_PATCHES_PREFIX in test/test.quiltrc would trigger a syntax error on "make check": /bin/bash: -c: line 3: syntax error near unexpected token `;' /bin/bash: -c: line 3: `; \' Makefile:410: recipe for target 'test/.add-filename-check.ok' failed make: *** [test/.add-filename-check.ok] Error 1 Signed-off-by: Jean Delvare <jdelvare@suse.de>
* diff/refresh: Add missing quotes in basename callJean Delvare2017-05-022-1/+56
| | | | | | | | | | The basename call in function diff_file would fail if any component of the path has a space in its name. This fixes bug #50862: https://savannah.nongnu.org/bugs/?50862 Signed-off-by: Jean Delvare <jdelvare@suse.de>
* header: Fix patch corruption when trailing newline is missingJean Delvare2017-05-022-0/+20
| | | | | | | | | | | | When editing a patch header with certain editors, it is possible that no trailing newline character is present on the last line. In such case, the patch would be corrupted. Prevent that by ensuring that a newline character is always present. This fixes bug #50841: https://savannah.nongnu.org/bugs/?50841 Signed-off-by: Jean Delvare <jdelvare@suse.de>
* Run tests again if test.quiltrc changesJean Delvare2017-05-021-2/+2
| | | | | | | If any change is made to the test environment, we want to run the checks again, to catch any problem early. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* test: Fix /bin/sh filteringJean Delvare2017-04-251-1/+1
| | | | | | | /bin/sh may not include a line number in its error messages, fix the filtering when it does not. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* test: Escape curly braces in regexJean Delvare2017-04-251-2/+2
| | | | | | | | | | Curly braces in perl regex are supposed to be escaped, recent versions of perl complain when they aren't: Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/%{ <-- HERE (\w+)}/ at ./run line 114. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/%{ <-- HERE \?}/ at ./run line 290. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* Run the tests again if test runner changesJean Delvare2017-04-251-2/+2
| | | | | | | If any change is made to the tester script, we want to run the checks again, to catch any problem early. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* quilt.el: don't strip directory in quilt-top-patch as it could be a part of ↵leo2017-04-251-3/+2
| | | | patch name
* quilt.el: replace deprecated toggle-read-only function call with read-only-modeleo2017-04-251-3/+3
|
* quilt.el: define quilt-edit-top-only before it's usedleo2017-04-251-1/+1
|
* test: Allow keeping the working directory on failureJean Delvare2017-04-141-2/+3
| | | | | | | | Add an option to keep the working directory after a test has failed. Also include the test name in the working directory name to make it easier to figure out which test failed later. Signed-off-by: Jean Delvare <jdelvare@suse.de>
* mail: Fix patch set threadingJean Delvare2017-04-141-1/+1
| | | | | | | | | | | | | | Let patches reference the introduction message so that the patch set is properly threaded by MUA. This used to work but was broken by accident. This fixes bug #50775: https://savannah.nongnu.org/bugs/index.php?50775 Signed-off-by: Jean Delvare <jdelvare@suse.de> Fixes: 1d659bebaf3d ("quilt/mail: Remove procmail dependency") Tested-by: Okash Khawaja <okash.khawaja@gmail.com> Acked-by: Kent R. Spillner <kspillner@acm.org>
* configure: md5sum is optionalJean Delvare2017-03-101-2/+2
| | | | | We only need md5sum for the setup command, like rpmbuild and 7z. So treat it as optional.
* setup: Add basic support for p7zip (.7z) archivesJean Delvare2017-03-104-2/+47
| | | | | | | | | | | | | | This should work in the most basic case (no option used.) This only covers the case of archives being compressed with 7z, not patches. This should fix bug #49754: http://savannah.nongnu.org/bugs/?49754 Remaining issues: * 7z could also be used to compress patches, although nobody sane would do that. * 7z is very verbose, and doesn't offer any option to be silent or even just quieter.
* patch-wrapper: Make the patch names that quilt shows more convenientAndreas Gruenbacher2016-12-081-14/+31
| | | | | | | | | When operating on an rpm package, create links to the rpm sources and build directories and make patch names relative to these links. This is more convenient than using ugly absolute paths. This patch has been in the SUSE quilt package since February 2006, it seems to be about time to get it upstream.
* Makefile: Add updatepo targetJean Delvare2016-11-151-1/+3
| | | | | | Add a target "updatepo" which translators can run to update the translation files. Then they can look for strings to translate and send us a patch or the updated translation file(s).
* Generate guards man page in doc/Jean Delvare2016-11-152-4/+4
| | | | | For consistency, the guards manual page should be generated under doc/, not bin/.
* Set version to 0.65.v0.65Jean Delvare2016-11-093-10/+41
|
* Update German translationHolger Wansing2016-11-081-3/+4
|
* Update Russian translationSergey Basalaev2016-11-081-3/+4
|
* Resync translation filesJean Delvare2016-11-072-20/+26
|
* Update the Japanese translation for Quilt 0.65Yasushi SHOJI2016-11-041-14/+19
|
* Update the French translationJean Delvare2016-11-032-21/+27
|
* mail: Describe the --charset option betterJean Delvare2016-11-031-1/+3
|
* Update German translationHolger Wansing2016-10-241-8/+11
|
* Update Russian translationSergey Basalaev2016-10-241-4/+6
|
* Update the French translationJean Delvare2016-10-211-1/+7
|
* Resync translation files + trivial updatesJean Delvare2016-10-215-72/+100
|
* bash_completion: Improve the handling of the push commandJean Delvare2016-10-211-1/+7
| | | | Some options need a parameter, invite the user to provide it.
* bash_completion: Add support for command "new"Jean Delvare2016-10-211-1/+11
| | | | Command "new" accepts options now, add support for them.
* bash_completion: Improve the handling of the mail commandJean Delvare2016-10-211-1/+14
| | | | Some options need a parameter, invite the user to provide it.
* bash_completion: Add new command optionsJean Delvare2016-10-211-15/+17
| | | | | Enable completion with the new options of commands diff, fold, import, mail, patches, pop, push, refresh, series and setup.
* bash_completion: Update the list of commandsJean Delvare2016-10-211-1/+1
| | | | Command "upgrade" was missing.
* push: Fix the synopsisJean Delvare2016-10-211-1/+1
| | | | Add missing options -m and --fuzz to the synopsis.
* mail: Fix the help textJean Delvare2016-10-211-1/+4
| | | | | Add options --charset and --signature to the synopsis, and describe the --charset option.
* bash_completion: Drop wrapping conditionalJean Delvare2016-10-211-4/+0
| | | | | | Drop the check for quilt being present. No other completion file has such a check, because it's useless. If the program isn't installed then its completion file isn't installed either.
* bash_completion: Drop duplicate functionsJean Delvare2016-10-211-66/+1
| | | | | | | | | | Completing long options of the grep command is not a fundamental feature. I can't remember ever needing it. The most useful grep options are short ones by design. Additionally, our internal implementation of _longopt is broken as far as I can tell. So drop our implementations of _expand, _filedir and _longopt, and only call _longopt if it is already available from bash-completion.
* bash_completion: Use _quilt_comfile for setupJean Delvare2016-10-211-3/+3
| | | | | Add support for directories to _quilt_comfile, and use that for the setup command. The drops the last direct dependency to _filedir.
* bash_completion: Use _quilt_comfile for diffJean Delvare2016-10-211-1/+1
| | | | | All other commands use _quilt_comfile so do the same for diff for consistency.
* bash_completion: Handle spaces in file namesJean Delvare2016-10-211-11/+12
| | | | | | | | Set IFS in _quilt_comfile so that file names containing spaces are properly handled. Also quote ${COMPREPLY[@]} everywhere so that such file names are preserved. This closes Debian bug #640551.
* bash_completion: Delete dead codeJean Delvare2016-10-211-6/+3
| | | | | | | | | | You can't change the caller's environment from the right hand side of a pipe. So that code was doing nothing, which means we can just delete it. What the code was supposedly doing, is actually implemented by complete's option "filenames". So move the comment to where this option is set, to clear the confusion.
* Update the French translationJean Delvare2016-10-211-1/+1
|
* Resync translation filesJean Delvare2016-10-215-135/+155
|
* Make one error message translatableJean Delvare2016-10-211-1/+1
|
* Fix a race condition in diff_fileJean Delvare2016-10-191-5/+7
| | | | | | Fix a race condition where we may check the return status of diff before the command has actually terminated. Make sure that we read all of diff's output to ensure this can't happen.
* diff: Report diff failuresJean Delvare2016-10-193-1/+2
| | | | | "quilt refresh" reports diff failures to the user. "quilt diff" should do the same.
* Reject binary files in patchesJean Delvare2016-09-151-1/+7
| | | | | | Since diffutils version 3.4, diff no longer returns an error code for binary files. Parse the first line of the output to detect this case and raise our own error.