[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 Jochen Schmitt changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CANTFIX Flag|fedora-review? |fedora-review- --- Comment #18 from Jochen Schmitt 2009-09-16 13:07:49 EDT --- Unfortunately, I have to recorgnise, that you didn't made your homework. I have the feeling, that you don't have the base skills to become a fedora packager. Because the package you want to provides is on the wishlist, I will close this review request and create an own one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #17 from Balaji G 2009-08-29 11:42:27 EDT --- Hi Jochen First of all sorry for the real delay. I was not in station unexpectedly and hence couldnt do it. I have made the changes and uploaded the SPEC file and the SRC RPM at the following location http://balajig8.fedorapeople.org/packages/ascii/ Thanks for your inputs and time. Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #16 from Balaji G 2009-07-14 12:29:10 EDT --- Hi Jochen Sorry couldn't work on this will start this again and repackage it with your comments :) Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #15 from Jochen Schmitt 2009-07-12 15:25:12 EDT --- Ping Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 Jason Tibbitts changed: What|Removed |Added CC||s...@sandro-mathys.ch --- Comment #14 from Jason Tibbitts 2009-07-11 13:13:39 EDT --- *** Bug 510856 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #13 from Jochen Schmitt 2009-05-24 15:27:11 EDT --- Please wrtie $RPM_BUILD_ROOT or $(RPM_BUILD_ROOT) instead of $"RPM_BUILD_ROOT" Please create a proper Buildroot defintion https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag The build step in your package doesn't use the compiler flags defined in $RPM_OPT_FLAGS. the Debuginfo package doesn't contains the source files of your package. This may be happen because the build step doesn't use the compiler flags defined in $RPM_OPT_FLAGS Good: + Rpmlint is silent on source rpm + Rpmlint is silent on binary rpm + rpmlint is silent on debuginfo rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #12 from Balaji G 2009-05-20 12:42:12 EDT --- Hi Jochen Thank you so much for your extensive review.I have addressed the review comments and have placed the package and the spec file at the following location for your review. http://balajig8.fedorapeople.org/packages/ascii/ Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #11 from Rex Dieter 2009-05-20 12:15:21 EDT --- imo, preserving historical/old changelog is ok, preferred actually, to provide history and attribution. (Provided it's not huge and doesn't impact pkg size anyway). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #10 from Jochen Schmitt 2009-05-19 13:26:14 EDT --- Good: + Basename of the SPEC files matches with packagename + Packagename fullfill naming guildelines + URL tag shows on proper project homepage + Package contains most recent stable release of the application + Could download upstream tar ball via spectool -g + Package sources matches with upstream (md5sum: 8fb7540bf2a7a8e1fa0086708ed9b881) + Consistently rpm macro usage in this package + Package has no subpackages + Package has proper license tag + License tag exclaim GPLv2 as a valid OSS license + Package contains verbatin copy of the license text + Package support SMP build + Local install and uninstall works fine + Call of the application works fine + Koji scratch build works fine + Package has small %doc stanza, so we need no extra doc subpackage + Files has proper file permission + All packaged files are owned by the package + %files stanza has no duplicated entries + No packaged file belong to another package Bad: - Copyright headers have only a refernce to the LICENSE/COPYING file. Please talk to upstream, that he should include a more clean copyright note on the source file - Please remove the Packager tag - You don't need to test the existance of the Buildroot before cleaning it. - build don't honour RPM_OPT_FLAGS - Rpmlint has warning on source rpm: pmlint ascii-3.8-2.fc10.src.rpm ascii.src: W: summary-not-capitalized interactive ASCII name and synonym chart ascii.src: W: name-repeated-in-summary ASCII ascii.src: W: non-standard-group Utilities/Text ascii.src: W: mixed-use-of-spaces-and-tabs (spaces: line 72, tab: line 82) 1 packages and 0 specfiles checked; 0 errors, 4 warnings. - Rpmlint complaints binary package: rpmlint ascii-3.8-2.fc10.x86_64.rpm ascii.x86_64: W: summary-not-capitalized interactive ASCII name and synonym chart ascii.x86_64: W: name-repeated-in-summary ASCII ascii.x86_64: W: non-standard-group Utilities/Text 1 packages and 0 specfiles checked; 0 errors, 3 warnings. - Debuginfo package doesn't contains sources - Please remove the old Changelog and beginning with a new one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 Jochen Schmitt changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|joc...@herr-schmitt.de Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #9 from Balaji G 2009-05-19 12:54:17 EDT --- Hi Jochen Sorry for the inconvenience. Yes i had uploaded the wrong file.I have updated the release tag in the Spec and uploaded the SRPM at the location for your review. http://balajig8.fedorapeople.org/packages/ascii/ Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #8 from Jochen Schmitt 2009-05-19 12:21:04 EDT --- It look like, that you didn't upload a correct release of your package. Please keep in mind do increase the release number each time if you make a change on your package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #7 from Balaji G 2009-05-19 12:07:54 EDT --- Hi Jochen I have addressed your review comments and have placed the SPEC file and the SRPM at the following location for your review. http://balajig8.fedorapeople.org/packages/ascii/ Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #6 from Balaji G 2009-05-19 12:06:53 EDT --- Hi Jochen I have addressed your review comments and have placed the SPEC file and the SRPM at the following location http://balajig8.fedorapeople.org/packages/ascii/ Please review the same Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 --- Comment #5 from Balaji G 2009-05-19 04:53:59 EDT --- Hi Jochen Thanks for your replies. I have uploaded the Source RPM and the Spec files at the location http://balajig8.fedorapeople.org/packages/ascii/ Thanks, Cheers, Balaji -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501353] Review Request: ascii - Interactive ASCII name and synonym chart
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=501353 Mamoru Tasaka changed: What|Removed |Added Summary|Review Request: - |Review Request: ascii - | | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review