https://bugzilla.redhat.com/show_bug.cgi?id=1348932
Bug ID: 1348932 Summary: Review Request: bash-git-prompt - An informative and fancy bash prompt for Git users Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nobody@fedoraproject.org Reporter: shopkeeper88-fedora@yahoo.co.in QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://gitlab.com/snippets/21406 SRPM URL: http://copr-dist-git.fedorainfracloud.org/cgit/ksinkar/prelude/bash-git-prom... Description: This prompt is a port of the "Informative git prompt for zsh" A bash prompt that displays information about the current git repository. In particular the branch name, difference with remote branch, number of files staged, changed, etc. This is my first package and I need a sponsor. Fedora Account System Username: ksinkar
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841 (FE-NEEDSPONSOR)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |panemade@gmail.com
--- Comment #1 from Parag AN(पराग) panemade@gmail.com --- We have this process http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group to get sponsored into the packager group. Can you either submit few more packages and/or some full detailed package reviews? This is needed to make sure package submitter understands the rpm packaging well and follows the fedora packaging guidelines.
Please go through the following links 1) http://fedoraproject.org/wiki/Package_Review_Process
2) https://fedoraproject.org/wiki/PackagingGuidelines
3) To find the packages already submitted for review, check http://fedoraproject.org/PackageReviewStatus/
4) http://fedoraproject.org/wiki/Packaging:ReviewGuidelines and http://fedoraproject.org/wiki/Package_Review_Process#Reviewer is useful while doing package reviews.
5) https://fedorahosted.org/FedoraReview/ this is fedora-review tool to help review packages in fedora. You need to use this and do un-official package reviews of packages submitted by other contributors. While doing so mention "This is un-official review of the package." at top of your review comment.
Good to review packages listed in http://fedoraproject.org/PackageReviewStatus/NEW.html
When you do full package review of some packages, provide that review comment link here so that I can look how you have reviewed those packages.
If you got any questions please ask :)
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
William Moreno williamjmorenor@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |williamjmorenor@gmail.com Assignee|nobody@fedoraproject.org |williamjmorenor@gmail.com Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
--- Comment #2 from William Moreno williamjmorenor@gmail.com --- Hello, I will take your review request, firt than all please fix the SPEC and SRPM links, it is fine to link to a copr build but please use the correct URL for the spec and srpm link.
Will be nice to see you do some informal reviews, while not mandatory is really usefull to see if a candidate is already motivated to become a package maintaier and also helps to see if you a proper undestand of Fedora Packaging Guidelines.
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|rawhide |23
--- Comment #3 from Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in --- Spec URL: https://raw.githubusercontent.com/ksinkar/bash-git-prompt/master/bash-git-pr...
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/ksinkar/prelude/fedora-23-x8...
The above are the updated URLs. I hope I do not have repeat the description information. I have read through the packaging guidelines and am looking at projects where I can do informal reviews. Will update this feed, of the reviews, once I do them.
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
William Moreno williamjmorenor@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|23 |rawhide
--- Comment #4 from William Moreno williamjmorenor@gmail.com --- Hi, just a few comments:
1. Al new packages must go to rawhide, I hace just fixed the tag.
2. There is no need of %%clean seption at less than you want to go to epel 6 or epel 5, also you must remove the rm -rf %{buildroot} command in the begginig of %install.
3. Add ad comment to the %build section to avoid a rmplint issue about ampy %%buil
4. The url tag shuld be https://github.com/magicmonty/bash-git-prompt without the .git
5. There is not problem with use a already existing spec but update the changelog:
Fri Aug 08 2014 Justin Zhang <schnell18@gmail.com - 1.0.1-1
I guess at less you made a versión bump.
6. Send a pull request with the updated spec.
7. Finally %post and %postun should go after %%install (or %%?check), can find a example here:
http://pkgs.fedoraproject.org/cgit/rpms/exaile.git/tree/exaile.spec
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
--- Comment #5 from Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in --- Does this spec file (https://github.com/ksinkar/bash-git-prompt/blob/rpm_spec/bash-git-prompt.spe...) look good? I have updated it to meet the patterns as given in the example file. I have also implemented all the suggestions as I understood them.
I am not deleting the %clean section, because I want this to be acceptable by epel repos as well.
Let me know if any changes are required in the spec file. I shall merge the spec file in upstream once it is finalized.
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
--- Comment #6 from William Moreno williamjmorenor@gmail.com --- Please remove this:
%clean rm -rf %{buildroot}
Sorry for the very late feedback, do you still want to go ahead to get this package in Fedora?
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
--- Comment #7 from Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in --- Yes, I am interested in going ahead with this package. And I am also interested in adding some other packages as well.
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
William Moreno williamjmorenor@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |shopkeeper88-fedora@yahoo.c | |o.in Flags| |needinfo?(shopkeeper88-fedo | |ra@yahoo.co.in)
--- Comment #8 from William Moreno williamjmorenor@gmail.com --- Can you please post the last version of the spec in the form of:
Spec URL: SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1348932
Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(shopkeeper88-fedo | |ra@yahoo.co.in) |
--- Comment #9 from Koustubh Sinkar shopkeeper88-fedora@yahoo.co.in ---
(In reply to William Moreno from comment #8)
Can you please post the last version of the spec in the form of:
SPEC URL: https://raw.githubusercontent.com/ksinkar/bash-git-prompt/rpm-spec/bash-git-...
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/ksinkar/prelude/srpm-builds/...
package-review@lists.fedoraproject.org