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 195412 (obconf) - Review Request: obconf
Summary: Review Request: obconf
Keywords:
Status: CLOSED ERRATA
Alias: obconf
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: cq92j9y+rlkr0w
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: openbox
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-06-15 03:09 UTC by Peter Gordon
Modified: 2012-04-08 17:03 UTC (History)
4 users (show)

Fixed In Version: obconf-2.0.3-9.20100212gitb04658.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-23 04:19:34 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Peter Gordon 2006-06-15 03:09:03 UTC
[ Resubmitting review request due to Bugzilla's earlier troubles. ]

Spec URL: http://www.thecodergeek.com/downloads/fedora/obconf.spec
SRPM URL: http://www.thecodergeek.com/downloads/fedora/obconf-1.6-1.src.rpm

Description: ObConf is a graphical configuration editor for the Openbox window manager.

Comment 1 Parag AN(पराग) 2006-06-16 06:30:05 UTC
Review for this package:-
MUST Items:
     - MUST: rpmlint shows no error 
     - MUST: The package is named according to the Package Naming Guidelines.
     - MUST: The spec file name matching the base package obconf, in the format
obconf.spec
      - MUST: This package meets the Packaging Guidelines.
      - MUST: The package is licensed with an open-source compatible license GPL.
      - MUST: The License field in the package obconf.spec file matches the
actual license in COPYING file in tarball.
      - MUST: The sources used to build the package matches the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task.
      - MUST: The package successfully compiled and build into binary rpms on i386.
      - MUST: This package owns all directories that it creates. 
      - MUST: This package did not contain any duplicate files in the %files
listing.
      - MUST: This package  have a %clean section, which contains rm -rf
%{buildroot}.
      - MUST: This package used macros.
      - MUST: Document files are included.
      - MUST: This Package did not contained any .la libtool archives
      - MUST: This Package include a obconf.desktop file, and that file is 
installed with desktop-file-install in the %install section with following warning
obconf.desktop: key "Categories" string list not semicolon-terminated, fixing
      - MUST: No duplicate files in installation

Comment 2 Peter Gordon 2006-06-16 14:34:20 UTC
Adding bug dependency on Openbox's review request, as this uses its -devel
subpackage as a build requirement.

Comment 3 Parag AN(पराग) 2006-06-19 10:57:43 UTC
Above is Not an official review as I'm not yet sponsored

Comment 4 cq92j9y+rlkr0w 2006-07-13 04:20:17 UTC
Review:

+ rpmlint shows no error.
+ package meets the naming guidelines.
+ spec-file is properly named.
+ package meets the packaging guidelines.
+ package license is open-source compatible (GPL).
+ license field matches the actual license.
+ license file included in %doc.
+ spec file is written in english.
+ spec file is legible.
+ source files match upstream:
    aaf62498b11d52dfce7a0b6060867a19  obconf-1.6.tar.gz
+ package successfully compiled, built and tested on i386 (rawhide).
+ all build dependencies are listed in BuildRequires.
+ package doesn't need to use %find_lang (no locales present).
+ package doesn't contain shared libraries.
+ package isn't relocatable.
+ package owns all directories that it creates.
+ no duplicate files in %files.
+ file permissions are properly set.
+ package has a %clean section.
+ package uses macros consistently.
+ package contains code, not content.
+ no -doc subpackage needed.
+ %docs don't affect application runtime.
+ package doesn't contain headers, static libraries or pkgconfig files (no devel
package).
? GUI application; desktop file is installed with a small warning (as pointed
out by Parag AN): I don't think this issue is worth patching the file (specially
since it is automatically fixed) but I do think a bug should be filed upstream.
+ package doesn't own directories owned by other packages. 

Comment 5 cq92j9y+rlkr0w 2006-07-13 23:18:25 UTC
Peter:

Don't forget to report a bug upstream for the .desktop semicolon issue.


APPROVED.

Comment 6 Peter Gordon 2006-07-14 02:24:45 UTC
Built for Devel. Once it's been branched to FC-4 and FC-5, I'll build for those
and close this as NEXTRELEASE.

Jorge: I have contacted the author about it. Thanks for your review. :)

Comment 7 Peter Gordon 2006-07-23 04:19:34 UTC
Built successfully for FC-4 and FC-5 branches too. Thanks for your time. 

Comment 8 Christian Iseli 2006-12-31 11:30:06 UTC
Please do not remove the FE-ACCEPT blocker.  Thanks.


Comment 9 Peter Gordon 2007-06-02 21:21:06 UTC
Package Change Request
======================
Package Name: obconf
Updated Fedora Owners: extras-orphan

I'm orphaning openbox, obconf, and obmenu as I no longer use them and feel that
my time is better spent dedicated to my other packages. Thanks.

Comment 10 Tom "spot" Callaway 2007-06-04 21:53:48 UTC
Orphaned.

Comment 11 Miroslav Lichvar 2007-06-13 08:07:29 UTC
Package Change Request
======================
Package Name: obconf
Updated Fedora Owners: mlichvar

Comment 12 Miroslav Lichvar 2012-03-22 16:11:09 UTC
Package Change Request
======================
Package Name: obconf
New Branches: el6
Owners: mlichvar splinux

Comment 13 Gwyn Ciesla 2012-03-22 16:43:16 UTC
Git done (by process-git-requests).

Comment 14 Fedora Update System 2012-03-22 22:27:58 UTC
obconf-2.0.3-9.20100212gitb04658.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/obconf-2.0.3-9.20100212gitb04658.el6

Comment 15 Fedora Update System 2012-04-08 17:03:49 UTC
obconf-2.0.3-9.20100212gitb04658.el6 has been pushed to the Fedora EPEL 6 stable repository.


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