Review Request: NCO and CDO

[Tom 'spot' Callaway]

On Fri, 2005-04-22 at 10:40 -0400, Ed Hill wrote:
> Hi folks (with a special plea to Spot!),
> 
> I'd like to humbly request reviews of NCO and CDO.  The former made it
> part-way through the "old" Fedora review process:
> 
>   https://bugzilla.fedora.us/show_bug.cgi?id=1893
> 
> and has been updated to fix (hopefully) all the outstanding problems
> from that earlier submission.

It does look like the outstanding problems from the b.f.u ticket have
been resolved.

A couple of items I found:

+BuildRequires:  udunits, udunits-devel
+BuildRequires:  curl-devel, libxml2-devel, opendap-devel, librx-devel

You're missing some BuildRequires. (Note that opendap-devel and
librx-devel are pending sponsorship and review.)

-export CXXFLAGS='-fpermissive'
+export CXXFLAGS="$RPM_OPT_FLAGS -fpermissive"

No reason I can see to drop the optimization flags entirely, it builds
great with them in rawhide.

You also need to apply one more package for udunits to work properly.
This patch is attached to the email.

~spot
-- 
Tom "spot" Callaway: Red Hat Sales Engineer || GPG Fingerprint: 93054260
Fedora Extras Steering Committee Member (RPM Standards and Practices)
Aurora Linux Project Leader: http://auroralinux.org
Lemurs, llamas, and sparcs, oh my!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nco-3.0.0-fixudunits.patch
Type: text/x-patch
Size: 811 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/fedora-extras-list/attachments/20050423/040e574e/attachment.bin>