Bug#342774: [patch] dpatch: new DPEP_OMIT_TIMESTAMPS
tags 342774 + pending thanks I applied the patch, and modified dpep to accept a -n|--notimestamp option aswell. Also documented it on the manual page. It will be part of the next upload. -- |8] -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#342774: [dpatch-maintainers] Bug#342774: [patch] dpatch: new DPEP_OMIT_TIMESTAMPS
Hi, As a sort of followup to #321320, I'd like the ability to tell dpatch-edit-patch not to put timestamps on patch header lines at all. They're not very useful, and as noted in the bug log for #321320, they create extra interdiff noise every time you run dpep. The following patch creates the trivial option variable DPEP_OMIT_TIMESTAMPS which goes into effect if set (normally set in 00options). I don't know if a corresponding command line option is useful. I'll add one if you think so. As with #342768, I'll wait until you indicate that you are interested in applying this patch, before patching the manpage and adding a test. This feature sounds much better, and fixes some drawbacks. If it's going to work with every 'patch' version, that would be ideal, if not, well, making it configurable would be a nice option. regards, junichi -- [EMAIL PROTECTED],netfort.gr.jp} Debian Project -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#342774: [patch] dpatch: new DPEP_OMIT_TIMESTAMPS
Package: dpatch
Version: 2.0.16
Severity: wishlist
Tags: patch
As a sort of followup to #321320, I'd like the ability to tell
dpatch-edit-patch not to put timestamps on patch header lines at all.
They're not very useful, and as noted in the bug log for #321320, they
create extra interdiff noise every time you run dpep.
The following patch creates the trivial option variable
DPEP_OMIT_TIMESTAMPS which goes into effect if set (normally set in
00options). I don't know if a corresponding command line option is
useful. I'll add one if you think so.
As with #342768, I'll wait until you indicate that you are interested
in applying this patch, before patching the manpage and adding a test.
Thanks,
Peter
--- orig/dpep/dpatch-edit-patch
+++ mod/dpep/dpatch-edit-patch
@@ -288,6 +288,7 @@
diff -urNad $DPEP_DIFF_EXCLUDE . $WORKDIR/$(basename $DPEP_SOURCEDIR) |
sed -e /^--- \|^diff /s: \./: $(basename $DPEP_SOURCEDIR)~/: \
-e /^+++ \|^diff /s: $WORKDIR/: : \
+ -e /^--- \|^+++ /s:${DPEP_OMIT_TIMESTAMPS+\t[^\t]*}\$:: \
-e /^diff /s: $DPEP_DIFF_EXCLUDE:: $DIFFHOLDER || true
# Diff created, let's switch back to the original directory, and start the
signature.asc
Description: Digital signature
