Bug 1338 - kchmviewer, a chm file viewer
Summary: kchmviewer, a chm file viewer
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: New RPM package request (show other bugs)
Version: 1
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL: http://www.kchmviewer.net/
Whiteboard:
Keywords: validated_update
: 1468 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-05-19 08:17 CEST by shankar rao
Modified: 2011-09-09 20:06 CEST (History)
11 users (show)

See Also:
Source RPM: not applicable
CVE:
Status comment:


Attachments
Encoding problem in kchmviewer (105.39 KB, image/png)
2011-07-31 15:22 CEST, Samuel Verschelde
Details
Image of menu showing icon (37.13 KB, image/jpeg)
2011-08-01 23:49 CEST, Dave Hodgins
Details

Description shankar rao 2011-05-19 08:17:30 CEST
new package request 

program name: kchmviewer

latest version : 5.2

released : April 27 2010.
shankar rao 2011-05-19 08:20:03 CEST

Summary: new package request => new package request : kchmviewer

shankar rao 2011-05-19 08:22:25 CEST

CC: (none) => lateralridge-a

Comment 1 Ahmad Samir 2011-05-20 04:24:37 CEST
Probably too late to catch Mageia1. Once the new cycle of Cauldron starts it'll be packaged and pushed to Mageia 1 backports.
Comment 2 Ahmad Samir 2011-05-29 03:30:48 CEST
*** Bug 1468 has been marked as a duplicate of this bug. ***

CC: (none) => luigiwalser

Michael Scherer 2011-06-05 17:47:24 CEST

CC: (none) => misc
Summary: new package request : kchmviewer => kchmviewer, a chm file viewer

Comment 3 Ahmad Samir 2011-06-07 23:04:31 CEST
Package imported in Cauldron; leaving the report open until it's backported to Mageia 1 (should be soon).

Assignee: bugsquad => ahmadsamir3891

Comment 4 Ahmad Samir 2011-06-27 01:04:54 CEST
The package should land in the "Core Backports Testing" repository soon, please test it.

Keywords: (none) => NEEDINFO

Comment 5 Dave Hodgins 2011-06-27 04:00:51 CEST
Tested on my i586 system. Once tested on a 64 bit system, kchmviewer is ready
to move from Core Backports Testing to Core Backports

CC: (none) => davidwhodgins

Comment 6 David Walser 2011-07-02 18:31:20 CEST
I can also confirm that the package is working fine.  Thanks!
Ahmad Samir 2011-07-02 18:37:37 CEST

Keywords: NEEDINFO => (none)
Assignee: ahmadsamir3891 => qa-bugs

Comment 7 José Jorge 2011-07-02 20:57:11 CEST
Tested on x86_64 : Ok.

CC: (none) => lists.jjorge

Comment 8 Dave Hodgins 2011-07-02 22:00:46 CEST
Can someone from the sysadmin team push the package
kchmviewer
from Core Backports Testing to Core Backports please.

The srpm is kchmviewer-5.2-0.1.mga1.src.rpm
Comment 9 Samuel Verschelde 2011-07-31 00:16:36 CEST
This package was present in Mandriva 2010.2 so it can go to updates_testing rather than backports_testing. Ahmad, do you want me to take care of it ?

CC: (none) => stormi

Comment 10 Ahmad Samir 2011-07-31 07:50:04 CEST
kchmviewer-5.2-2.mga1 should land in core/updates_testing shortly.
Comment 11 Samuel Verschelde 2011-07-31 15:21:47 CEST
It works (tested on i586), but :
* I have encoding problems (no problems with the same file in okular except that okular takes ages to open it). See attached screenshot. The test file is the french php manual : http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror
* .chm files are not associated with kchmviewer
Comment 12 Samuel Verschelde 2011-07-31 15:22:19 CEST
Created attachment 695 [details]
Encoding problem in kchmviewer
Comment 13 David Walser 2011-07-31 15:28:35 CEST
Look in the Settings and in Version-specific settings, what do you have selected under To show HTML content, use?  If it's QTextBrowser, try changing it t KHTMLPart.  I had to do that to get it to work correctly.
Comment 14 Samuel Verschelde 2011-07-31 15:48:04 CEST
(In reply to comment #11)
> It works (tested on i586), but :
> * I have encoding problems (no problems with the same file in okular except
> that okular takes ages to open it). See attached screenshot. The test file is
> the french php manual :
> http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror
> * .chm files are not associated with kchmviewer

Also, kchmviewer has no icon in the menu.
Comment 15 Samuel Verschelde 2011-07-31 15:50:31 CEST
(In reply to comment #13)
> Look in the Settings and in Version-specific settings, what do you have
> selected under To show HTML content, use?  If it's QTextBrowser, try changing
> it t KHTMLPart.  I had to do that to get it to work correctly.

Better indeed. Maybe there's a way to set that as the default ?
Comment 16 Ahmad Samir 2011-07-31 21:46:34 CEST
-3.mga1:
- Add a patch to make it use KHTMLPART by default
- Install some more icons

kchmviewer is associated with .chm files here, using dolphin.
Comment 17 David Walser 2011-07-31 22:35:24 CEST
kchmviewer is associated with .chm files in Dolphin and Konqueror, although Okular is the default association, as .chm files sort of work in Okular (they work better in kchmviewer).
Comment 18 Samuel Verschelde 2011-07-31 22:38:22 CEST
(In reply to comment #16)
> kchmviewer is associated with .chm files here, using dolphin.

Right, now it's ok, dolphin mhad a lag concerning file associations
Comment 19 Samuel Verschelde 2011-07-31 22:39:04 CEST
Hey, computer, I didn't want to validate this previous comment !
Comment 20 Dave Hodgins 2011-08-01 00:10:26 CEST
Testing of ths srpm
kchmviewer-5.2-3.mga1.src.rpm
From Core Updates Testing complete on i586.

The menu entry is available under the Office entry.
Opening a .chm file from dolphin or konqueror works with a right click/open with.
Comment 21 Samuel Verschelde 2011-08-01 21:55:42 CEST
Everything is ok except icons, I still don't see any icon next to the menu entry. Do others see the icon (should be a green icon showing a book put on a second book) ?
Comment 22 David Walser 2011-08-01 23:39:44 CEST
Don't forget to add your fixes to the Cauldron version of the package also.
Comment 23 Dave Hodgins 2011-08-01 23:49:42 CEST
Created attachment 702 [details]
Image of menu showing icon

Regarding Comment 21, the attached image shows the tiny green icon on
menu of my i586 system.
Comment 24 Samuel Verschelde 2011-08-02 09:31:23 CEST
(In reply to comment #23)
> Created attachment 702 [details]
> Image of menu showing icon
> 
> Regarding Comment 21, the attached image shows the tiny green icon on
> menu of my i586 system.

Here on another computer the icon is fine. I don't know why it would update on the other computer, even after removing then reinstalling the software.

Consider the icon issue fixed.
Comment 25 Samuel Verschelde 2011-08-04 19:11:40 CEST
Testing complete on i586, could someone test on x86_64 ?
Comment 26 Derek Jennings 2011-08-05 23:41:06 CEST
Testing complete on x86_64

Okular is  the default viewer. The test document I was using would not open in okular (locked up), but opens OK on kchmviewer
http://www.help-info.de/download/CHM-example.chm

CC: (none) => derekjenn

Comment 27 Dave Hodgins 2011-08-06 00:09:11 CEST
Can someone from the sysadmin team push the srpm
kchmviewer-5.2-3.mga1.src.rpm
From Core Updates Testing to Core Updates and
remove the version that's in Core Backports Testing.

Advisory:  This update adds KchmViewer, a chm (MS HTML help file format) viewer,
which was present in Mandriva 2010.2 but missing in Mageia 1.

Keywords: (none) => validated_update
CC: (none) => mageia-sysadm

Comment 28 David GEIGER 2011-08-06 08:33:36 CEST
(In reply to comment #11)

>The test file is the french php manual :
> http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror
> * .chm files are not associated with kchmviewer


*Tested on Mageia release 1 (Official) for x86_64 and
It does not work with me ,kchmviewer crash when opening the file:
http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror

*Here The result in console:

[david@david ~]$ kchmviewer
QPainter::end: Painter ended with 2 saved states
QPainter::end: Painter ended with 2 saved states
KCrash: Application 'kchmviewer' crashing...
KCrash: Attempting to start /usr/lib64/kde4/libexec/drkonqi from kdeinit
sock_file=/home/david/.kde4/socket-david.david/kdeinit4__0

[1]+  Stoppé                 kchmviewer

*Here is the error of fault manager for KDE:

Application: kchmviewer (kchmviewer), signal: Segmentation fault
[KCrash Handler]
#6  0x00007f8b8249b102 in __strlen_sse2 () from /lib64/libc.so.6
#7  0x00007f8b830b9a0b in QTextCodec::toUnicode(char const*) const () from /usr/lib64/libQtCore.so.4
#8  0x0000000000449947 in _start ()


*By cons, it works with this file:

http://www.help-info.de/download/CHM-example.chm

*I have the icon in the menu-> Applications -> Office

CC: (none) => geiger.david68210

Comment 29 Dave Hodgins 2011-08-06 09:24:32 CEST
http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror
opens fine in kchmviewer on my i586 system.

David GEIGER, what version of KDE do you have installed?

I have ...
$ kde4-config -version
KDE Development Platform: 4.6.5 (4.6.5)
Comment 30 David GEIGER 2011-08-06 09:52:27 CEST
(In reply to comment #29)
> 
> David GEIGER, what version of KDE do you have installed?
> 
> I have ...
> $ kde4-config -version
> KDE Development Platform: 4.6.5 (4.6.5)

Sorry,I did not specify the versions.

I have :
$ kde4-config -version
Plate-forme de développement de KDE : 4.6.5 (4.6.5)

$ rpm -qa | grep kchmviewer
kchmviewer-5.2-3.mga1
Comment 31 Derek Jennings 2011-08-07 11:49:08 CEST
I get the same crash viewing 
http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror
on my x86_64
Comment 32 Samuel Verschelde 2011-08-09 00:51:03 CEST
un-validating the update 

Setting version to mageia 1 to make it appear in saved searches.

Keywords: validated_update => (none)
Version: Cauldron => 1

Comment 33 Samuel Verschelde 2011-08-09 00:52:28 CEST
(In reply to comment #32)
> un-validating the update 

unfinished phrase : I meant "un-validated the update per comment #28"
Comment 34 David Walser 2011-08-09 00:53:26 CEST
Why un-validate it?  Because it crashes on one file on one platform?  It works on plenty of files and certainly works better than Okular.  The crash is probably an upstream bug.  That can be dealt with later.  Why not push the package?  Having it, when it works well for all but this one case is better than not having it at all.
Comment 35 Samuel Verschelde 2011-08-09 01:09:33 CEST
(In reply to comment #34)
> Why un-validate it?  Because it crashes on one file on one platform?  It works
> on plenty of files and certainly works better than Okular.  The crash is
> probably an upstream bug.  That can be dealt with later.  Why not push the
> package?  Having it, when it works well for all but this one case is better
> than not having it at all.

Because we don't know on how many computers it will crash, so potentially this could give a non-working app for many people. Maybe you're right and this is a very isolated bug, but I think it is better to check that before submitting the update. Also, maybe the problem is as simple as a missing requires and would be easy to solve. We can't know until we look at it closer. 

Also, 2 different testers had the crash so it's a bug that can be reproduced, which gives hope that we can find the reason of the crash without too much trouble.

However, it would be good to have the crash studied very quickly so that the update is not delayed too much.

Ahmad, can you take care of it ?
Manuel Hiebel 2011-09-02 22:39:41 CEST

CC: mageia-sysadm => ahmadsamir3891

Comment 36 Samuel Verschelde 2011-09-09 00:31:37 CEST
I made a quick search (should have done it before, but you know, things take time and sometimes you think it would take more time than it really takes...) and it looks like people were having the same problem with the german php doc 2 years ago :

http://kde-apps.org/content/show.php?content=25125

So this definitely looks like an upstream bug. I'll update kchmviewer to the latest version in case it fixes the bug and if not we'll validate the update regardless of this bug and create a separate bug report for it.
Comment 37 Samuel Verschelde 2011-09-09 01:16:17 CEST
I pushed kchmviewer-5.3-1.1.mga1 to Core Updates Testing. Please test :
- that it works well 
- whether the crash still occurs (David Geiger and Derek Jennings) or not
Comment 38 Dave Hodgins 2011-09-09 04:05:46 CEST
I've tested kchmviewer-5.3-1.1.mga1.src.rpm on my i586 system, and
it's working.

As in comment 29, it does not crash here.  Still need testing on x86-64.
Comment 39 Dave Hodgins 2011-09-09 04:08:40 CEST
When pushed to updates, kchmviewer-5.2-0.1.mga1.src.rpm should be removed
from Core Backports Testing.
Comment 40 Derek Jennings 2011-09-09 13:54:39 CEST
Still has the same crash observed in the previous version but otherwise works fine on x86_64, validating.


Can someone from the sysadmin team please push the srpm
kchmviewer-5.3-1.1.mga1.src.rpm
From Core Updates Testing to Core Updates and
remove the version that's in Core Backports Testing.

Advisory:  This update adds KchmViewer, a chm (MS HTML help file format)
viewer, which was present in Mandriva 2010.2 but missing in Mageia 1. Some documents have been observed to crash kchmviewer on x86_64 systems due to an upstream bug. If you have problems on x86_64 install the i586 package.

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs
Hardware: i586 => All

Comment 41 D Morgan 2011-09-09 15:38:43 CEST
update pushed.

Status: NEW => RESOLVED
CC: (none) => dmorganec
Resolution: (none) => FIXED

Comment 42 David GEIGER 2011-09-09 17:07:17 CEST
(In reply to comment #37)
> I pushed kchmviewer-5.3-1.1.mga1 to Core Updates Testing. Please test :
> - that it works well 
> - whether the crash still occurs (David Geiger and Derek Jennings) or not

Hello,

Tested kchmviewer-5.3-1.1.mga1 on Mageia release 1 (Official) for x86_64 ,For me it crashes again when opening the file:
http://www.php.net/get/php_enhanced_fr.chm/from/fr.php.net/mirror



*Here is the error of fault manager for KDE:

Application: kchmviewer (kchmviewer), signal: Segmentation fault
[KCrash Handler]
#6  0x00007f3297495102 in __strlen_sse2 () from /lib64/libc.so.6
#7  0x00007f32980b3a0b in QTextCodec::toUnicode(char const*) const () from /usr/lib64/libQtCore.so.4
#8  0x0000000000449947 in _start ()


*Here The result in console:

[david@david ~]$ kchmviewer
KCrash: Application 'kchmviewer' crashing...
KCrash: Attempting to start /usr/lib64/kde4/libexec/drkonqi from kdeinit
sock_file=/home/david/.kde4/socket-david.david/kdeinit4__0

[1]+  Stoppé                 kchmviewer


$ rpm -qa | grep kchmviewer
kchmviewer-5.3-1.1.mga1
Comment 43 Samuel Verschelde 2011-09-09 20:06:47 CEST
bug #2685 submitted for the crash problem

Note You need to log in before you can comment on or make changes to this bug.