This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 458698 - Review Request: libgdbus - D-Bus helper library
Review Request: libgdbus - D-Bus helper library
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Parag AN(पराग)
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2008-08-11 13:11 EDT by Marcel Holtmann
Modified: 2008-10-16 01:49 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-10-16 01:49:55 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:

Attachments (Terms of Use)

  None (edit)
Description Marcel Holtmann 2008-08-11 13:11:50 EDT
Spec URL:
Description: libgdbus is a helper library for D-Bus integration with GLib
Comment 1 Parag AN(पराग) 2008-08-12 01:32:22 EDT
1) remove following from %files

2) Your build section should look like this
%configure --disable-static
make %{?_smp_mflags}

3) Add "Requires: pkgconfig" to devel package
From Review Guidelines you can see
- MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for directory ownership and usability).

4) rpmlint complain that 
libgdbus.i386: E: zero-length /usr/share/doc/libgdbus-0.1/NEWS
==> Remove NEWS from %docs

5) you should drop following line from SPEC
/sbin/ldconfig -n $RPM_BUILD_ROOT/%{_libdir}
Comment 2 Marcel Holtmann 2008-08-12 02:06:26 EDT
Thanks for the review.

Spec URL:
Description: libgdbus is a helper library for D-Bus integration with GLib
Comment 3 Dan Horák 2008-08-12 02:27:37 EDT
Reassigning back to Parag, the "Review request" bug should be assigned to reviewer during its whole life.

And one note - you should remove the explicit "Requires: glib2 dbus" as such dependencies are created automagically during the build process.
Comment 4 Parag AN(पराग) 2008-08-12 02:30:44 EDT
aah. All I can see about you is that this is your first package submission to
Fedora. If not then tell me which packages you are/was maintaining. 
You should look first

You actually removed me from official review process. Have you done enough
package reviews before? I see that you are already member of packager group.

Thanks danny for your comment.
Comment 5 Marcel Holtmann 2008-08-12 03:11:02 EDT
Fixed the Requires mistake.

Spec URL:
Description: libgdbus is a helper library for D-Bus integration with GLib
Comment 6 Parag AN(पराग) 2008-08-13 05:48:02 EDT
1) I think you should also keep timestamps using following command
    make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"

2) I see there is no documentation/APIs for this library but there are some example files in test directory. you should also add them to %doc of -devel package. Don't include irrelevant files from test/* to %doc

3) Drop INSTALL from %doc as its really of no use.

BTW, Is this your first package submission?
Comment 7 Parag AN(पराग) 2008-10-06 01:15:01 EDT
any updates here?
Comment 8 Parag AN(पराग) 2008-10-11 23:10:11 EDT
Will Close this as NOTABUG by this Thursday if updates are not submitted.
Comment 9 Parag AN(पराग) 2008-10-16 01:46:06 EDT
marking as DEADREVIEW as no updates from submitter.

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