Bug 1547631 - Review Request: nodejs-git-raw-commits - Get raw git commits out of your repository using git-log
Summary: Review Request: nodejs-git-raw-commits - Get raw git commits out of your repo...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Robert-André Mauchin 🐧
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: nodejs-reviews
TreeView+ depends on / blocked
 
Reported: 2018-02-21 16:18 UTC by Jared Smith
Modified: 2018-02-26 10:22 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-02-26 10:22:56 UTC
Type: ---
Embargoed:
eclipseo: fedora-review+


Attachments (Terms of Use)

Description Jared Smith 2018-02-21 16:18:58 UTC
Spec URL: https://jsmith.fedorapeople.org/Packaging/nodejs-git-raw-commits/nodejs-git-raw-commits.spec
SRPM URL: https://jsmith.fedorapeople.org/Packaging/nodejs-git-raw-commits/nodejs-git-raw-commits-1.2.0-1.fc27.src.rpm
Description: Get raw git commits out of your repository using git-log(1)
Fedora Account System Username: jsmith

Comment 1 Robert-André Mauchin 🐧 2018-02-21 22:12:52 UTC
 - SPEC file and SRPM SPEC file are different, one refers to a Source2:

Source2:	LICENSE-MIT-stevemao.txt

   The other uses a patch to include the license:

Patch0:		git-raw-commits_license.patch

Ha now I see, you included the wrong SRPM, the correct one is:

https://jsmith.fedorapeople.org/Packaging/nodejs-git-raw-commits/nodejs-git-raw-commits-1.3.3-1.fc28.src.rpm

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors

Package approved.

Comment 2 Jared Smith 2018-02-25 16:13:18 UTC
Repo requested at https://pagure.io/releng/fedora-scm-requests/issue/4724.

Comment 3 Gwyn Ciesla 2018-02-25 20:34:32 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/nodejs-git-raw-commits

Comment 4 Jared Smith 2018-02-26 10:22:56 UTC
Built in Rawhide


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