By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 1937

Summary: Directories to be Removed from the TDE Git Tree
Product: TDE Reporter: David C. Rankin <trin>
Component: non-core programsAssignee: Timothy Pearson <kb9vqf>
Status: NEW ---    
Severity: normal CC: bugwatch, kb9vqf, michele.calgaro, slavek.banko, trin
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: All   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 2967    

Description David C. Rankin 2014-02-16 01:35:17 CST
In an effort to clean up the TDE Git Tree, remove duplication, and reduce maintenance requirements, those directories that no longer serve a purpose in the Git tree should be removed. The current candidates initially discussed for removal are:

kwickshow      - duplicated in tdegraphics
filelight l10n - duplicated in applications/filelight
mlt/mlt++      - (no longer in use? - need confirmation)

Other candidates for removal should be identified and added to this bug for discussion. When adding a candidate for deletion, please use due diligence to provide information about the directors, whether it is needed as a dependency, whether it is duplicated somewhere else in the tree, whether it currently builds, and a short concise reason for suggesting removal.

Example Information:

Package: mlt/mlt++

Builds: yes - i686

Background: mlt & mtl++ are video frameworks that provide mechanisms for connecting to remote video sources and accessing the audio/video MLT frames in either NTSC or PAL from a Miracle server via telnet of custom interface. The Miracle server is provided as part of mlt. mlt++ is a c++ wrapper for mlt. There is no duplication of mlt/mtl++ in the remainder of the git tree. Origin - unknown.

Reason: Is this needed in TDE?
Comment 1 Michele Calgaro 2014-02-16 03:36:51 CST
Proposal for:
-applications/knetworkmanager8
-applications/kpowersave

Reason: AFAIK, replaced by
-applications/tdenetworkmanager
-applications/tdepowersave

Background:
knetworkmanager8 - tdenetworkmanager: systray applet for controlling network connections
kpowersave - tdepowersave: systray applet for controlling power management
Comment 2 Michele Calgaro 2014-02-16 03:39:51 CST
Proposal for cleaning up the Ubuntu folders in the packaging git repo.
In particular:
- lucid_automake
- maverick_automake
- kdegraphics.automake (in most Ubuntu distro folders. Also package renamed to tdegraphics)
- kdevelop.cmake (in most Ubuntu distro folders. Also package renamed to tdevelop)

Reason: are those distros/folders still needed/used? In comparison, Debian distro folders are much neater.
Comment 3 Michele Calgaro 2014-02-16 05:55:59 CST
Correction:
-applications/knetworkmanager8 still needed until support for NM0.8 is added to tdehw-lib.
See http://trinity-devel.pearsoncomputing.net/?0::12085 for details
Comment 4 Michele Calgaro 2014-02-17 05:40:24 CST
Correction: 
kpowersave still theoretically needed. See details here http://trinity-devel.pearsoncomputing.net/?0::12764
Comment 5 David C. Rankin 2014-02-18 13:02:10 CST
tde-construct (old Kconstruct) is another package consider removing. It is an old meta-build package that advertises the ability to automatically download all sources and build TDE for you with just:

"cd meta/kde;make install"

  By default "Konstruct" installs to ~/trinity.5.7/ which means you don't have
to possess root privileges or risk to damage your system or affect another KDE.

  Something like this is an excellent idea in concept, but I suspect that tde-construct is littered with links pointing to old kde3 sources that does next to nothing now. If it can be fixed to work with TDE - it is worth keeping - if it doesn't do anything - it is just dead weight on the git tree.
Comment 6 David C. Rankin 2014-02-18 13:47:34 CST
This bug should be read referencing bug 1944:

R14 Broken Packages (minor apps) - future fix or delete?
http://bugs.pearsoncomputing.net/bugzilla/show_bug.cgi?id=1944

1944 lists packages that currently build but are broken (or have sever limitation in function) That list currently includes:

kasablanca
kftpgrabber
kmymoney

If those are not going to be fixed, they can be considered here for deletion.
Comment 7 Michele Calgaro 2014-02-22 01:59:44 CST
Add dependencies/tqca as possible candidate for removal.
Reason: AFAIK it is not needed for R14.
Comment 8 Michele Calgaro 2014-03-08 21:16:03 CST
ubuntu distro folder (see comment 2) has been clean up in commit 4538fd673
Comment 9 Timothy Pearson 2014-03-08 22:29:12 CST
(In reply to David C. Rankin from comment #6)
> This bug should be read referencing bug 1944:
> 
> R14 Broken Packages (minor apps) - future fix or delete?
> http://bugs.pearsoncomputing.net/bugzilla/show_bug.cgi?id=1944
> 
> 1944 lists packages that currently build but are broken (or have sever
> limitation in function) That list currently includes:
> 
> kasablanca
> kftpgrabber
> kmymoney
> 
> If those are not going to be fixed, they can be considered here for deletion.

Those applications need to be fixed, not removed!

Please do not delete the debian/lenny directory as the autobuild system requires it to function correctly.

Please also keep a log of all deleted directories of applications that are currently active in the autobuild system.  Without corrections to that system upon deletion all autobuilds will stall.

tqca was used in Konversation to enable OTR chat IIRC.  Has this changed?
Comment 10 Michele Calgaro 2015-02-04 21:44:08 CST
tde-construct has been removed from current r14.x trunk in commit b461778.
Comment 11 Michele Calgaro 2018-08-19 09:40:06 CDT
tde-construct removed (this time for good) in commits 242be7e3 (R14.1) and 04a72d17 (R14.0)
Comment 12 Michele Calgaro 2018-08-19 10:19:06 CDT
mlt and mlt++ removed in commits 77e233a1 (R14.1) and 2e02a442 (R14.0)
Comment 13 Michele Calgaro 2018-08-19 10:24:23 CDT
mlt amd mlt++ removed from tde-packaging in commit 1ea97c304 (R14.1) and 8cc0c66cb (R1.0).