[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

Christoph Cullmann  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/kde/
   ||kate/commit/a592bf54ba7d0ab
   ||4c7a2fa967447b12dd76d768d
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Christoph Cullmann  ---
Git commit a592bf54ba7d0ab4c7a2fa967447b12dd76d768d by Christoph Cullmann.
Committed on 27/09/2019 at 13:00.
Pushed by cullmann into branch 'master'.

disable ctags indexing per default

one can enable it via

"index": true

in the .kateproject.

The GUI tells this now.
Related: bug 412351

M  +7-5.kateproject
M  +4-4addons/project/kateproject.cpp
M  +9-2addons/project/kateprojectinfoviewindex.cpp
M  +11   -4addons/project/kateprojectworker.cpp
M  +3-3addons/project/kateprojectworker.h

https://invent.kde.org/kde/kate/commit/a592bf54ba7d0ab4c7a2fa967447b12dd76d768d

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

Christoph Cullmann  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/kde/ |https://commits.kde.org/kat
   |kate/commit/a592bf54ba7d0ab |e/a592bf54ba7d0ab4c7a2fa967
   |4c7a2fa967447b12dd76d768d   |447b12dd76d768d

--- Comment #9 from Christoph Cullmann  ---
Git commit a592bf54ba7d0ab4c7a2fa967447b12dd76d768d by Christoph Cullmann.
Committed on 27/09/2019 at 13:00.
Pushed by scmsync into branch 'master'.

disable ctags indexing per default

one can enable it via

"index": true

in the .kateproject.

The GUI tells this now.
Related: bug 412351

M  +7-5.kateproject
M  +4-4addons/project/kateproject.cpp
M  +9-2addons/project/kateprojectinfoviewindex.cpp
M  +11   -4addons/project/kateprojectworker.cpp
M  +3-3addons/project/kateprojectworker.h

https://commits.kde.org/kate/a592bf54ba7d0ab4c7a2fa967447b12dd76d768d

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[zeta]

https://bugs.kde.org/show_bug.cgi?id=387830

zeta  changed:

   What|Removed |Added

   Severity|normal  |minor

--- Comment #7 from zeta  ---
Or let the user type a whitelist/blackstic of repos to include or exclude from
autoload.

Are you saying that it could be possible to have the 'Proyects' tab enabled but
without ctags? Maybe an extra checkbox on the config.

I'll lower the importance of the bug to minor, as I think other bugs may be
more relevant. Still, if you are already invested go ahead. 

I'm not a C programmer but still, tell me if you need anything.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

--- Comment #6 from Christoph Cullmann  ---
I think I still need to make the ctags stuff "optional", to let people turn it
off if they have issues (or just don't want it).

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[zeta]

https://bugs.kde.org/show_bug.cgi?id=387830

--- Comment #5 from zeta  ---
I cannot reproduce it anymore:
It was either fixed in recent versions, or maybe it was a specific symlink
configuration that was messing things up.
Current versions:

➤ $ kate --version
kate 17.12.0
➤ $ ctags --version
Exuberant Ctags 5.9~svn20110310, Copyright (C) 1996-2009 Darren Hiebert
➤ $ git --version  
  git version 2.13.2

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

--- Comment #4 from Christoph Cullmann  ---
More or less we just feed in git ls-files as stdin for the ctags.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[zeta]

https://bugs.kde.org/show_bug.cgi?id=387830

--- Comment #3 from zeta  ---
Yea, I did turn it off. Which is bad for other repos :( 

(Is it possible to configure the autoload per repo?)

I think of duplication or recursion loop because the sizes really do not make
any sense. Out of the 230 MB in the repo, there are probably just 50 MB of
actual code, so Im thinking of a problem in ctags itself. 

If you can guide me on process tracing to see what goes in that ctags stdin,
I'll be more then happy to investigate.

Best regards.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

Christoph Cullmann  changed:

   What|Removed |Added

   Assignee|kwrite-bugs-n...@kde.org|cullm...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

Christoph Cullmann  changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 CC||cullm...@kde.org
 Ever confirmed|0   |1

--- Comment #2 from Christoph Cullmann  ---
Hi, I have taken a look at the code.

The 

QProcess: Destroyed while process ("ctags") is still running.

is I guess the fail of the code to catch the case "ctags takes longer than 30
seconds."

For the "files are duplicated added to the ctags call", I think that is not the
case, but ctags might just be a bad idea for that repo. I should reconsider if
ctags should be auto-on, looks like a bad idea in some cases :/

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[Christoph Cullmann]

https://bugs.kde.org/show_bug.cgi?id=387830

Christoph Cullmann  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

-- 
You are receiving this mail because:
You are watching all bug changes.

[kate] [Bug 387830] Autoload repositories causes 3.4 gig ctags file

[zeta]

https://bugs.kde.org/show_bug.cgi?id=387830

--- Comment #1 from zeta  ---
*** Bug 386944 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.