[Bug 435125] New: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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/show_bug.cgi?id=435125

Summary: Review Request: xerces-c27 - Compatibility package for
                    Xerces C++ 2.7.0
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://washington.kelkoo.net/fedora/SPECS/xerces-c27.spec
SRPM URL: http://washington.kelkoo.net/fedora/SRPMS/xerces-c27-2.7.0-4.fc8.src.rpm
Description: Compatibility package for Xerces C++ 2.7.0

This is needed for perl-XML-Xerces-2.7.0 which cannot be rebuild against xerces-c-2.8.0

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |434487
              nThis|                            |

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2008-03-07 16:06 EST -------
Review
=======

Good:

- rpmlint checks return:
xerces-c27-devel.x86_64: W: no-documentation
xerces-c27-devel.x86_64: W: dangling-relative-symlink
/usr/lib64/xerces-c-2.7.0/libxerces-c.so ../libxerces-c.so.27
xerces-c27-devel.x86_64: W: dangling-relative-symlink
/usr/lib64/xerces-c-2.7.0/libxerces-depdom.so ../libxerces-depdom.so.27
xerces-c27-devel.x86_64: E: only-non-binary-in-usr-lib

All are safe to ignore.

- package meets naming guidelines
- package meets packaging guidelines
- license (ASL 2.0) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- doc subpackage OK
- nothing in %doc affects runtime
- no need for .desktop file
- devel subpackage OK, depends on base package

Approved.

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2008-03-07 16:15 EST -------
Wow, you're fast. Thanks Spot.

New Package CVS Request
=======================
Package Name: xercesc-27
Short Description: Compatibility package for Xerces C++ 2.7.0
Owners: xavierb
Branches: devel
InitialCC: 
Cvsextras Commits: yes

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2008-03-07 16:28 EST -------
cvs done.

please get this built in rawhide asap. :)

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2008-03-07 18:45 EST -------
Sorry, I made a typo in the above cvs request. It should read
Package Name: xerces-c27
instead of
Package Name: xercesc-27

Can someone please fix this ?

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2008-03-07 19:34 EST -------
Done.  Let us know if there's any more troubles.

[Bug 435125] Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

[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: Review Request: xerces-c27 - Compatibility package for Xerces C++ 2.7.0

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

------- Additional Comments From [e-mail]  2008-03-08 07:41 EST -------
All is fine now, thanks Toshio :-) For the record, I had to fix devel/Makefile
(s/xercesc-27/xerces-c27/) to be able to import the srpm.

Imported and built for devel.