Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1348932 - Review Request: bash-git-prompt - An informative and fancy bash prompt for Git users
Summary: Review Request: bash-git-prompt - An informative and fancy bash prompt for G...
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-NEEDSPONSOR
TreeView+ depends on / blocked
 
Reported: 2016-06-22 10:48 UTC by Koustubh Sinkar
Modified: 2021-06-04 00:46 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Koustubh Sinkar 2016-06-22 10:48:03 UTC
Spec URL: https://gitlab.com/snippets/21406
SRPM URL: http://copr-dist-git.fedorainfracloud.org/cgit/ksinkar/prelude/bash-git-prompt.git
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

Comment 1 Parag AN(पराग) 2016-06-22 11:32:30 UTC
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 :)

Comment 2 William Moreno 2016-07-01 18:17:48 UTC
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.

Comment 3 Koustubh Sinkar 2016-07-04 14:27:53 UTC
Spec URL: https://raw.githubusercontent.com/ksinkar/bash-git-prompt/master/bash-git-prompt.spec

SRPM URL: https://copr-be.cloud.fedoraproject.org/results/ksinkar/prelude/fedora-23-x86_64/00365282-bash-git-prompt/bash-git-prompt-2.5.1-1.fc23.src.rpm

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.

Comment 4 William Moreno 2016-07-04 14:47:11 UTC
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 - 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

Comment 5 Koustubh Sinkar 2016-07-30 10:06:12 UTC
Does this spec file (https://github.com/ksinkar/bash-git-prompt/blob/rpm_spec/bash-git-prompt.spec) 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.

Comment 6 William Moreno 2017-12-16 04:24:30 UTC
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?

Comment 7 Koustubh Sinkar 2017-12-20 06:05:20 UTC
Yes, I am interested in going ahead with this package. And I am also interested in adding some other packages as well.

Comment 8 William Moreno 2018-03-16 16:54:28 UTC
Can you please post the last version of the spec in the form of:

Spec URL: 
SRPM URL:

Comment 9 Koustubh Sinkar 2018-04-01 09:59:40 UTC
(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-prompt.spec

SRPM URL: https://copr-be.cloud.fedoraproject.org/results/ksinkar/prelude/srpm-builds/00735465/bash-git-prompt-2.7.1-1.fc27.src.rpm

Comment 10 Package Review 2021-04-25 00:45:23 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 11 Package Review 2021-06-04 00:46:00 UTC
This is an automatic action taken by review-stats script.

The ticket reviewer failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we reset the status and the assignee of this ticket.


Note You need to log in before you can comment on or make changes to this bug.