[Bug 225779] New: Merge Review: GConf2

[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=225779

Summary: Merge Review: GConf2
           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: GConf2

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

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

[e-mail] changed:

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

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2007-02-03 17:14 EST -------
Good:
* Source URL is canonical
* Upstream source tarball verified
* Group Tag is from the official list
* Buildroot has all required elements
* All paths begin with macros
* All directories are owned by this or other packages
* All necessary BuildRequires listed.

Minor:
* Could use the -disable-static config flag, and not even bother building the
static libs.
* Duplicate BuildRequires: pkgconfig (by libxml2-devel), glib2-devel (by
ORBit2-devel), autoconf (by libtool), automake (by libtool), perl-XML-Parser (by
intltool)
* rpmlint errors (Non-blockers):
E: GConf2-devel tag-not-utf8 %changelog
W: GConf2-devel no-documentation

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

------- Additional Comments From [e-mail]  2007-02-03 19:58 EST -------
I've added --disable-static now, and fixed one gtk-doc related directory
ownership problem that you have overlooked.

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |MODIFIED

[Bug 225779] Merge Review: GConf2

[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: GConf2

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium
            Product|Fedora Extras               |Fedora

[e-mail] changed:

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

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