Bug 982705
Summary: | Review Request: meme - Use the command line to generate memes on memegenerator.co | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Allan Roque <allanrafaelroque> |
Component: | Package Review | Assignee: | Douglas Schilling Landgraf <dougsland> |
Status: | CLOSED INSUFFICIENT_DATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | allanrafaelroque, dougsland, marcelo.barbosa, package-review |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-05-14 17:11:10 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Allan Roque
2013-07-09 15:52:43 UTC
Hello Allan, Few comments: - * Fri Jul 9 2013 Allan Roque <allanrafaelroque> - 1.0.5-1 ==> Jul 9 is not Friday. ;-) - meme.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/meme/meme.py 0644L /usr/bin/env ===> This one can be ignored. However, I would suggest you to talk with upstream people to remove the shebang since users aren't really going to execute the file directly, so why have a shebang line? - %check # No present in source ==> Your spec say that there are no tests but looks like the source code provides (see below). Can you please double check? $ ls upstream-unpacked/Source0/memepy-a1e32491a9996ef881c401b9dc4e753d2f8af24b/meme/tests/ action_result.json info.json __init__.py popular.json search.json test_meme.py - Please make sure you increase the release for next review. To get sponsored you need to do at least 2 informal package review [1]. Basically, you can pick up a random bugzilla that is requesting a package review and push your comments there (don't need to assign it) and share the links here. To help you in this task, I can suggest: - FedoraReview - Fedora Hosted https://fedorahosted.org/FedoraReview/ - Package Review Guidelines http://fedoraproject.org/wiki/Packaging:ReviewGuidelines - Package Review Guidelines for Java http://fedoraproject.org/wiki/Packaging:Java - Package Review Guidelines for Python http://fedoraproject.org/wiki/Packaging:Python [1] https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group Please let me know if you have any question. Thanks! Hello Douglas, New SRPM & SPEC (Adjusted) Spec URL: http://aroque.fedorapeople.org/meme/meme.spec SRPM URL: http://aroque.fedorapeople.org/meme/meme-1.0.5-2.fc19.src.rpm koji_results: http://koji.fedoraproject.org/koji/taskinfo?taskID=5607361 - About %changelog Adjust date - About %install Adjust shebang - link - (http://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_files) - About %check The test directory is related to test software and not installing the rpm. change release for this review (1.0.5-2) thanks for the suggestions. Hi Allan, - Please address my comment#1: > - %check > # No present in source > ==> Your spec say that there are no tests but looks like the source code > provides (see below). Can you please double check? - Also update the spec with the last version of package (2.0.0 as I can see) No answer, closing for now as insufficient data, fell free to reopen it when you have spec/src.rpm ready. The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days |