#308 - bug in diff web view with lines starting with minus characters - Codeberg/Community

sh2d commented

2020-10-12 18:27:05 +02:00

In Lua, single line comments start with two minus characters --. Codeberg's diff web view seems to have problems showing removed Lua comment lines that start at the first column. Added comment lines or comment lines starting with indentation seem not to be affected.

Recipe:

$ git clone https://codeberg.org/sh2d/padrinoma.git
$ git log -p fb32cf2 lua-modules/pdnm_nl_manipulation.lua

The latter command shows correctly:

diff --git a/lua-modules/pdnm_nl_manipulation.lua b/lua-modules/pdnm_nl_manipulation.lua
index 1f485fd..3e73c04 100644
--- a/lua-modules/pdnm_nl_manipulation.lua
+++ b/lua-modules/pdnm_nl_manipulation.lua
@@ -59,7 +59,7 @@
 
 
 
--- Table for module identification (see below).
+-- Table for module identification.

Watching the same commit at Codeberg, sh2d/padrinoma@fb32cf2f69 (scroll down to see file lua-modules/pdnm_nl_manipulation.lua) the removed comment line is missing (ln. 62).

Guesswork: Problematic lines in the diff start with multiple minus characters -. One minus character being diff's removed line indicator and two introducing a Lua comment.

In Lua, single line comments start with two minus characters `--`. Codeberg's diff web view seems to have problems showing _removed_ Lua comment lines that start at the first column. Added comment lines or comment lines starting with indentation seem not to be affected. Recipe: ``` sh $ git clone https://codeberg.org/sh2d/padrinoma.git $ git log -p fb32cf2 lua-modules/pdnm_nl_manipulation.lua ``` The latter command shows correctly: ``` text diff --git a/lua-modules/pdnm_nl_manipulation.lua b/lua-modules/pdnm_nl_manipulation.lua index 1f485fd..3e73c04 100644 --- a/lua-modules/pdnm_nl_manipulation.lua +++ b/lua-modules/pdnm_nl_manipulation.lua @@ -59,7 +59,7 @@ --- Table for module identification (see below). +-- Table for module identification. ``` Watching the same commit at Codeberg, https://codeberg.org/sh2d/padrinoma/commit/fb32cf2f6908e3645aca1590ae4c56ad05f17cae (scroll down to see file `lua-modules/pdnm_nl_manipulation.lua`) the removed comment line is missing (ln. 62). Guesswork: Problematic lines in the diff start with multiple minus characters `-`. One minus character being diff's removed line indicator and two introducing a Lua comment.

sh2d changed title from ~~bug in diff web view with Lua comments~~ to bug in diff web view with lines starting with minus characters

2020-10-12 18:46:27 +02:00

hw added the

bug

s/Forgejo

labels

2020-10-12 20:32:10 +02:00

6543 commented

2020-10-12 22:26:39 +02:00

@sh2d did you create an issue upstream?

👍 1

6543 commented

2020-10-12 22:29:21 +02:00

I think this is a regex issue in modules/git

I think this is a regex issue in **modules/git**

sh2d commented

2020-10-12 22:34:16 +02:00

Author

@6543 Not yet. I'm just browsing the tracker. Hell, 1.300 open issues? Is that a software I wanna use? Quiet disappointing. Will add one more issue after I finished browsing.

👍 1 ❤️ 1

hw commented

2020-10-12 22:46:00 +02:00

Member

:) yes. Check the rate of closed issues too ;)

sh2d commented

2020-10-12 23:11:01 +02:00

Author

Could anybody shed some light on how to find out the Gitea version Codeberg is running?

lhinderberger commented

2020-10-12 23:13:27 +02:00

@sh2d It's explained at https://docs.codeberg.org/faq/ - currently it's 1.12.5+20-g7494b24c6

@sh2d It's explained at https://docs.codeberg.org/faq/ - currently it's `1.12.5+20-g7494b24c6`

👍 1

lhinderberger commented

2020-10-12 23:16:25 +02:00

By the way, you're right - Gitea is a bit of a bug-fest. But at the moment, it seems like it's the best alternative available for a project like Codeberg. And while the issue tracker seems to be overflowing, PRs seem to get processed quite quickly :)

sh2d commented

2020-10-12 23:20:01 +02:00

Author

@lhinderberger Ah, thank you! About the number of Gitea bugs, as @hw hinted to, last month saw more bugs closed than opened at least.

👍 1

6543 commented

2020-10-13 00:11:38 +02:00

issues with bug lable: 138 open | 1394 closed 😅 - gitea get a lot of feature requests too

issues with bug lable: `138 open | 1394 closed` :sweat_smile: - gitea get **a lot** of feature requests too

👍 1

6543 commented

2020-10-13 00:12:45 +02:00

@sh2d gita's maintainer are now aware of this ... so keep clam and hope we find time to fix it 😓

👍 1

sh2d commented

2020-10-13 18:49:29 +02:00

Author

One more observation, lines with leading plus sign characters are also affected. You can find a test repo at https://codeberg-test.org/sh2d/plus

$ git show -1
commit dd03689a37ab1adfb7fcb122693b54339d2c819f (HEAD -> master, origin/master)
Author: Stephan Hennig <sh2d@posteo.net>
Date:   Tue Oct 13 18:25:27 2020 +0200

    Change leading -- to ++

diff --git a/test.txt b/test.txt
index c7ddeba..076580a 100644
--- a/test.txt
+++ b/test.txt
@@ -1,4 +1,4 @@
 A test file.
--- Starting with minus sign characters.
+++ Starting with plus sign characters.
 Another line.
-Trailing line.
+Trailing long line.

Web view of commit sh2d/plus@dd03689a37 shows none of the removed/added lines containing the text "Starting with ...", but the change of the trailing line only.

One more observation, lines with leading plus sign characters are also affected. You can find a test repo at https://codeberg-test.org/sh2d/plus ``` text $ git show -1 commit dd03689a37ab1adfb7fcb122693b54339d2c819f (HEAD -> master, origin/master) Author: Stephan Hennig <sh2d@posteo.net> Date: Tue Oct 13 18:25:27 2020 +0200 Change leading -- to ++ diff --git a/test.txt b/test.txt index c7ddeba..076580a 100644 --- a/test.txt +++ b/test.txt @@ -1,4 +1,4 @@ A test file. --- Starting with minus sign characters. +++ Starting with plus sign characters. Another line. -Trailing line. +Trailing long line. ``` Web view of commit https://codeberg-test.org/sh2d/plus/commit/dd03689a37ab1adfb7fcb122693b54339d2c819f shows none of the removed/added lines containing the text "Starting with ...", but the change of the trailing line only.

zeripath commented

2020-10-13 22:19:23 +02:00

OK the cause of this is the incorrect parsing of the patch files in services/gitdiff/gitdiff.go:449:ParsePatch

It's just wrong.

OK the cause of this is the incorrect parsing of the patch files in `services/gitdiff/gitdiff.go:449:ParsePatch` It's just wrong.

👍 3 ❤️ 1

6543 commented

2020-10-15 16:52:37 +02:00

@zeripath rocks!

-> https://github.com/go-gitea/gitea/pull/13157

@zeripath rocks! -> https://github.com/go-gitea/gitea/pull/13157

👍 4

hw commented

2020-10-15 17:55:17 +02:00

Member

hehe :)

ashimokawa commented

2020-10-15 19:11:07 +02:00

Owner

Cool, will watch the PR discussion, seems a bit too early to cherry pick, but the fix is near :)

👍 1

ashimokawa commented

2020-10-16 23:50:44 +02:00

Owner

Deployed!

@sh2d

Should be fixed

Deployed! @sh2d Should be fixed

ashimokawa commented

2020-10-17 00:17:17 +02:00

Owner

EDIT: Sorry it has not been merged in 1.13 yet, only 1.12 and master, codeberg-test.org runs 1.13-rc1+

@zeripath

~~This still looks broken though:~~

sh2d/plus@dd03689a37

~~The patch stats are ok now (2 lines changed) but the diff still misses one line afaics...~~

**EDIT:** Sorry it has not been merged in 1.13 yet, only 1.12 and master, codeberg-test.org runs 1.13-rc1+ @zeripath ~~This still looks broken though:~~ https://codeberg-test.org/sh2d/plus/commit/dd03689a37ab1adfb7fcb122693b54339d2c819f ~~The patch stats are ok now (2 lines changed) but the diff still misses one line afaics...~~

sh2d commented

2020-10-17 00:27:10 +02:00

Author

Sorry, hit the wrong button.

ashimokawa commented

2020-10-17 12:22:23 +02:00

Owner

@sh2d
It should be fixed on codeberg.org just not on codeberg-test.org, can you confirm?

@sh2d It should be fixed on codeberg.org just not on codeberg-test.org, can you confirm?

ashimokawa commented

2020-10-17 12:45:01 +02:00

Owner

The fix is now also deployed on codeberg-test.org!

I think this can be closed. @sh2d

The fix is now also deployed on codeberg-test.org! I think this can be closed. @sh2d

sh2d commented

2020-10-17 13:45:36 +02:00

Author

I can confirm that I cannot reproduce the issue anymore. Another test commit sh2d/padrinoma@0bdd9bbef5 looks fine, too (cf. file examples/luatex/break-ligatures/pdnm_break-ligatures.lua).

The thing is, I didn't wanted to close the issue myself, but was actually looking for a "discard" button. Closing the issue now.

I can confirm that I cannot reproduce the issue anymore. Another test commit https://codeberg.org/sh2d/padrinoma/commit/0bdd9bbef59b421d7727c279ef3ada7082cb8742 looks fine, too (cf. file examples/luatex/break-ligatures/pdnm_break-ligatures.lua). The thing is, I didn't wanted to close the issue myself, but was actually looking for a "discard" button. Closing the issue now.

👍 1 ❤️ 1

sh2d closed this issue

2020-10-17 13:45:47 +02:00