>>> Junio C Hamano schrieb am 13.07.2020 um 17:12 in Nachricht : > "Ulrich Windl" writes: > >> I inspected some changes using "git diff", and it was hard to me >> to find the actual differences (see git‑diff‑png): I had to look a >> long time at those olcAccess lines to find any difference, so >> finally I pasted both blocks into separate Emacs files a and b >> (via PuTTY) and used "ediff‑files" (see diff‑emacs.png). Within >> there, the was only a part of one line changed! > > With redactions and making them into two images, you did a good job > killing interested parties' willingness to help, as they made using > tools almost impossible. Junio, we had some discussions in the past, and you should know that I'm willing to provide information if needed. Part of the problem is that I can't (to my knowledge) copy and paste colored text into plain text emails. Thus screenshots. My intention was to bring up the issue, making it (more or less obvious) that the differences are much harder to see in git output than in Emacs output. The redaction was due to a password that could easily be cracked if presented and some resource name I wanted to avoid to get a clearance to publish first. It still could be a "stupid me", but can't we talk like adults? > > So here is only based on eyeballing the images, and I probably have > missed what you wanted readers to find, but don't blame me ;‑) > > It seems that the file got 6 lines removed, that begins with > "olcAccess: {$n}" (0 <= n <= 6), at around 8th line of the file, and > then very similar 6 lines that are not identical to what got removed > at the end of the file. The ediff you showed tells me that the 6 > lines removed from very early part of the file and 6 lines added to > the end of the file are similar but not identical. These two 6‑line > blocks differ only by the 5th line (i.e. "olcAccess: {4}") having > ",pwdGraceUseTime,pwdChangedTime" in the newer version, but not in > the older version. > > Am I reading what you sent us correctly so far? Yes, if you look at the highlighted part (difference in cyan, "chink" in yellow) in the Emacs screenshot, it should point out the "real" differences, while git output made it harder to see. > > Now, if this change to the "olcAccess: {4}" line in the new file > weren't there, i.e. if the 6‑line block that disappeared from the > early part of the file and the 6‑line block that was added near the > end of the file were identical (in such a case, your ediff would > show no changes, I presume), then what would your desired output > from "git diff"? > > I would imagine that the answer would still be a very similar looking > > ‑‑‑ a/0/... > +++ b/0/... > @@ ‑5,12 +5,6 @@ > olcDatabase: {1}hdb > olcDbDirectory: /var/lib/ldap > olcSuffix: dc=... > ‑olcAccess: {0} ... > ‑olcAccess: {1} ... > ‑olcAccess: {2} ... > ‑olcAccess: {3} ... > ‑olcAccess: {4} ... > ‑olcAccess: {5} ... > olcLimits: {0}dn.exact=.... > olcRootDn: cn=Admin,dc=... > olcRootPW: {SSHA}yZ... > @@ ‑52,6 +46,12 @@ > entryUUID: ... > creatorsName: cn=config > createTimestamp: ... > ‑entryCSN: ... > +olcAccess: {0} ... > +olcAccess: {1} ... > +olcAccess: {2} ... > +olcAccess: {3} ... > +olcAccess: {4} ... > +olcAccess: {5} ... > +entryCSN: ... > modifiersName: cn=config > ‑modifyTimestamp: ... > +modifyTimestamp: ... > > with identical "olcAccess: {$n}" lines removed from earlier part the > file and added to the end of the file. > > So I am not sure what you are asking. It would not be correct to show > something like According to my eyes (just using Emacs to make sure) those olcAccess lines "numbered" 0 to 3, and 5 should be "context lines" as they are not changed. Only line numbered {4} has a difference, and as the order 0 to 5 is unchanged I don't see why the diff-block is that large. I see that some lines that are not changed were reorderd, but that should not affect the olcAccess block. Maybe I just need different diff options; I don't know. > > ‑‑‑ a/0/... > +++ b/0/... > @@ ‑52,6 +46,6 @@ > entryUUID: ... > creatorsName: cn=config > createTimestamp: ... > ‑entryCSN: ... > +entryCSN: ... > modifiersName: cn=config > ‑modifyTimestamp: ... > +modifyTimestamp: ... > > and pretend that the original "olcAccess: {$n}" lines did not get > moved from the earlier part of the file to the end of the file. > > Perhaps you may be looking for "git diff ‑‑color‑moved"? I dunno. I kind of understand: The 5-line block of olcAccess was moved within a larger block, preventing diff from seeing the small change within. I'm attaching the "git diff --color-moved" output as another screenshot (this time with very wide terminal to make one line a line). Still I redacted ("blued-out") some information I may not publish, but believe me that the parts "blued out" are identical. In addition I made a test-case that includes only the first 18 columns (cut -c1-18) of the file, both before (b0) and after (b1) the commit. That should be good enough for the reordering effect. A plain "diff -u shows": # diff -u b0 b1 --- b0 2020-07-14 10:43:05.000000000 +0200 +++ b1 2020-07-14 10:41:37.000000000 +0200 @@ -5,12 +5,6 @@ olcDatabase: {1}hd olcDbDirectory: /v olcSuffix: dc=sap, -olcAccess: {0}to * -olcAccess: {1}to d -olcAccess: {2}to a -olcAccess: {3}to a -olcAccess: {4}to a -olcAccess: {5}to * olcLimits: {0}dn.e olcRootDN: cn=Admi olcRootPW: {SSHA}y @@ -52,6 +46,12 @@ entryUUID: db3ffe4 creatorsName: cn=c createTimestamp: 2 -entryCSN: 20170706 +olcAccess: {0}to * +olcAccess: {1}to d +olcAccess: {2}to a +olcAccess: {3}to a +olcAccess: {4}to a +olcAccess: {5}to * +entryCSN: 20200114 modifiersName: cn= modifyTimestamp: 2 So it seems the actual problem was that the block had moved, and I did not notice that when looking at the diffs. An "sdiff" illustrates: # sdiff b0 b1 dn: olcDatabase={1 dn: olcDatabase={1 objectClass: olcDa objectClass: olcDa objectClass: olcHd objectClass: olcHd olcDatabase: {1}hd olcDatabase: {1}hd olcDbDirectory: /v olcDbDirectory: /v olcSuffix: dc=sap, olcSuffix: dc=sap, olcAccess: {0}to * < olcAccess: {1}to d < olcAccess: {2}to a < olcAccess: {3}to a < olcAccess: {4}to a < olcAccess: {5}to * < olcLimits: {0}dn.e olcLimits: {0}dn.e olcRootDN: cn=Admi olcRootDN: cn=Admi olcRootPW: {SSHA}y olcRootPW: {SSHA}y olcSecurity: ssf=1 olcSecurity: ssf=1 olcSyncrepl: {0}ri olcSyncrepl: {0}ri olcSyncrepl: {1}ri olcSyncrepl: {1}ri olcSyncrepl: {2}ri olcSyncrepl: {2}ri olcUpdateRef: ldap olcUpdateRef: ldap olcUpdateRef: ldap olcUpdateRef: ldap olcUpdateRef: ldap olcUpdateRef: ldap olcMirrorMode: TRU olcMirrorMode: TRU olcDbCacheSize: 10 olcDbCacheSize: 10 olcDbCheckpoint: 1 olcDbCheckpoint: 1 olcDbConfig: {0}se olcDbConfig: {0}se olcDbConfig: {1}se olcDbConfig: {1}se olcDbConfig: {2}se olcDbConfig: {2}se olcDbConfig: {3}se olcDbConfig: {3}se olcDbConfig: {4}se olcDbConfig: {4}se olcDbConfig: {5}se olcDbConfig: {5}se olcDbIDLcacheSize: olcDbIDLcacheSize: olcDbIndex: object olcDbIndex: object olcDbIndex: uidNum olcDbIndex: uidNum olcDbIndex: gidNum olcDbIndex: gidNum olcDbIndex: member olcDbIndex: member olcDbIndex: member olcDbIndex: member olcDbIndex: cn eq, olcDbIndex: cn eq, olcDbIndex: uid eq olcDbIndex: uid eq olcDbIndex: sn eq, olcDbIndex: sn eq, olcDbIndex: givenN olcDbIndex: givenN olcDbIndex: entryU olcDbIndex: entryU olcDbIndex: entryC olcDbIndex: entryC olcDbIndex: ipServ olcDbIndex: ipServ olcDbIndex: ipServ olcDbIndex: ipServ olcDbIndex: roleOc olcDbIndex: roleOc olcDbIndex: mail e olcDbIndex: mail e olcDbIndex: displa olcDbIndex: displa olcDbIndex: modify olcDbIndex: modify structuralObjectCl structuralObjectCl entryUUID: db3ffe4 entryUUID: db3ffe4 creatorsName: cn=c creatorsName: cn=c createTimestamp: 2 createTimestamp: 2 entryCSN: 20170706 | olcAccess: {0}to * > olcAccess: {1}to d > olcAccess: {2}to a > olcAccess: {3}to a > olcAccess: {4}to a > olcAccess: {5}to * > entryCSN: 20200114 modifiersName: cn= modifiersName: cn= modifyTimestamp: 2 modifyTimestamp: 2 Regards, Ulrich