SRPM Name or Url: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.4-1.src.rpm Description: xCHM is a wxWidgets-based .chm viewer. xCHM can show the contents tree if one is available, print the displayed page, change fonts faces and size, work with bookmarks, do the usual history stunts (forward, back, home), provide a searchable index and seach for text in the whole book. The search is a fast B-tree search, based on the internal $FIftiMain file found inside indexed .chm archives, and it can be customized to search in content or just the topics' titles.
What about wxGTK? There is wxGTK < 2.6 prevents update.
Could you be a bit more precise, please. What update does it prevent? The xchm version I packaged Buildrequires wxGTK-devel >= 2.6, but I didn't add an explicit version in the spec file, because it is intended for devel. It is more clean to do it, though. Updated at: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.4-2.src.rpm
I mean that FE current repo does not consist the wxGTK 2.6. It's a problem.
Oh, I see. But the current xchm release requires wxGTK 2.6. I would have liked to have xchm in FE3/4. I plan on submitting wxGTK26 for FE4, I'll see if I can complete it.
Oops, in the updated srpm, I added support for xmlrpc, but I don't think it is a good idea yet... Please use this srpm instead: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.4-3.src.rpm And indeed it only builds and installs in FC devel.
Why do you not build last xchm (now is the 1.8)?
Because the review request is from about 2006-03-10. I'll update.
New version here: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.8.src.rpm
New version here: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.8-1.src.rpm
New version available: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xchm-1.9-1.src.rpm
Hi, I will review this package. ---------------------------------------- Review for release 1: * RPM name is NO You need to have %{?dist} in the release tag. * This is the latest version * Builds fine in mock (rawhide) * rpmlint looks OK * File list looks OK Notes: Looks good. I am happy to APPROVE this submission, just fix up the release tag. Please close this report once you have imported the package into CVS. Just a note, in the future, if you can use the following example for submitting both srpms and spec, it will make it easier for people to quickly review your submissions: Spec URL: http://www.example.com/extras/foo.spec SRPM URL: http://www.example.com/extras/foo-1.2.3-1.src.rpm
(In reply to comment #11) > * RPM name is NO > > You need to have %{?dist} in the release tag. If I'm not wrong this is not an obligation. However I always use %{?dist}, it is just a mistake of mine. I'll change it right after importing to cvs. > > Looks good. I am happy to APPROVE this submission, just fix up the release tag. Thanks. > Just a note, in the future, if you can use the following example for submitting > both srpms and spec, it will make it easier for people to quickly review your > submissions: > > Spec URL: http://www.example.com/extras/foo.spec > SRPM URL: http://www.example.com/extras/foo-1.2.3-1.src.rpm Ok. I am lazy, but you are the second one making that request, so I'll change my bad habits ;-). Thanks.
Built in devel, branche requested in FC-5. Thanks!