[Bug 220931] Review Request: ZoneMinder - Linux CCTV package

[bugzilla at redhat.com]

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: ZoneMinder - Linux CCTV package


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


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|normal                      |medium




------- Additional Comments From tibbs at math.uh.edu  2007-06-10 00:14 EST -------
Hmm, I'd really like to have an up-to-date zm package.  Are you still interested
in getting this into the distro?

Some quick comments:

You'll need a build dependency on at least perl(ExtUtils::MakeMaker); it fails
to build for me otherwise.

If possible, Source1: should be a full URL to the upstream cambozola source.

Is it not possible to direct the program to look directly in /var/lib/zm/*
instead of needing symlinks?  Otherwise, I do think they should be relative, and
it's trivial to accomplish:

ln -sf ../../../../var/lib/zm/$dir $RPM_BUILD_ROOT%{_datadir}/%{name}/www/$dir

in the %install section.

You can use %{perl_vendorlib} instead of %{_prefix}/lib/perl5/vendor_perl/*

I have to say that putting the cgi bits in /usr/libexec seems odd to me; I'll
have to ask around to see if other folks think it's OK.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.