Bug 2152

Slavek, are you working on this report?

Thanks!

(In reply to Timothy Pearson from comment #1)
> Slavek, are you working on this report?
> 
> Thanks!

At this time not.
I am currently focused on bug 2110.

Looks like Digikam is going to require a lot of work; lcms2 is not 1:1 feature compatible:
http://littlecms2.blogspot.com/2009/11/backwards-compatibility.html

Anyone up to the challenge?

(In reply to Timothy Pearson from comment #3)
> Looks like Digikam is going to require a lot of work; lcms2 is not 1:1
> feature compatible:
> http://littlecms2.blogspot.com/2009/11/backwards-compatibility.html
> 
> Anyone up to the challenge?

OK, after digging a bit more it seems that Digikam uses lprof, which in turn uses lcms1.  lprof is not maintained since 2008 and KDE's Digikam simply dropped it and the features it relied on entirely:
http://marc.info/?l=kde-commits&m=132813939610177&w=2

Not sure if that's an acceptable solution; personally I'd be more inclined to move lprof to a 3rdparty folder and add a copy of liblcms1 to the 3rdparty folder; this would at least buy us a bit more time to figure out how best to re-implement the broken features.

(In reply to Timothy Pearson from comment #4)
> (In reply to Timothy Pearson from comment #3)
> > Looks like Digikam is going to require a lot of work; lcms2 is not 1:1
> > feature compatible:
> > http://littlecms2.blogspot.com/2009/11/backwards-compatibility.html
> > 
> > Anyone up to the challenge?
> 
> OK, after digging a bit more it seems that Digikam uses lprof, which in turn
> uses lcms1.  lprof is not maintained since 2008 and KDE's Digikam simply
> dropped it and the features it relied on entirely:
> http://marc.info/?l=kde-commits&m=132813939610177&w=2
> 
> Not sure if that's an acceptable solution; personally I'd be more inclined
> to move lprof to a 3rdparty folder and add a copy of liblcms1 to the
> 3rdparty folder; this would at least buy us a bit more time to figure out
> how best to re-implement the broken features.

I was looking for information in digikam git:

http://quickgit.kde.org/?p=digikam.git&a=search&h=HEAD&st=commit&s=lcms2

However, so far I have not looked at the complexity of individual patches.

Challenging seems to be also adjustments in KOffice:

http://quickgit.kde.org/?p=calligra.git&a=search&h=HEAD&st=commit&s=lcms2

So basically it looks like there is a lot of backporting to do.

After you are done with Bug 2110 do you want to start working on this?  I probably won't be able to, at least not in time for R14; I've been working on TDE pretty much non-stop for a month now and need to tend to other duties soon. :-)

Thanks!

(In reply to Timothy Pearson from comment #7)
> So basically it looks like there is a lot of backporting to do.
> 
> After you are done with Bug 2110 do you want to start working on this?  I
> probably won't be able to, at least not in time for R14; I've been working
> on TDE pretty much non-stop for a month now and need to tend to other duties
> soon. :-)
> 
> Thanks!

For patches from bug 2110 I now waiting for confirmation from François. Besides, I have to prepare a patch for the r14-xdg-update to add processing KNewStuff => TDENewStuff in existing user profile. It is also important / necessary, and I expect that it will be relatively easy. Then I will be able to delve into the processing digikam and chalk patches.

(In reply to Slávek Banko from comment #8)
> (In reply to Timothy Pearson from comment #7)
> > So basically it looks like there is a lot of backporting to do.
> > 
> > After you are done with Bug 2110 do you want to start working on this?  I
> > probably won't be able to, at least not in time for R14; I've been working
> > on TDE pretty much non-stop for a month now and need to tend to other duties
> > soon. :-)
> > 
> > Thanks!
> 
> For patches from bug 2110 I now waiting for confirmation from François.
> Besides, I have to prepare a patch for the r14-xdg-update to add processing
> KNewStuff => TDENewStuff in existing user profile. It is also important /
> necessary, and I expect that it will be relatively easy. Then I will be able
> to delve into the processing digikam and chalk patches.

OK, sounds great!  I'll be poking at Bug 1850 as I have time.

As additional info, digikam and koffice FTBFS in Jessie due to this problem.

(In reply to Michele Calgaro from comment #10)
> As additional info, digikam and koffice FTBFS in Jessie due to this problem.

Slavek, what is the status of this?

Thanks!

Tim

(In reply to Timothy Pearson from comment #11)
> (In reply to Michele Calgaro from comment #10)
> > As additional info, digikam and koffice FTBFS in Jessie due to this problem.
> 
> Slavek, what is the status of this?
> 
> Thanks!
> 
> Tim

I started to work on KOffice.

(In reply to Slávek Banko from comment #12)
> (In reply to Timothy Pearson from comment #11)
> > (In reply to Michele Calgaro from comment #10)
> > > As additional info, digikam and koffice FTBFS in Jessie due to this problem.
> > 
> > Slavek, what is the status of this?
> > 
> > Thanks!
> > 
> > Tim
> 
> I started to work on KOffice.

How is the fix progressing?  If it ends up being too complex to finish before R14 release we'll need to bundle lcms1 with R14.

Thanks!

Digikam is failing to build on Precise with lcms2 errors:
https://librarian.quickbuild.pearsoncomputing.net/6576869/buildlog_ubuntu-precise-i386.digikam-trinity_4%3A14.0.0-r221-0ubuntu12.04.0%2Bpr29_FAILEDTOBUILD.txt.gz

Also kipi-plugins-trinity:
https://librarian.quickbuild.pearsoncomputing.net/6576917/buildlog_ubuntu-precise-i386.kipi-plugins-trinity_4%3A14.0.0-r194-0ubuntu12.04.0%2Bpr25_FAILEDTOBUILD.txt.gz

This is blocking the release of RC1; any chance of a quick fix?

Thanks!

Ok, I think I know where the problem is. Library libkdcraw already uses lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.

Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).

(In reply to Slávek Banko from comment #15)
> Ok, I think I know where the problem is. Library libkdcraw already uses
> lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.
> 
> Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).

Unfortunately this doesn't look fixed:
https://librarian.quickbuild.pearsoncomputing.net/6590834/buildlog_ubuntu-saucy-amd64.digikam-trinity_4%3A14.0.0-r221-0ubuntu13.10.0%2Bpr29_FAILEDTOBUILD.txt.gz

(In reply to Timothy Pearson from comment #16)
> (In reply to Slávek Banko from comment #15)
> > Ok, I think I know where the problem is. Library libkdcraw already uses
> > lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.
> > 
> > Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).
> 
> Unfortunately this doesn't look fixed:
> https://librarian.quickbuild.pearsoncomputing.net/6590834/buildlog_ubuntu-
> saucy-amd64.digikam-trinity_4%3A14.0.0-r221-0ubuntu13.10.
> 0%2Bpr29_FAILEDTOBUILD.txt.gz

Variable dependency apparently was not enough. Therefore I added strict dependency on liblcms*-dev, which were used for building libkdcraw.

I hope that has really been fixed by commit c5c42041.

(In reply to Slávek Banko from comment #17)
> (In reply to Timothy Pearson from comment #16)
> > (In reply to Slávek Banko from comment #15)
> > > Ok, I think I know where the problem is. Library libkdcraw already uses
> > > lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.
> > > 
> > > Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).
> > 
> > Unfortunately this doesn't look fixed:
> > https://librarian.quickbuild.pearsoncomputing.net/6590834/buildlog_ubuntu-
> > saucy-amd64.digikam-trinity_4%3A14.0.0-r221-0ubuntu13.10.
> > 0%2Bpr29_FAILEDTOBUILD.txt.gz
> 
> Variable dependency apparently was not enough. Therefore I added strict
> dependency on liblcms*-dev, which were used for building libkdcraw.
> 
> I hope that has really been fixed by commit c5c42041.

Closer; now only Wheezy is failing:
https://librarian.quickbuild.pearsoncomputing.net/6591513/buildlog_ubuntu-wheezy-amd64.digikam-trinity_4%3A14.0.0-r221-0debian7.0.0%2Bpr27_FAILEDTOBUILD.txt.gz

(In reply to Timothy Pearson from comment #18)
> (In reply to Slávek Banko from comment #17)
> > (In reply to Timothy Pearson from comment #16)
> > > (In reply to Slávek Banko from comment #15)
> > > > Ok, I think I know where the problem is. Library libkdcraw already uses
> > > > lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.
> > > > 
> > > > Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).
> > > 
> > > Unfortunately this doesn't look fixed:
> > > https://librarian.quickbuild.pearsoncomputing.net/6590834/buildlog_ubuntu-
> > > saucy-amd64.digikam-trinity_4%3A14.0.0-r221-0ubuntu13.10.
> > > 0%2Bpr29_FAILEDTOBUILD.txt.gz
> > 
> > Variable dependency apparently was not enough. Therefore I added strict
> > dependency on liblcms*-dev, which were used for building libkdcraw.
> > 
> > I hope that has really been fixed by commit c5c42041.
> 
> Closer; now only Wheezy is failing:
> https://librarian.quickbuild.pearsoncomputing.net/6591513/buildlog_ubuntu-
> wheezy-amd64.digikam-trinity_4%3A14.0.0-r221-0debian7.0.
> 0%2Bpr27_FAILEDTOBUILD.txt.gz

Looks fixed--thanks!

What is the status of this report?  As R14 RC1 is almost fully uploaded we will be opening the merge window for RC2 in the next day or so.

Thanks!

(In reply to Timothy Pearson from comment #19)
> (In reply to Timothy Pearson from comment #18)
> > (In reply to Slávek Banko from comment #17)
> > > (In reply to Timothy Pearson from comment #16)
> > > > (In reply to Slávek Banko from comment #15)
> > > > > Ok, I think I know where the problem is. Library libkdcraw already uses
> > > > > lcms2 support and therefore kipi-plugins and digikam tries to link lcms2.
> > > > > 
> > > > > Fixed in GIT hash f8131d40 (tde-packaging - package libkdcraw).
> > > > 
> > > > Unfortunately this doesn't look fixed:
> > > > https://librarian.quickbuild.pearsoncomputing.net/6590834/buildlog_ubuntu-
> > > > saucy-amd64.digikam-trinity_4%3A14.0.0-r221-0ubuntu13.10.
> > > > 0%2Bpr29_FAILEDTOBUILD.txt.gz
> > > 
> > > Variable dependency apparently was not enough. Therefore I added strict
> > > dependency on liblcms*-dev, which were used for building libkdcraw.
> > > 
> > > I hope that has really been fixed by commit c5c42041.
> > 
> > Closer; now only Wheezy is failing:
> > https://librarian.quickbuild.pearsoncomputing.net/6591513/buildlog_ubuntu-
> > wheezy-amd64.digikam-trinity_4%3A14.0.0-r221-0debian7.0.
> > 0%2Bpr27_FAILEDTOBUILD.txt.gz
> 
> Looks fixed--thanks!

Oh, yes, I did not get to write that problem with multiarch on Wheezy was treated in commit b9049d0f (tde-packaging).

(In reply to Timothy Pearson from comment #20)
> What is the status of this report?  As R14 RC1 is almost fully uploaded we
> will be opening the merge window for RC2 in the next day or so.
> 
> Thanks!

The last few days I unfortunately did not have enough time to could move with this forward. I hope that maybe this weekend I'll find some time.

(In reply to Slávek Banko from comment #22)
> (In reply to Timothy Pearson from comment #20)
> > What is the status of this report?  As R14 RC1 is almost fully uploaded we
> > will be opening the merge window for RC2 in the next day or so.
> > 
> > Thanks!
> 
> The last few days I unfortunately did not have enough time to could move
> with this forward. I hope that maybe this weekend I'll find some time.

For information: last weekend I was intensely work on patch for KOffice, but as I suspected, it is a challenging task and completion it will still take some time.

Because solving is currently required only for Debian Jessie, this bug can be considered non-blocking for R14.0.0.

That is what I expected.  Thank you for trying at least; the work will still need to be done for our next point release.

I will probably work around the Jessie issue by building old libcms in the deps repo.

(In reply to Timothy Pearson from comment #24)
> That is what I expected.  Thank you for trying at least; the work will still
> need to be done for our next point release.
> 
> I will probably work around the Jessie issue by building old libcms in the
> deps repo.

lcms v1 is now in Jessie R14 build deps archive; removing from R14 block list.