Bug 1380420 - Review Request: gomtree - Go CLI tool for mtree support
Summary: Review Request: gomtree - Go CLI tool for mtree support
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: Jan Chaloupka
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-29 14:37 UTC by Lokesh Mandvekar
Modified: 2016-12-16 03:52 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-12-15 23:29:37 UTC
Type: ---
Embargoed:
jchaloup: fedora-review+


Attachments (Terms of Use)

Description Lokesh Mandvekar 2016-09-29 14:37:05 UTC
Spec URL: https://lsm5.fedorapeople.org/gomtree/gomtree.spec
SRPM URL: https://lsm5.fedorapeople.org/gomtree/SRPMS/gomtree-0-0.1.git8c6b32c.fc26.src.rpm

Description:
Go CLI tool for mtree support

Fedora Account System Username: lsm5

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=15860784

Comment 1 Jan Chaloupka 2016-09-30 08:52:07 UTC
Change 

%gobuild -o bin/%{name} %{import_path}

to

%gobuild -o bin/%{name} %{import_path}/cmd/gomtree

Otherwise rpmlint complains about shebang. Otherwise LGTM:

After the change:

$ rpmlint gomtree-0-0.1.git8c6b32c.fc20.src.rpm gomtree-0-0.1.git8c6b32c.fc20.x86_64.rpm gomtree-devel-0-0.1.git8c6b32c.fc20.noarch.rpm gomtree-unit-test-devel-0-0.1.git8c6b32c.fc20.x86_64.rpm
gomtree.src: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree.src: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree.x86_64: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree.x86_64: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree.x86_64: W: unstripped-binary-or-object /usr/bin/gomtree
gomtree.x86_64: W: no-manual-page-for-binary gomtree
gomtree-devel.noarch: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree-devel.noarch: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree-devel.noarch: W: spelling-error %description -l en_US github -> git hub, git-hub, thuggish
gomtree-unit-test-devel.x86_64: W: spelling-error %description -l en_US github -> git hub, git-hub, thuggish
gomtree-unit-test-devel.x86_64: W: spelling-error %description -l en_US mtree -> tree, m tree
4 packages and 0 specfiles checked; 0 errors, 11 warnings.

$ gofed lint
W: Incorrect package name gomtree, should be golang-github-vbatts-go-mtree
1 golang specfile checked; 0 errors, 1 warnings.

You can ignore the warning, it is recommendation only.

Comment 2 Lokesh Mandvekar 2016-09-30 17:13:54 UTC
(In reply to Jan Chaloupka from comment #1)
> Change 
> 
> %gobuild -o bin/%{name} %{import_path}
> 
> to
> 
> %gobuild -o bin/%{name} %{import_path}/cmd/gomtree
> 
> Otherwise rpmlint complains about shebang. Otherwise LGTM:

No big deal to me either way, but just curious where you're seeing this in rpmlint. I don't seem to see this anywhere. I'm on rawhide if it matters. See: 

$ rpmlint SRPMS/gomtree-0-0.1.git8c6b32c.fc26.src.rpm RPMS/* gomtree.spec
gomtree.src: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree.src: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree-devel.noarch: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree-devel.noarch: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree-devel.noarch: W: spelling-error %description -l en_US github -> git hub, git-hub, GitHub
gomtree.x86_64: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree.x86_64: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree.x86_64: W: binaryinfo-readelf-failed /usr/bin/gomtree 
gomtree.x86_64: W: no-manual-page-for-binary gomtree
gomtree-unit-test-devel.x86_64: W: spelling-error %description -l en_US github -> git hub, git-hub, GitHub
gomtree-unit-test-devel.x86_64: W: spelling-error %description -l en_US mtree -> tree, m tree
4 packages and 1 specfiles checked; 0 errors, 11 warnings.

Comment 3 Jan Chaloupka 2016-10-03 10:21:28 UTC
$ rpmlint gomtree-0-0.1.git8c6b32c.fc20.x86_64.rpm
gomtree.x86_64: W: spelling-error Summary(en_US) mtree -> tree, m tree
gomtree.x86_64: W: spelling-error %description -l en_US mtree -> tree, m tree
gomtree.x86_64: W: binaryinfo-readelf-failed /usr/bin/gomtree 
gomtree.x86_64: E: script-without-shebang /usr/bin/gomtree
gomtree.x86_64: W: no-manual-page-for-binary gomtree
1 packages and 0 specfiles checked; 1 errors, 4 warnings.

No sure if %gobuild -o bin/%{name} %{import_path} does anything good. I have always assumed you need to specify entire path to the main package file, i.e. %{import_path}/cmd/gomtree.

Checking the compiled binary:

$ ./usr/bin/gomtree
./usr/bin/gomtree: line 1: syntax error near unexpected token `newline'
./usr/bin/gomtree: line 1: `!<arch>'

$ file ./usr/bin/gomtree
./usr/bin/gomtree: current ar archive

Comment 4 Lokesh Mandvekar 2016-10-03 15:07:04 UTC
yup I'm seeing this too, weird. Anyway, changed build step like you mentioned. 

Spec URL: https://lsm5.fedorapeople.org/gomtree/gomtree.spec
SRPM URL: https://lsm5.fedorapeople.org/gomtree/SRPMS/gomtree-0-0.2.git8c6b32c.fc26.src.rpm

PTAL

Comment 5 Jan Chaloupka 2016-10-03 15:22:17 UTC
Approved

Comment 6 Gwyn Ciesla 2016-10-03 15:36:27 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/gomtree

Comment 7 Fedora Update System 2016-10-05 11:21:52 UTC
gomtree-0-0.2.git8c6b32c.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-3f4edf70de

Comment 8 Fedora Update System 2016-10-05 20:30:07 UTC
gomtree-0-0.2.git8c6b32c.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-8cdf84f347

Comment 9 Fedora Update System 2016-12-15 23:29:37 UTC
gomtree-0-0.2.git8c6b32c.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 10 Fedora Update System 2016-12-16 00:25:00 UTC
gomtree-0-0.2.git8c6b32c.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2016-12-16 03:52:54 UTC
gomtree-0-0.2.git8c6b32c.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.


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