Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: urlwatch - Tool for monitoring webpages for updates
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Summary: Review Request: urlwatch - Tool for monitoring webpages for updates Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: cassmodiah@fedoraproject.org QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://cassmodiah.fedorapeople.org/urlwatch-1.7/urlwatch.spec
SRPM URL: http://cassmodiah.fedorapeople.org/urlwatch-1.7/urlwatch-1.7-1.fc10.src.rpm
Description: This script is intended to help you watch URLs and get notified (via email or in your terminal) of any changes. The change notification will include the URL that has changed and a unified diff of what has changed.
The script supports the use of a filtering hook function to strip trivially-varying elements of a webpage. Basic features
* Simple configuration (text file, one URL per line) * Easily hackable (clean Python implementation) * Can run as a cronjob and mail changes to you * Always outputs only plaintext - no HTML mails :) * Supports removing noise (always-changing website parts) * Example hooks to filter content in Python
RPMLINT: silent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |jussi.lehtola@iki.fi AssignedTo|nobody@fedoraproject.org |jussi.lehtola@iki.fi Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #1 from Jussi Lehtola jussi.lehtola@iki.fi 2009-04-01 10:18:44 EDT --- rpmlint output is clean.
MUST: The spec file for the package is legible and macros are used consistently. OK MUST: The package must be named according to the Package Naming Guidelines. OK MUST: The spec file name must match the base package %{name}. OK MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. OK MUST: The License field in the package spec file must match the actual license. OK MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. OK MUST: The package MUST successfully compile and build into binary rpms. OK MUST: The spec file MUST handle locales properly. OK MUST: Optflags are used and time stamps preserved. Python bytecode, OK MUST: Packages containing shared library files must call ldconfig. OK MUST: A package must own all directories that it creates or require the package that owns the directory. OK MUST: Files only listed once in %files listings. OK MUST: Permissions on files must be set properly. OK MUST: Clean section exists. OK MUST: Large documentation files must go in a -doc subpackage. OK MUST: All relevant items are included in %doc. Items in %doc do not affect runtime of application. OK MUST: Header files must be in a -devel package. OK MUST: Static libraries must be in a -static package. OK MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. OK MUST: If a package contains library files with a suffix then library files ending in .so must go in a -devel package. OK MUST: In the vast majority of cases, devel packages must require the base package using a fully versioned dependency. OK MUST: Packages does not contain any .la libtool archives. OK MUST: Desktop files are installed properly. OK MUST: No file conflicts with other packages and no general names. OK MUST: Buildroot cleaned before install. OK SHOULD: %{?dist} tag is used in release. OK SHOULD: If the package does not include license text(s) as separate files from upstream, the packager should query upstream to include it. OK SHOULD: The package builds in mock. OK
The package is APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Simon Wesp cassmodiah@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs+
--- Comment #2 from Simon Wesp cassmodiah@fedoraproject.org 2009-04-01 15:38:57 EDT --- Thank you for your very fast review! :-)
New Package CVS Request ======================= Package Name: urlwatch Short Description: Tool for monitoring webpages for updates Owners: casssmodiah Branches: F-10 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Simon Wesp cassmodiah@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #3 from Simon Wesp cassmodiah@fedoraproject.org 2009-04-01 15:43:59 EDT --- New Package CVS Request ======================= Package Name: urlwatch Short Description: Tool for monitoring webpages for updates Owners: cassmodiah Branches: F-10 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fabian@bernewireless.net
--- Comment #4 from Fabian Affolter fabian@bernewireless.net 2009-04-01 18:10:02 EDT --- This package is already in Fedora.
https://admin.fedoraproject.org/pkgdb/packages/name/urlwatch
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |DUPLICATE Flag|fedora-review+, fedora-cvs? |
--- Comment #5 from Jason Tibbitts tibbs@math.uh.edu 2009-04-01 18:18:30 EDT ---
*** This bug has been marked as a duplicate of 480646 ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
--- Comment #6 from Jussi Lehtola jussi.lehtola@iki.fi 2009-04-02 02:10:35 EDT --- (In reply to comment #4)
This package is already in Fedora.
Whoops, didn't remember to check BZ. Adding as first item on my checklist :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493299
--- Comment #7 from Simon Wesp cassmodiah@fedoraproject.org 2009-04-02 04:18:43 EDT --- My fault, the auto-completion of my browser said i searched for urlwtch. :-( sorry jussi...
package-review@lists.fedoraproject.org