Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 165960
Summary: | Review Request: gnome-translate - A natural language translator | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Dmitry Butskoy <dmitry> |
Component: | Package Review | Assignee: | Aurelien Bompard <gauret> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review |
Target Milestone: | --- | Flags: | gwync:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://www.nongnu.org/libtranslate/gnome-translate | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-09-12 17:02:55 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 165353 | ||
Bug Blocks: | 163779 |
Description
Dmitry Butskoy
2005-08-15 10:43:48 UTC
Needs work: * BuildRoot: should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) * Build: missing SMP flags ( make %{?_smp_mflags} ) * Duplicate buildRequires: glib2-devel, gtk2-devel, GConf2-devel, libgnomeui-devel, gnome-vfs2-devel, libglade2-devel, libxml2-devel, pango-devel * Desktop file: vendor should be fedora * Desktop file: the Categories tag should contain X-Fedora, not X-Red-Hat-Base * %install : /var should be replaced by %{_localstatedir} * Check the required post and postun scriptlets on http://fedoraproject.org/wiki/ScriptletSnippets ("GConf" section and "GTK+ icon cache" section) * %makeinstall should be make install DESTDIR=$RPM_BUILD_ROOT * Remove the explicit dependency on libtranslate, rpm will pick it up by itself. All remarks are done, except I yet have not understood about "Duplicate BuildRequires". Give me please little bit more information on it... New SRPM: http://dmitry.butskoy.name/libtranslate/gnome-translate-0.99-2.src.rpm New SPEC: http://dmitry.butskoy.name/libtranslate/gnome-translate.spec Sure. For example, libgnomeui-devel already depends on GConf2-devel, glib2-devel, and gnome-vfs2-devel, so you don't have to list those explicitely. All the BR I have listed are already required by other BRs, so you can remove them. OK I yet leave gtk2-devel and GConf2-devel, because they requires '>= 2.4.0', but implicit dependencies give '>= 2.3.0' only. New SRPM and SPEC at the same location (comment #2) (No release increment due to nobody still download it... :-)) Sooner or later you will run into the trap of not updating the versioned Buildrequires, and then you will be out-of-sync. Notice that even FC2 comes with a version of gtk2-devel, which is new enough, so all the distributions your Fedora Extras package targets, are sufficient. Versioned [build] dependencies make much more sense, when the package from original version of a distribution is insufficient, but an Update is available and required. For comment #5 : It is possible that someone will rebuild this package under some "old enough" distribution (fc1, rhl7.3 etc.), or even under some another Fedora-based distribution. Therefore I think this versioning is a little bit useful. I hope that it is not required to update these dependences in the future (at least, visible future), because it is ">=", not "==" ... > No release increment due to nobody still download it... Please do increment. It helps when saying "Release x approved", and it also helps the reviewer. > I hope that it is not required to update these dependences in the future If the upstream project depends on newer libs, you'll have to reflect this in your RPM. But if you're willing to follow that, that's fine. You're the maintainer in the end. Last problem in the spec file: since it installs a scrollkeeper file, you have to add the required bits in the scriptlets. Please see http://fedoraproject.org/wiki/ScriptletSnippets, scrollkeeper section. Done. New SRPM: http://dmitry.butskoy.name/libtranslate/gnome-translate-0.99-3.src.rpm New SPEC: http://dmitry.butskoy.name/libtranslate/gnome-translate.spec === Review for release 3 === * RPM name is OK * Source gnome-translate-0.99.tar.gz is the same as upstream * Builds fine in mock * rpmlint of gnome-translate looks OK * File list of gnome-translate looks OK * Seems to work fine Package Change Request ====================== Package Name: gnome-translate New Branches: el6 Owners: buc Git done (by process-git-requests). |