Bug 183322

Several things showed up in another package I submitted, so I've made changes to
correct the same issues in this package. New versions are here:

Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec
SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/4/srpms/conexus-0.1.16-1.src.rpm

A few minor tweaks, including cleanup of the Source tag and the %setup section.

Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/4/srpms/conexus-0.1.16-2.src.rpm

* Missing "BuildRequires: libpcap"

* The two %doc files AUTHORS and COPYING are included in both the -devel
package and the main package, which should be avoided.

* The -devel package is missing "Requires: libpcap" for /usr/include/pcap.h

* The -devel should "Requires: pkgconfig" since without using
pkg-config's --cflags and --libs commands it would be difficult
to find out all the include search paths and libraries

* DNS-based random mirror download location for SF.net actually is of
the generic form (and command-line compatible!):
http://download.sourceforge.net/conexus/%{name}-%{version}.tar.bz2
Also available: http://dl.sf.net/....

With those changes applied (you're upstream): APPROVED

What's the status of the examples?

$ cd widget_tests
$ ./main
terminate called after throwing an instance of 'conexus::error::address::no_name'
  what():  conexus:address:errno[-2]: The  name  does  not  resolve for the
supplied parameters.
Aborted

Regarding the widget_test example, there have been some major changes to the
structure of the library itself. All the examples are good to go now; IPv6
support is now present for UDP and multicasting along with manipulation of IPv6
addresses including convenience wrappers for getaddrinfo() and gethostinfo()
that work with std::string's.

Despite the internal changes to the library itself, the spec hasn't changed much
except for cleanup of the mentioned items in the review.

The gtkmm widgets have also been split into a separate library (conexusgtkmm)
and now also support IPv6 names. Should I submit conexusgtkmm as a separate
package for review? The spec for conexusgtkmm is directly based on the
previously reviewed spec, while the new conexus spec is the same with the gtkmm
dependency removed and replaced by glibmm.

> Should I submit conexusgtkmm as a separate package for review?

Better is. So this late package split gets another check.

I'm getting ready to release a new version of this library in the next couple of
weeks.

Since it needs another review, it would probably be best to wait until the new
release comes out.

Also, I'll clean up the spec a bit to include the hints you provided on the bit
library.

I've submitted another bug report for the conexusmm library which now has the
Gtkmm widgets. The spec file for the conexusmm library is identical to this one
with the exception of the dependencies and the inclusion of a conexusmm-demo
application that is based on gtkdemo.

All the issues of the previous review have been taken care of in the revised
spec along with several other issues that came up in the bit library review.

Here are the new files:
Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexus-0.2.0-1.src.rpm

New release.

Here are the new files:
Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexus-0.2.1-1.src.rpm

make DESTDIR=%{buildroot} install

Again, be consistent with your macros

%{__rm} -rf %{buildroot}%{_bindir}

Why?

%{__cp} -arv docs/reference/html .

As with papyrus, can you include a comment on this

%{_libdir}/pkgconfig/*

No. For pkgconfig please use the name of the spec. I know you're using generic
templates for these, but where possible, include the names of libraries,
binaries and spec files.

%configure --enable-static=no

(niggle) %configure --disable-static not permitted (usually is)

Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexus-0.2.1-2.src.rpm

* Sun Jul 30 2006 Rick L Vinyard Jr <rvinyard.edu> - 0.2.1-2
- Added cppunit-devel to BuildRequires
- Changed make to %%{__make}
- Changed %%{name} to autoconf subst that puts specific name in devel requires
- Added comment regarding why cp occurs for docs
- Added package name to globs in so libs, .pc and demos

Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexus-0.2.2-1.src.rpm

- Added pkgconfig to Requires
- New release

Created attachment 133701 [details]
patch against conexus-0.2.2

Build failure with FC Development. Trivial patch attached.

[...]

In Fedora Core Development, libpcap has been split into libpcap-devel
and libpcap finally. So, it must be "BuildRequires: libpcap-devel"
and "Requires: libpcap-devel" in the -devel package now.

[...]

Errors during configure:

> checking for PROJECT... yes
> ./configure: line 24662: m4: command not found
> ./configure: line 24662: m4: command not found

Missing "BuildRequires m4"?

With those easy-to-fix issues fixed: APPROVED