[Bug 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 --- Comment #2 from Johan Vromans jvrom...@squirrel.nl 2009-01-30 09:38:13 EDT --- Thanks for the kind and constructive comments. I've added a patch for the Makefile to eliminate the double install of the man pages, and applied your other suggestions as well. I'll submit a new review request. As for the other questions: I have some more packages I'd like to submit but since this is the first time I first try one package to get used to the procedures. -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 --- Comment #3 from manuel wolfshant wo...@nobugconsulting.ro 2009-01-30 09:56:56 EDT --- As you please, I am waiting. 2-3-4 correct packages (depending on complexity and your abilities to fix -- if needed -- errors) will[/might] bring sponsorship :) Please do no forget to increment the release tag each time you submit a modified spec. -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added CC||panem...@gmail.com --- Comment #4 from Parag AN(पराग) panem...@gmail.com 2009-01-30 10:23:49 EDT --- Make sure which bug to mark duplicate I see this package submitted by submitter twice. see bug483250 -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 Johan Vromans jvrom...@squirrel.nl changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE --- Comment #5 from Johan Vromans jvrom...@squirrel.nl 2009-01-30 10:32:49 EDT --- *** This bug has been marked as a duplicate of 483250 *** -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 Johan Vromans jvrom...@squirrel.nl changed: What|Removed |Added Blocks||177841 -- 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)
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=483108 --- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-01-29 18:16:00 EDT --- I suggest to remove the first paragraph of the description. The description tag cannot refer to anything but the chordii package, hence This is a description of a utility called chordii. is redundant. The starting phrase of the second paragraph should be modified accordingly, for instance The purpose of chordii is to provide guitar players with [...] Please examine if the build process can take advantage of compiling in parallel on several processors ( http://fedoraproject.org/wiki/Packaging/Guidelines#Parallel_make ) If the program doesn't build with it, please add a comment otherwise please use it. It would also be a good idea to also add INSTALL=install -p to the make line, this should preserve the timestamps of the man pages. Please also take a look at the Makefile, according to http://koji.fedoraproject.org/koji/getfile?taskID=1092213name=build.log the manpages are installed twice. It's not critical now but, if I am not mistaken, patching the Makefile in order to remove the redundant lines (if this is the case) would be a good idea. Did you submit any other packages ? Or maybe you have performed some pre-reviews ? -- 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