Connor Smith suggests there might be a fencepost error that could be remedied with:
diff --git a/src/cset.c b/src/cset.c
index 27ba3d9ae..4841b2e56 100644
--- a/src/cset.c
+++ b/src/cset.c
@@ -755,7 +755,7 @@ doDiff(sccs *sc)
printf("\n");
return;
}
- e = PARENT(sc, e);
+ e--;
if (e == d) return;
sccs_diffs(sc, REV(sc, e), REV(sc, d), &dop, stdout);
}
… and offers this test case from bk’s own source:
cosmith@hilbert:~/tmp/work/bk-7.3.1ce$ bk changes -v -d'$if(:REPOTYPE:=product){$if(:CHANGESET:){${1=:REV:}}$else{$if(:MERGE:){ChangeSet@$1\t:GFILE:@:REV:\n}}}' | head
ChangeSet@1.2869 src/Makefile@1.552
ChangeSet@1.2869 src/utils/Makefile@1.67
ChangeSet@1.2859 src/slib.c@1.1073
ChangeSet@1.2858 src/csetprune.c@1.122
ChangeSet@1.2858 src/fast-export.c@1.17
ChangeSet@1.2858 src/ndiff.c@1.35
ChangeSet@1.2858 src/sccs.h@1.867
ChangeSet@1.2858 src/slib.c@1.1072
ChangeSet@1.2831.1.2 src/ndiff.c@1.31.1.1
ChangeSet@1.2831.1.2 src/slib.c@1.1068.1.1
cosmith@hilbert:~/tmp/work/bk-7.3.1ce$ bk changes -v -d'$if(:GFILE:=src/utils/Makefile){:GFILE:@:REV:\n}' -r1.2868..1.2869
src/utils/Makefile@1.67
src/utils/Makefile@1.66
cosmith@hilbert:~/tmp/work/bk-7.3.1ce$ bk-graph -r1.64..1.67 src/utils/Makefile
* src/utils/Makefile@1.67, 2016-07-05 13:07:32-04:00, wscott@x99.wscott.bitkeeper.com +0 -0
|\
| * src/utils/Makefile@1.65.1.1, 2016-07-05 13:07:22-04:00, wscott@x99.wscott.bitkeeper.com +1 -0
* | src/utils/Makefile@1.66, 2016-07-01 17:02:39-04:00, wscott@x99.wscott.bitkeeper.com +1 -0
|/
* src/utils/Makefile@1.65, 2016-04-21 14:29:35-07:00, ob@dirac.bitkeeper.com +6 -6
cosmith@hilbert:~/tmp/work/bk-7.3.1ce$ bkargs='bk makepatch -d -r1.2868..1.2869'; diff -u <($bkargs) <(~/tmp/work/bitkeeper/src/$bkargs)
--- /dev/fd/63 2024-01-24 16:20:18.870258417 +0000
+++ /dev/fd/62 2024-01-24 16:20:18.870258417 +0000
@@ -5,6 +5,14 @@
# Host: hilbert.dev.bluearc.com
# Root: /home/cosmith/tmp/work/bk-7.3.1ce
+#--- 1.392.1.2/BitKeeper/etc/attr 2016-07-05 15:09:19 +01:00
+#+++ 1.394/BitKeeper/etc/attr 2016-07-05 18:07:32 +01:00
+#@@ -5,4 +5,4 @@
+# @ID
+# lm@lm.bitmover.com|ChangeSet|19990319224848|02682|B:1000:6a5ce40345b2dee1
+# @VERSION
+#-20160701210239
+#+20160630190712
#--- 1.23/BitKeeper/etc/ignore 2016-06-06 18:16:53 +01:00
#+++ 1.24/BitKeeper/etc/ignore 2016-07-01 22:02:39 +01:00
#@@ -128,3 +128,4 @@
@@ -590,16 +598,16 @@
# out("\r\n");
# out("Cache-Control: no-cache\r\n"); /* for http 1.1 */
# out("Pragma: no-cache\r\n"); /* for http 1.0 */
-#--- 1.66/src/utils/Makefile 2016-07-01 22:02:39 +01:00
+#--- 1.65.1.1/src/utils/Makefile 2016-07-05 18:07:22 +01:00
#+++ 1.67/src/utils/Makefile 2016-07-05 18:07:32 +01:00
-#@@ -42,6 +42,7 @@
-# SYS=unix
-# EXE=
-# EXT=.bin
-#+ LIBS=-lz
-# H=../unix.h
-# CFLAGS=-Os
-# LD=$(CC)
+#@@ -12,6 +12,7 @@
+# # See the License for the specific language governing permissions and
+# # limitations under the License.
+#
+#+BK=../bk
+#
+#
+# # Override Solaris make.rules
#--- 1.59/src/bkd_version.c 2016-03-09 13:22:25 +00:00
#+++ 1.60/src/bkd_version.c 2016-07-01 22:02:39 +01:00
#@@ -71,7 +71,7 @@
@@ -706,7 +714,7 @@
# W=-Wall -Wno-parentheses -Wno-strict-aliasing
#
-# Diff checksum=2170eeaa
+# Diff checksum=076c4ebd
# Patch vers: 1.4
cosmith@hilbert:~/tmp/work/bk-7.3.1ce$
… wherein we see that the shipping makepatch missed changes and included an extra one.
Our-ref D158146.