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 169513 - Review Request: obby - A library which provides synced document buffers
Summary: Review Request: obby - A library which provides synced document buffers
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
URL: http://releases.0x539.de/obby
Whiteboard:
Depends On: 169512
Blocks: FE-ACCEPT 169514
TreeView+ depends on / blocked
 
Reported: 2005-09-29 06:53 UTC by Luke Macken
Modified: 2016-09-20 02:36 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-10-06 17:21:03 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Luke Macken 2005-09-29 06:53:57 UTC 
Spec Name or Url: http://people.redhat.com/lmacken/obby.spec
SRPM Name or Url: http://people.redhat.com/lmacken/obby-0.2.0-1.src.rpm
Description:
libobby is a library which provides synced document buffers. It supports
multiple documents in one session and is portable to both Windows and
Unix-like platforms.
Comment 1 Luke Macken 2005-09-30 01:52:10 UTC 
Added gmp-devel to BuildRequires.
Comment 2 Adrian Reber 2005-09-30 05:58:23 UTC 
After a quick glance at the spec file I can see the same issues as with net6:

-devel requires at least pkgconfig

use the %find_lang macro
Comment 3 Luke Macken 2005-09-30 06:26:08 UTC 
Done.

http://people.redhat.com/lmacken/obby.spec
http://people.redhat.com/lmacken/obby-0.2.0-2.src.rpm
Comment 4 Adrian Reber 2005-10-05 07:14:21 UTC 
-devel subpackage needs to require: net6-devel as the include files from
obby-devel include net6 headers files.

The BuildRequires libsigc++20-devel is not necessary as this is already required
by net6-devel.

The rest looks good.
Comment 5 Luke Macken 2005-10-05 07:50:02 UTC 
Updated.

http://people.redhat.com/lmacken/gobby.spec
http://people.redhat.com/lmacken/gobby-0.2.2-1.src.rpm
Comment 6 Luke Macken 2005-10-05 07:50:51 UTC 
Fixed srpm: http://people.redhat.com/lmacken/gobby-0.2.2-3.src.rpm
Comment 7 Adrian Reber 2005-10-05 11:52:38 UTC 
Forgot one more thing: the -devel subpackage also requires gmp-devel as it can
be seen it buffer.hpp and rsa.hpp

By the way: SRPM gives a 404 and the spec file is the SRPM.
Comment 8 Luke Macken 2005-10-05 15:42:47 UTC 
updated.. links should be good now.

http://people.redhat.com/lmacken/gobby.spec
http://people.redhat.com/lmacken/gobby-0.2.2-3.src.rpm
Comment 9 Rex Dieter 2005-10-05 16:10:59 UTC 
Per Adrian's comment #2, I'd suggest adding to %install:
%find_lang %{name}
and to %files:
%files -f %{name}.lang

instead of hard-coding
%{_datadir}/locale/*/LC_MESSAGES/gobby.mo
Comment 10 Rex Dieter 2005-10-05 16:39:37 UTC 
Updating bugzilla summary to match package name/summary.
Comment 11 Rex Dieter 2005-10-05 16:45:17 UTC 
OK, it appears there are 2 packages for Review here (so I got confused):
* obby: library
* gobby: editor, which BuildRequires: obby-devel

For reviewers' sanity, I'd suggest submitting each in its own bugzilla entry.
Comment 12 Luke Macken 2005-10-05 18:01:34 UTC 
(In reply to comment #11)
> OK, it appears there are 2 packages for Review here (so I got confused):
> * obby: library
> * gobby: editor, which BuildRequires: obby-devel
> 
> For reviewers' sanity, I'd suggest submitting each in its own bugzilla entry.

Obby: This bug
Gobby: Bug #169514

I'm to blame for this confusion seeing as I pasted links to gobby spec/srpm
starting in comment #5; all of the review in this bug is for the obby library.
Comment 13 Luke Macken 2005-10-05 18:05:22 UTC 
Latest spec/srpm:

http://people.redhat.com/lmacken/obby-0.2.0-3.src.rpm
http://people.redhat.com/lmacken/obby.spec

Sorry for the confusion guys.
Comment 14 Adrian Reber 2005-10-06 06:18:27 UTC 
* builds in mock
* rpmlint is happy
* correct usage of -devel
* spec looks good
* clean installation and removal
* use of find_lang macro
* correct Requires and BuildRequires
* scriptlets look sane

APPROVED
Comment 15 Luke Macken 2005-10-06 17:21:03 UTC 
* Imported into cvs
* Added to owners.list
* Requested FC3/FC4 branches
* Build sucessfully in devel branch

Thanks!
Comment 16 Adrian Reber 2005-10-07 08:18:19 UTC 
You could add --with-howl to configure and the BR howl-devel to the spec for
FC-4 and greater.
Comment 17 Luke Macken 2005-10-07 16:37:54 UTC 
Built howl support into the devel branch.  Will add it to FC4 shortly after.
Note You need to log in before you can comment on or make changes to this bug.