[Bug 491617] Review Request: mingw32-libxml++ - MinGW Windows C++ wrapper for libxml2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


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


Kalev Lember <kalev@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |kalev@xxxxxxxxxxxx
               Flag|                            |fedora-review?

Bug 491617 depends on bug 492113, which changed state.

Bug 492113 Summary: Review Request: mingw32-glibmm24 - MinGW Windows C++ interface for GTK2 (a GUI library for X)
https://bugzilla.redhat.com/show_bug.cgi?id=492113

           What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         Resolution|                            |NEXTRELEASE
             Status|ASSIGNED                    |CLOSED



--- Comment #4 from Kalev Lember <kalev@xxxxxxxxxxxx>  2009-04-16 15:52:15 EDT ---
mingw32-glibmm24 is now in Fedora and we can go on with this review.

- In Rawhide the native Fedora libxml++ package has been updated to 2.26.0, and
according to MinGW packaging guidelines the versions should match native
package if possible.

- Please clean up the %install section. It only makes the spec file unreadable
if most of a section contains leftovers that aren't actually used.

- Removing *.a in %install section only matches the (needed) import library,
because static libraries aren't built (configure --disable-static). It seems
unnecessary to first delete the import lib and then manually copy it over from
.libs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]