[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Andrew Bauer changed: What|Removed |Added Status|ASSIGNED

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #50 from Nicolas Chauvet --- (In reply to Andrew Bauer from comment #49) > Pardon my newness to rpmfusion. After seeing the note stating bohdi is not > running yet, I did an "rfpkg update" from the command line,

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #49 from Andrew Bauer --- Pardon my newness to rpmfusion. After seeing the note stating bohdi is not running yet, I did an "rfpkg update" from the command line, similar to what I would do to push a

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 leigh scott changed: What|Removed |Added CC|

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #46 from Nicolas Chauvet --- (In reply to Antonio Trande from comment #45) @Antonio can you push the "fedora-review" flag to "+" in this review. This is the new way to approve package (same as Fedora IIRC). --

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #44 from Antonio Trande --- $ mock --no-clean -r fedora-rawhide-x86_64-rpmfusion_free --shell 'rpmlint zoneminder-debuginfo' INFO: mock.py version 1.4.2 starting (python version = 3.6.1)...

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Antonio Trande changed: What|Removed |Added Blocks|3 |4 --- Comment

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #41 from Antonio Trande --- (In reply to Andrew Bauer from comment #40) > > F26 SRPM: > http://zmrepo.zoneminder.com/f/testing/26/x86_64/zoneminder-1.30.4-3.fc26. > x86_64.rpm > SRPM?? -- You are

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #38 from Andrew Bauer --- ZoneMinder uses the CakePHP project as the framework for ZoneMinder's API. We've also added the Crud project as a plugin to the CakePHP framework. Crud is configured as a

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #36 from Andrew Bauer --- After upgrading my workstation F25 -> F26, I observed that rpmlint -v zoneminder takes an unusually long time to complete. If I wait 1-2 minutes, it eventually completes

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #35 from Andrew Bauer --- (In reply to Antonio Trande from comment #34) > - There something wrong after installing zoneminder: 'rpmlint' does not > finish its checks: > > $ rpmlint -v zoneminder >

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #34 from Antonio Trande --- - There something wrong after installing zoneminder: 'rpmlint' does not finish its checks: $ rpmlint -v zoneminder zoneminder.x86_64: I: checking zoneminder.x86_64: E:

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #33 from Andrew Bauer --- Understood. Thanks for the heads up. perl-Net-SFTP-Foreign is experiencing dependency issues in rawhide due to some per-math package not building on latest perl. -- You

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #32 from Antonio Trande --- I can't build on rawhide yet, because of some broken dependencies among packages. I will review this package as soon as possible. -- You are receiving this mail because: You are

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #31 from Andrew Bauer --- Thanks for the feedback. It turns out I needed to make some additional changes as well. I can mock build the following on f27. srpm:

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #30 from Michael Cronenworth --- (In reply to Andrew Bauer from comment #29) > Right, you guys are building against F27 now, which I have not tested yet. > This does build against f24, f25, el6, & el7, but I image

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #29 from Andrew Bauer --- Right, you guys are building against F27 now, which I have not tested yet. This does build against f24, f25, el6, & el7, but I image you will want this to build against

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #28 from Antonio Trande --- (In reply to Andrew Bauer from comment #27) > Thank you Antonio for reviewing this. > > The latest release, 1.30.4, srpm & specfile are here: >

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #27 from Andrew Bauer --- Thank you Antonio for reviewing this. The latest release, 1.30.4, srpm & specfile are here:

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Antonio Trande changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #25 from Andrew Bauer --- Greetings Xavier, Thank you for the prodding me. Following up here has been in the back of mind for a couple weeks now. With the help of Damian, all of ZoneMinder's

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #24 from Xavier Bachelot --- The review was on hold because of missing dependencies in Fedora. Andrew, are all the dependencies available now ? And is the specfile up to date ? I can take care of the review

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #23 from Nicolas Chauvet --- Is there any spec file update for this review ? Anyone can step in as a reviewer ? -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #22 from Andrew Bauer --- This is just a ping to let everyone know I am still making progress on this, even though it doesn't show in this particular bug report. I have been making slow, but steady

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #21 from Andrew Bauer --- I've made further progress on this. (In reply to Xavier Bachelot from comment #19) > A couple more comments on the spec... > > - Which supported distro/releases don't have

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #20 from Andrew Bauer --- I can absolutely knock those those changes out, but what follows are some questions to help get it right in one pass. (In reply to Xavier Bachelot from comment #19) > -

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #19 from Xavier Bachelot --- A couple more comments on the spec... - Which supported distro/releases don't have %make_build ? Thus, is this really needed : """ # In some cases older distros do not have this

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #18 from Xavier Bachelot --- No worries, keep on learning, you are on the right track :-) -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #17 from Andrew Bauer --- You bet. These are my first two Fedora package requests: perl-X10: https://bugzilla.redhat.com/show_bug.cgi?id=1409869 perl-Astro-SunTime:

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #16 from Xavier Bachelot --- Answering myself : perl-X10 : https://bugzilla.redhat.com/show_bug.cgi?id=1409869 perl-Astro-SunTime : https://bugzilla.redhat.com/show_bug.cgi?id=1409866 -- You are

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #15 from Xavier Bachelot --- Can you please provide links to the review requests for the missing perl modules ? perl-Astro-Suntime at least, others if any. Error: nothing provides perl(Astro::SunTime) needed by

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #14 from Andrew Bauer --- A new SRPM and specfile is ready for peer review: srpm: https://zmrepo.zoneminder.com/f/testing/25/SRPMS/zoneminder-1.30.1-2.fc25.src.rpm spec:

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #13 from Andrew Bauer --- Excellent feedback. Seems we are getting some momentum. I am part of the upstream development team and will push the changes concerning runtimedirectory. I'm falling

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #12 from Damian Wrobel --- (In reply to Andrew Bauer from comment #9) > (In reply to Damian Wrobel from comment #8) > > > > 2) Missing directory for pid file: > > Creating /var/run/zoneminer directory

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Tomasz Torcz changed: What|Removed |Added CC||zdzi...@irc.pl ---

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Xavier Bachelot changed: What|Removed |Added CC|

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #9 from Andrew Bauer --- (In reply to Damian Wrobel from comment #8) > Please find a few additional comments from an attempt to run version: > 1.30.1-1 on a new F25: > > 1)

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Damian Wrobel changed: What|Removed |Added CC|

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #7 from Andrew Bauer --- (In reply to Vasiliy Glazov from comment #4) > CentOS/RHEL 6 must have %make_build macro. You not need to define it > manually. I took a harder look at where %make_build

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #6 from Vasiliy Glazov --- And it is realy require vlc-devel? -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #5 from Vasiliy Glazov --- Next suggestions: 1. %global zmuid $(id -un) %global zmgid $(id -gn) Not used and can be removed. 2. If COPYING - is a license text it must be not in %doc, but %license COPYING -- You

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #4 from Vasiliy Glazov --- (In reply to Andrew Bauer from comment #2) > I noticed that my centos 6 environment did not have the %make_build macro so > I defined it at the top of the specfile should it not exist at

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Dominik 'Rathann' Mierzejewski changed: What|Removed |Added CC|

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 --- Comment #2 from Andrew Bauer --- Thank you for the feedback. I have made all five suggested changes. I noticed that my centos 6 environment did not have the %make_build macro so I defined it at the top of

[Bug 4393] Review Request: zoneminder - A camera monitoring and analysis tool

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4393 Vasiliy Glazov changed: What|Removed |Added CC||vasc...@gmail.com