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 1575

Summary: tdelibs SONAME needs changing
Product: TDE Reporter: Aleksey Midenkov <midenok>
Component: other (any)Assignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: blocker CC: bugwatch, darrella, kb9vqf, slavek.banko
Priority: P5    
Version: 3.5.13 [Trinity]   
Hardware: All   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:

Description Aleksey Midenkov 2013-07-15 10:51:27 CDT
Some packages that depend on kdelibs4c2a (like sim, kkbswitch) are forced to uninstall.
Comment 1 Timothy Pearson 2013-07-15 13:33:06 CDT
TDE is not binary compatible with KDE 3.5.10.  As a result, adding a Provides line to the deb file is not likely to work at all; even in those cases where it still appears to work, future changes to TDE will likely render this method inoperable.

Instead of attempting to use the old unsupported binary packages mentioned, you should file Request for Packaging bug reports on this bugtracker, requesting that those packages be rebuilt against the latest versions of the TDE core libraries.  This will ensure that those packages continue to be available into the future.
Comment 2 Aleksey Midenkov 2013-07-16 04:37:57 CDT
Then, you should increase second version number (f.ex. kdelibs4c3).
Comment 3 Timothy Pearson 2013-07-16 14:31:54 CDT
(In reply to comment #2)
> Then, you should increase second version number (f.ex. kdelibs4c3).

This is a valid point.  We now use tdelibs, and the API/ABI have both changed significantly, so I suspect the sonames in tdelibs should be bumped to 5.0 and therefore the package name itself should be changed to tdelibs5.

Raising to BLOCKER to make sure that this gets in to R14.
Comment 4 Aleksey Midenkov 2013-07-18 03:16:10 CDT
I think, in case of rebranding you should change version to 1.0 or at least continue 4 major. Version 5 is not very good idea, because it makes confusion with kdelibs 5.

Anyway, I suspect that core architecture was not significantly changed. Are the API changes documented somewhere? Do you plan to merge API of kdelibs 5 (to achieve compatibility with their apps)?
Comment 5 Darrell 2013-08-16 14:37:04 CDT
Is this report something one of us "junior" hackers can resolve? What is needed?

Do we want to bump to 5.0 or as suggested, 1.0?
Comment 6 Timothy Pearson 2013-08-16 14:47:08 CDT
(In reply to comment #5)
> Is this report something one of us "junior" hackers can resolve? What is
> needed?
> 
> Do we want to bump to 5.0 or as suggested, 1.0?

It is Bad Practice to decrement the version number.  Either we go to 5.0 or 14.0.

Thoughts?

Tim
Comment 7 Darrell 2013-08-16 15:20:15 CDT
Bad Practice. Okay. :-)

14.0 then.

Is this something someone like me can do (I need instructions), so you can focus on the challenging bugs?
Comment 8 Timothy Pearson 2013-08-26 15:44:24 CDT
As no one has raised any concerns on this report regarding a version jump to 14.0.0, I have committed this change to GIT in hashes cb17faa (tdelibs) and a3fed09 (tde-packaging).

Someone should probably double-check my work as this is the first time I have done this sort of thing. :-)
Comment 9 Slávek Banko 2013-08-27 10:19:22 CDT
I think that would be appropriate for Debian / Ubuntu packages also rename tdelibs4-trinity-dev and tdelibs4-trinity-doc => update 4 to 14.

What do you think?
Comment 10 Slávek Banko 2013-08-27 10:24:42 CDT
Note: Because this change will require a rebuild almost the entire git tree, it would be good to push proposed patch for common admin module, because it also causes rebuild almost the entire tree - see:

http://trinity-devel.pearsoncomputing.net/?0::11395

I have to push the patch quickly, for this reason?
Comment 11 Timothy Pearson 2013-08-27 14:03:37 CDT
(In reply to comment #10)
> Note: Because this change will require a rebuild almost the entire git tree, it
> would be good to push proposed patch for common admin module, because it also
> causes rebuild almost the entire tree - see:
> 
> http://trinity-devel.pearsoncomputing.net/?0::11395
> 
> I have to push the patch quickly, for this reason?

Yes, go ahead and rename tdelibs4-trinity-dev and tdelibs4-trinity-doc after pushing the referenced admin/ module patch.  To avoid problems during this process, I have deactivated the autobuild system until all changes have propagated fully.
Comment 12 Darrell 2013-08-27 17:55:32 CDT
What should we watch for when we rebuild after applying these patches?
Comment 13 Timothy Pearson 2013-08-27 18:36:09 CDT
(In reply to comment #12)
> What should we watch for when we rebuild after applying these patches?

The tdelibs4 rename only affects Debian and Ubuntu.  As far as building from source goes, the only changes would be that the various tdelibs library files will be installed with a .14.0.0 suffix instead of a 4.2.0 suffix.  This change will largely be invisible due to the fact that each library also installs with two symlinks pointing to the .14.0.0-suffixed file; one symlinks as <libraryfilename>.14 and the other symlinks as <libraryfilename>.
Comment 14 Darrell 2013-08-27 18:39:48 CDT
Ok. I ran a build set with the sources updated from last night and I see the renamed lib files. No build problems and no usability problems thus far.
Comment 15 Slávek Banko 2013-08-27 18:48:34 CDT
I can confirm - the largest patch applies to Debian / Ubuntu trees in tde-packaging. I just checking patch, commit will be for a while.

Update in admin module does not have a direct connection with this renaming of the library suffix. The only thing that both are done simultaneously, because both leads to rebuild the entire GIT tree.
Comment 16 Slávek Banko 2013-08-27 19:38:03 CDT
Both finished!

It is perfect as GIT deal with parallel processing update_all_modules - one was run on my machine, because I'm not guessed that already runs on automated system. Everything went well - GIT is wonderful :)
Comment 17 Slávek Banko 2013-08-28 13:18:40 CDT
I watched that tdelibs14-trinity has set Provides: tdelibs4c2a-trinity. However, it does not seem like a good idea.

While it is good that now, before rebuild all other packages, update to tdelibs14-trinity will not uninstall all other packages, but it will not work. For example tdm_gree now report that not found libtdeui.so.4 and crashes.

The package tdelibs14-trinity simply does not provide the same libraries as tdelibs4c2a-trinity.
Comment 18 Timothy Pearson 2013-08-28 14:27:58 CDT
(In reply to comment #17)
> I watched that tdelibs14-trinity has set Provides: tdelibs4c2a-trinity.
> However, it does not seem like a good idea.
> 
> While it is good that now, before rebuild all other packages, update to
> tdelibs14-trinity will not uninstall all other packages, but it will not work.
> For example tdm_gree now report that not found libtdeui.so.4 and crashes.
> 
> The package tdelibs14-trinity simply does not provide the same libraries as
> tdelibs4c2a-trinity.

Fixed in GIT hash ac583d0.
Comment 19 Timothy Pearson 2013-09-09 22:34:10 CDT
It looks like this report is resolved, closing.