Bug 457547
| Summary: | Review Request: mediascrapper - A script to scrap media files from different sites | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | kushaldas@gmail.com <mail> |
| Component: | Package Review | Assignee: | Rahul Sundaram <sundaram> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | christoph.wickert, fedora-package-review, notting, pahan, panemade, smohan, sundaram |
| Target Milestone: | --- | Flags: | sundaram:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2008-08-13 18:15:30 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
kushaldas@gmail.com
2008-08-01 13:29:40 UTC
will you please tell me how can I test package? any sample movie.txt you would like to provide here? (In reply to comment #1) > will you please tell me how can I test package? any sample movie.txt you would > like to provide here? Put the following line in a movie.txt in the current folder http://www.apunkabollywood.net/browser/category/view/3949/rock-on-(2008) Remove the sitearch macro definition and CFLAGS line. Those are not needed for a noarch package. Include a copy of the license under a file called COPYING. The license of the source should match the tag in the spec file. You might also want to move your project to fedorahosted.org and make the script more generic. (In reply to comment #3) > Remove the sitearch macro definition and CFLAGS line. Those are not needed for > a noarch package. Include a copy of the license under a file called COPYING. > The license of the source should match the tag in the spec file. > > You might also want to move your project to fedorahosted.org and make the > script more generic. Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec SRPM URL: http://kushal.fedorapeople.org/packages/mediascrapper-0.1-2.fc9.src.rpm I will move the project under fedorahosted. The upcoming release will have feature to download photos from flickr (In reply to comment #4) > (In reply to comment #3) > > Remove the sitearch macro definition and CFLAGS line. Those are not needed for > > a noarch package. Include a copy of the license under a file called COPYING. > > The license of the source should match the tag in the spec file. > > > > You might also want to move your project to fedorahosted.org and make the > > script more generic. > > Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec > SRPM URL: > http://kushal.fedorapeople.org/packages/mediascrapper-0.1-2.fc9.src.rpm > > I will move the project under fedorahosted. The upcoming release will have > feature to download photos from flickr Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec SRPM URL: http://kushal.fedorapeople.org/packages/mediascrapper-0.1-3.fc9.src.rpm Koji scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=763547 Remove the note on the specific website. Approved. New Package CVS Request ======================= Package Name: mediascrapper Short Description: A script to scrap media files from different sites Owners: kushal Branches: F-9 EL-5 InitialCC: kushal Cvsextras Commits: yes cvs done. mediascrapper-0.1-3.fc9 has been submitted as an update for Fedora 9 mediascrapper-0.1-3.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. (In reply to comment #3) > Include a copy of the license under a file called COPYING. > The license of the source should match the tag in the spec file. Although this package is already in the repo for a while, the License tag is wrong. Should be GLPv2+ instead of GPLv2. Kushal, please update the Source0 on your fedorapeople account to really match the one included in the package. TIA. |