Bug 1302871 - Review Request: golang-github-vbatts-tar-split - tar archive assembly/disassembly
Summary: Review Request: golang-github-vbatts-tar-split - tar archive assembly/disasse...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jan Chaloupka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-28 20:39 UTC by Antonio Murdaca
Modified: 2020-05-30 14:01 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-05-30 14:01:55 UTC
Type: ---
Embargoed:
jchaloup: fedora-review+


Attachments (Terms of Use)

Description Antonio Murdaca 2016-01-28 20:39:45 UTC
Spec URL: https://github.com/runcom/fedora-pkgs/blob/master/golang-github-vbatts-tar-split/fedora/golang-github-vbatts-tar-split/golang-github-vbatts-tar-split.spec
SRPM URL: https://github.com/runcom/fedora-pkgs/raw/master/golang-github-vbatts-tar-split/golang-github-vbatts-tar-split-0.9.11-1.fc23.src.rpm

Koji builds:

- f23:     http://koji.fedoraproject.org/koji/taskinfo?taskID=12715095
- rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=12715092

Description: Pristinely disassembling a tar archive, and stashing needed raw bytes and offsets to reassemble a validating original archive.
Fedora Account System Username: runcom
Additional information: this is just providing the devel package for v0.9.11

Comment 1 Antonio Murdaca 2016-01-28 22:48:44 UTC
This is missing cmd/ command directory which needs to be packaged as well - so tar-split provides a cmd/ and devel.

Comment 4 Jan Chaloupka 2016-02-01 07:16:06 UTC
Can you create PR with your spec file and srpm here [1]? Github allows us to comment each line of a spec file. So we don't have to comment affected lines here and get confused what does it actually means.

Steps:
1) create directory with expected package name
2) create spec file (with patches and other important files) and srpm in that directory
3) create PR.
4) post link to the PR here.

In future, I would like to recommend using Github for golang related comments.

[1] https://github.com/gofed/reviews

Comment 5 Antonio Murdaca 2016-02-01 16:28:12 UTC
https://github.com/gofed/reviews/pull/1

Comment 6 Jan Chaloupka 2016-02-01 18:31:31 UTC
$ rpmlint /home/jchaloup/rpmbuild/SRPMS/golang-github-vbatts-tar-split-0.9.12-1.fc20.src.rpm
golang-github-vbatts-tar-split.src: W: spelling-error Summary(en_US) disassembly -> disassemble, dis assembly, dis-assembly
golang-github-vbatts-tar-split.src: W: summary-not-capitalized C tar archive assembly/disassembly
golang-github-vbatts-tar-split.src: W: spelling-error %description -l en_US Pristinely -> Pristine
golang-github-vbatts-tar-split.src: E: description-line-too-long C Pristinely disassembling a tar archive, and stashing needed raw bytes and offsets to reassemble a validating original archive.
golang-github-vbatts-tar-split.src: W: invalid-license BSD 3-clause
1 packages and 0 specfiles checked; 1 errors, 4 warnings.

In summary
- license should be "BSD"
- capitalize summary
- break description into two lines

Comment 7 Jan Chaloupka 2016-02-01 19:03:01 UTC
Summary:
- golang side of the spec file fine
- license ok
- rpmlint ok

Comment 9 Gwyn Ciesla 2016-02-03 13:25:55 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/tar-split

Comment 10 Gwyn Ciesla 2016-02-06 00:08:37 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/golang-github-vbatts-tar-split

Comment 11 Mattia Verga 2020-05-30 14:01:55 UTC
This package was approved and imported in repositories, but this review ticket was never closed.
I'm closing it now.


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