Spec URL: http://silxnet.org/~alex/fedora/picprog/picprog.spec SRPM URL: http://silxnet.org/~alex/fedora/picprog/picprog-1.9.0-1.fc10.src.rpm Description: Picprog is a simple program for burning programs to various Microchip PIC micro-controllers via several types of serial port programmers. This is my first package for Fedora, so I'm looking for a sponsor.
Hello Alex, I'm not a sponsor but nevertheless some quick comments from me. :) - according to the source headers, the license is GPLv2 only - add comments to the referenced patches about what they do - it shouldn't be necessary to give the BuildRequires and Requires dependencies - "make dep" is redundant
Made appropriate changes as per above. Spec URL: http://silxnet.org/~alex/fedora/picprog/picprog.spec SRPM URL: http://silxnet.org/~alex/fedora/picprog/picprog-1.9.0-2.fc10.src.rpm
I think my note about the patch comments was a bit unclear. I didn't mean to add the comments to the patch files but before Patch0 and Patch1 in the spec file.
Sorry about that. Comments added to spec file regarding patches as per above. Spec URL: http://silxnet.org/~alex/fedora/picprog/picprog.spec SRPM URL: http://silxnet.org/~alex/fedora/picprog/picprog-1.9.0-2.fc10.src.rpm
Well, * ExclusiveArch - This package does not build on ppc/ppc64. This is because this package contains some codes which really won't work on ppc/ppc64. http://koji.fedoraproject.org/koji/taskinfo?taskID=1681058 This spec file needs: "ExclusiveArch: %{ix86} x86_64". ! About Patch1 - I would rather install man file and binary file manually for this package, however I will leave it to you where you use patch this patch or not. * %attr(644,root,root) - I would suggest to change the permission explicitly at %install stage (if really needed) rather than using %attr for this issue like: --------------------------------------------------------- %install .... .... chmod 0644 $RPM_BUILD_ROOT%{_mandir}/man1/picprog.1* --------------------------------------------------------- Anyway this package is rather simple, so: ------------------------------------------------------------- NOTE: Before being sponsored: This package will be accepted with another few work. But before I accept this package, someone (I am a candidate) must sponsor you. Once you are sponsored, you have the right to review other submitters' review requests and approve the packages formally. For this reason, the person who want to be sponsored (like you) are required to "show that you have an understanding of the process and of the packaging guidelines" as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Usually there are two ways to show this. A. submit other review requests with enough quality. B. Do a "pre-review" of other person's review request (at the time you are not sponsored, you cannot do a formal review) When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Fedora package collection review requests which are waiting for someone to review can be checked on my wiki page: http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets (Check "No one is reviewing") Review guidelines are described mainly on: http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets ------------------------------------------------------------
> * ExclusiveArch > - This package does not build on ppc/ppc64. This is because > this package contains some codes which really won't work > on ppc/ppc64. > http://koji.fedoraproject.org/koji/taskinfo?taskID=1681058 > > This spec file needs: "ExclusiveArch: %{ix86} x86_64". Done. Looks like it *might* be possible to patch picprog to build on the ppc and ppc64, but I'll leave it for now. > ! About Patch1 > - I would rather install man file and binary file manually > for this package, however I will leave it to you where you > use patch this patch or not. Changed it to manual install using `install`. This removes the need for the chmod you mentioned. As for sponsorship... I have a few other packages I'd like to work on such as an assembler for Atmel AVR chips (as well as some packages for the RPM Fusion repos). I'll post the bug number here when I submit that package for review. Also, if I see any package review requests I'll have a look and see if I can help. Spec URL: http://silxnet.org/~alex/fedora/picprog/picprog.spec SRPM URL: http://silxnet.org/~alex/fedora/picprog/picprog-1.9.0-3.fc10.src.rpm
Submitted my second package - see https://bugzilla.redhat.com/show_bug.cgi?id=524346
Created attachment 361772 [details] picprog document html snapshot Well, for -3: * Timestamps - Add "-p" option when using "install" or "cp" commands to keep timestamps on installed files: https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps * Documents - Some png files are needed for installed document html file. Please also install them (see attached).
Changes as per above. Spec URL: http://silxnet.org/~alex/fedora/picprog/picprog.spec SRPM URL: http://silxnet.org/~alex/fedora/picprog/picprog-1.9.0-4.fc10.src.rpm
Okay. --------------------------------------------------- This package (picprog) is APPROVED by mtasaka --------------------------------------------------- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". After you request for sponsorship a mail will be sent to sponsor members automatically (which is invisible for you) which notifies that you need a sponsor. After that, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 10/11, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me.
I've applied for the 'packager' group on the FA2. My FAS name is 'musolinoa'.
Now I am sponsoring you. Please follow "Join" wiki again.
New Package CVS Request ======================= Package Name: picprog Short Description: Microchip PIC micro-controller serial programmer software Owners: musolinoa Branches: F-10 F-11 InitialCC:
CVS done.
Please submit push request for F-11 on bodhi.
Done.
Thanks.
Package Change Request ====================== Package Name: picprog New Branches: el6 Owners: musolinoa
Please chage fedora-cvs flag to ?.
Git done (by process-git-requests).