Bug 303843

Summary: okular cannot find words
Product: [Applications] okular Reporter: Munzir Taha <munzirtaha>
Component: generalAssignee: Okular developers <okular-devel>
Status: RESOLVED DOWNSTREAM    
Severity: major CC: aacid, gilbertd+kde, jr, kde, kevin.kofler, manisandro, perezmeyer, rdieter
Priority: NOR    
Version: 0.14.90   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: www.kde.org/.../QtContributionLicenseAgreement-1-1.pdf

Description Munzir Taha 2012-07-20 12:24:54 UTC
Searching for a text does not find any result.

Reproducible: Always

Steps to Reproduce:
1. Open Okular
2. Edit -> Find...
3. Type a word
Actual Results:  
Nothing found

Expected Results:  
Any word in the document should be found

munzir@GT683R:~$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu quantal (development branch)
Release:        12.10
Codename:       quantal
munzir@GT683R:~$ apt-cache policy okular
okular:
  Installed: 4:4.8.90-0ubuntu1
  Candidate: 4:4.8.90-0ubuntu1
  Version table:
 *** 4:4.8.90-0ubuntu1 0
        500 http://archive.ubuntu.com/ubuntu/ quantal/universe amd64 Packages
        100 /var/lib/dpkg/status
Comment 1 Dave Gilbert 2012-07-20 12:30:05 UTC
Same here, also 4:4.8.90-0ubuntu1 ubuntu quantal; Okular identifies itself as 0.14.90 , using KDE Development Platform 4.8.90 (4.8.90)

Just keeps spinning the rotating progress icon, never finishes, even on a couple of page doc.
Comment 2 Albert Astals Cid 2012-07-20 12:34:06 UTC
Please, attach a document with such a problem.
Comment 3 Dave Gilbert 2012-07-20 12:37:12 UTC
Created attachment 72647 [details]
www.kde.org/.../QtContributionLicenseAgreement-1-1.pdf

Try searching for anything in this (It's just the Contribution agreement of kde.org for qt); e.g.

ctrl-f
Agreement

never stops, shows feedback or anything.
Comment 4 Albert Astals Cid 2012-07-20 21:53:38 UTC
Works like a charm here, any change you can try with rc1 or rc2?
Comment 5 Dave Gilbert 2012-07-22 15:21:17 UTC
Still seems to happen for me with okular 0.15.70 on kde 4.8.97 (that I think is rc2) using the project-neon package build 2+git20120721+r4029-1~quantal1; this run is in a 32bit KVM guest, as opposed to my original run on 64bit host; so it's not word size or graphics card dependent.
Comment 6 Albert Astals Cid 2012-07-22 15:35:44 UTC
okular 0.15.70 is not rc2, if your package for rc2 contains that, tell your packager, rc2 is okular 0.14.97
Comment 7 Dave Gilbert 2012-07-22 15:52:54 UTC
OK, thanks - I don't have access to a specific rc2 set of .deb's - although if you have pointers I'd be interested.

Having said that, given that I'm able to recreate this on 0.14.90 prior to the rc's, and you're saying the 0.15.70 is after the rc's it would be surprising if it had been fixed just for the rc version.

(I probably won't have a chance to do another test until next weekend).
Comment 8 Albert Astals Cid 2012-07-22 16:09:37 UTC
Well, it works for me as i said in rc2

I'm not an ubuntu packager, the only thing i can say is you can try building from source tarball of the rc2 packages if you know how to do that.
Comment 9 Albert Astals Cid 2012-07-22 17:28:06 UTC
Adding the kubuntu packagers so they can see the fact that they decided to pckage rc2 is hurting our users
Comment 10 Dave Gilbert 2012-07-22 17:34:12 UTC
Albert:
  I've added an Ubuntu bug ( https://bugs.launchpad.net/ubuntu/+source/okular/+bug/1027657 )
that links back to this one so it can be tracked.

I think the only reason rc2 isn't packaged was a lack of resource.

Of course it's not obvious that it would work in rc2 either and it might be something more subtle.
Comment 11 Dave Gilbert 2012-07-22 17:48:41 UTC
One thing I hadn't previosly noticed (and don't know if is relevant); if I start okular on the command line and do the search I see:


QMetaObject::invokeMethod: No such method Okular::Document::doContinueNextMatchSearch(void *,void *,int,int,QString,int,bool,QColor,bool)

(both on the older and newer version I tried).
Comment 12 Albert Astals Cid 2012-07-22 18:14:43 UTC
Yes, this is very important.

Anyway i've confirmed with ubuntu developers that precise+okular from project neon can search and that quantal+okular from project neon can not search so it is a downstream (aka Ubuntu issue) and they'll have to deal with it since there's nothing i can fix in Okular itself
Comment 13 Dave Gilbert 2012-07-22 18:29:32 UTC
Ok thanks Albert.
Comment 14 Albert Astals Cid 2012-07-22 18:31:48 UTC
Thanks for caring about Okular, let's hope the kubuntu devels can find a fix soon.
Comment 15 Albert Astals Cid 2012-07-22 18:46:43 UTC
*** Bug 303933 has been marked as a duplicate of this bug. ***
Comment 16 Albert Astals Cid 2012-07-22 18:48:07 UTC
Rex, adding you since the duplicated 303933 affects fedora too, maybe you guys share some bad patch in Qt or gcc? People have confirmed it works fine in Arch gcc 4.7 and Qt 4.8.2
Comment 17 Rex Dieter 2012-07-22 19:27:16 UTC
I can't reproduce on my own f17/x86_64 box (ie, it searches pdf's just fine).  Is there a chance of a missing builld or runtime dependency or is okular just using facilities directly from poppler here?
Comment 18 Albert Astals Cid 2012-07-22 19:44:53 UTC
No poppler, it's just okular code.

The
QMetaObject::invokeMethod: No such method Okular::Document::doContinueNextMatchSearch(void *,void *,int,int,QString,int,bool,QColor,bool)
means something was very wrong on compilation/execution since that is never invoked, what is invoked is Okular::Document::doContinueNextMatchSearch(void *,void *,int,int,QString,int,bool,QColor,bool,int) in lines 1224 and 2956 of document.cpp so there is something wrong somewhere, i'd blame either gcc or qt, but to be honest i have no clue
Comment 19 Lisandro Damián Nicanor Pérez Meyer 2012-08-08 21:03:53 UTC
This bug has hitted qt4-x11/4:4.8.2-2 but no qt4-x11/4:4.8.2-1. Apart from the patches between them (two very small and contained), the only difference seems to be GCC.

I'm currently rebuilding qt4-x11/4:4.8.2-1 with current GCC to see if it gets triggered.
Comment 20 Rex Dieter 2012-08-08 21:19:20 UTC
Tracking downstream @ 
https://bugzilla.redhat.com/show_bug.cgi?id=846847

fedora users,  please  followup there.
Comment 21 Lisandro Damián Nicanor Pérez Meyer 2012-08-08 21:26:47 UTC
(In reply to comment #19)
> This bug has hitted qt4-x11/4:4.8.2-2 but no qt4-x11/4:4.8.2-1. Apart from
> the patches between them (two very small and contained), the only difference
> seems to be GCC.

I forgot to mention that this is Debian ;)

The Debian bug is in <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=684177>
Comment 22 Kevin Kofler 2012-08-08 21:56:50 UTC
GCC is also the main difference between Fedora 17 and 18 (gcc-4.7.0-5.fc17 vs. gcc-4.7.1-5.fc18). So it may well be a regression in GCC.
Comment 23 Lisandro Damián Nicanor Pérez Meyer 2012-08-08 22:08:33 UTC
Indeed, qt4-x11/4:4.8.2-1 was compiled with gcc-4.7.0-1 and qt4-x11/4:4.8.2-2 with gcc-4.7.1-2 (both Debian revisions).
Comment 24 Lisandro Damián Nicanor Pérez Meyer 2012-08-09 10:50:42 UTC
The problem is clearly a gcc bug. It has gone with version 4.7.1-6 currently in Debian Sid, you may want to look for patches there, although the correct one could be between versions -2 and -6 inclusive.

Kinds regards, Lisandro.
Comment 25 Albert Astals Cid 2012-08-09 10:55:20 UTC
Good work :-)

Is any of you guys reporting this upstream to gcc?
Comment 26 Lisandro Damián Nicanor Pérez Meyer 2012-08-09 11:34:14 UTC
(In reply to comment #25)
> Is any of you guys reporting this upstream to gcc?

I haven't spotted the right patch in gcc (nor I tried, to be honest). In other words, I don't have a clue what could have possibly changed.
Comment 27 Albert Astals Cid 2012-08-09 17:16:58 UTC
Well, but we have confirmed it's a gcc regression, no?

We can tell them and maybe they can find faster?
Comment 28 Kevin Kofler 2012-08-09 17:41:13 UTC
Comment #24 implies this is already fixed in current GCC, at least in Debian, but it probably got fixed by something from the GCC 4.7 branch. So I'm not sure reporting it to upstream GCC is useful.
Comment 29 Albert Astals Cid 2012-08-09 17:52:35 UTC
Oh, i can't read, i thought he had gone back in versions, not forward, ignore my suggestion then