[Bug 225826] New: Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

Summary: Merge Review: gnome-netstatus
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: normal
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail]

Fedora Merge Review: gnome-netstatus

http://cvs.fedora.redhat.com/viewcvs/devel/gnome-netstatus/
Initial Owner: [e-mail]

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review-

------- Additional Comments From [e-mail]  2007-02-09 10:27 EST -------
Hi,

NEEDSWORK:
 * Fail to build in mock (rawhide x86_64), seems to need a BR on libxslt;
<<
make[3]: Leaving directory `/builddir/build/BUILD/gnome-netstatus-2.12.0/help'
xsltproc -o gnome-netstatus-C.omf --stringparam db2omf.basename gnome-netstatus
--stringparam db2omf.format 'docbook' --stringparam db2omf.dtd "-//OASIS//DTD
DocBook XML V4.1.2//EN" --stringparam db2omf.lang C --stringparam db2omf.omf_dir
"/usr/share/omf" --stringparam db2omf.help_dir "/usr/share/gnome/help"
--stringparam db2omf.omf_in "`pwd`/./gnome-netstatus.omf.in"
`/usr/bin/pkg-config --variable db2omf gnome-doc-utils` C/gnome-netstatus.xml
db2omf: Could not construct the OMF maintainer element.
  Add an author, corpauthor, editor, othercredit, or publisher
  element with the role attribute set to "maintainer" to gnome-netstatus.xml.
make[2]: *** [gnome-netstatus-C.omf] Error 10
make[2]: Leaving directory `/builddir/build/BUILD/gnome-netstatus-2.12.0/help'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/builddir/build/BUILD/gnome-netstatus-2.12.0'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.59866 (%build)
> >

* The BuildRoot tag is very short, it doesn't conform to the packaging guildelines
 * Is "Prereq: gtk2 >..." neccesary at all? And also the explicit Requires on
gtk2, libglade2, libgnomeui, and gnome-panel
 * Not very sure about this (since the build failed for me half-way), does the
build really depends on gnome-vfs2, maybe that BR ought to be gnome-vfs2-devel.
 * Packaging guildelines discourage the use of '%makeinstall' macro, but advises
using 'make DESTDIR=$RPM_BUILD_ROOT install' instead.

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

------- Additional Comments From [e-mail]  2007-02-09 11:41 EST -------
I've done some cleanups in the spec now.

libxslt gets pulled in by gnome-doc-utils, your problem is something else. 
I'll see if the buildsystem is more successful in getting it built.

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review-              |fedora-review?

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium

------- Additional Comments From [e-mail]  2007-06-17 00:43 EST -------
Stalled review

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Product|Fedora Extras               |Fedora

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2007-06-19 16:14 EST -------
I'm sorry for the stalled review, I'd thought it was resolved.

GOOD:
 * Build Ok in mock (x86_64)
 * License (GPL) and rpm Group tag OK
 * Naming meets the packaging guildlines
 * Spec file clean and legible
 * Handles locales correctly
 * Build dependencies properly listed
 * Source file matches upstream's
[deji@agape reviews]$ md5sum gnome-netstatus-2.12.1.tar.bz2 
8651ca1694a6c222ae5cad6e21814d24  gnome-netstatus-2.12.1.tar.bz2
 * Own its files and directories correctly.
 * No rpmlint error/warning on the srpm.

rmplint however gave a long list of (similar) warnings [1] on the binary; I'm
sure you'll fix that later.

APPROVED.

[1]. rpmlint warnings;
[deji@agape reviews]$ rpmlint gnome-netstatus-2.12.1-1.fc8.x86_64.rpm
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-rx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-rx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/it/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png
W: gnome-netstatus non-conffile-in-etc /etc/gconf/schemas/netstatus.schemas
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_CN/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-disconn.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-disconn.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-tx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-tx.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ja/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/de/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_TW/figures/gnome-netstatus-error.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-error.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/ko/figures/gnome-netstatus-idle.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-idle.png
W: gnome-netstatus symlink-should-be-relative
/usr/share/gnome/help/gnome-netstatus/zh_HK/figures/gnome-netstatus-txrx.png
/usr/share/icons/hicolor/48x48/apps/gnome-netstatus-txrx.png

[Bug 225826] Merge Review: gnome-netstatus

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-netstatus

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225826

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |CLOSED
         Resolution|                            |CURRENTRELEASE

------- Additional Comments From [e-mail]  2007-08-10 21:39 EST -------
Review done.