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 1688075 - Review Request: golang-github-yudai-golcs - Go Longest Common Subsequence
Summary: Review Request: golang-github-yudai-golcs - Go Longest Common Subsequence
Keywords:
Status: CLOSED ERRATA
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: 1670656
TreeView+ depends on / blocked
 
Reported: 2019-03-13 04:42 UTC by Nathan Scott
Modified: 2019-04-22 05:09 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-15 05:18:23 UTC
Type: ---
Embargoed:
zebob.m: fedora-review+


Attachments (Terms of Use)

Description Nathan Scott 2019-03-13 04:42:28 UTC
Spec URL: https://nathans.fedorapeople.org/golcs/golang-github-yudai-golcs.spec
SRPM URL: https://nathans.fedorapeople.org/golcs/golang-github-yudai-golcs-0-1.20190313gitecda9a5.fc29.src.rpm
Description: Go Longest Common Subsequence
Fedora Account System Username: nathans

Comment 1 Nathan Scott 2019-03-13 04:49:31 UTC
https://koji.fedoraproject.org/koji/taskinfo?taskID=33445301

(most OK but i686 and armv7hl builds failing tests with ENOMEM - needs some further investigation)

Comment 2 Robert-André Mauchin 🐧 2019-03-13 21:30:23 UTC
 - License ok
 - Latest version packaged
 - Conforms to Packaging Guidelines

 - %global commitdate	20170316

should be snapshotdate i.e. the day you made the snapshot not the day of the commit. 
Also it is automated by the macros, it is set to the date you upload the archive to dist-git.

 - the release field for a pre-release dev snapshot is not okay, it should start at 0.1:

Release:        0.1%{?dist}

 - use a better description:

%global common_description %{expand:
A package to calculate LCS of slices.}

[…]

%description
%{common_description}


%package devel
Summary:       %{summary}
BuildArch:     noarch

%description devel
%{common_description}

(In reply to Nathan Scott from comment #1)
> https://koji.fedoraproject.org/koji/taskinfo?taskID=33445301
> 
> (most OK but i686 and armv7hl builds failing tests with ENOMEM - needs some
> further investigation)

Please fill a bug with upstream for failure on 32bits arches. Or patch the tests so it doesn't try to allocate 100,000×100,000 int.

 - Don't mix tabs and spaces:

golang-github-yudai-golcs.src:5: W: mixed-use-of-spaces-and-tabs (spaces: line 5, tab: line 2)

Comment 3 Robert-André Mauchin 🐧 2019-03-13 21:45:48 UTC
The changelog should also reflect the release tag:

* Wed Mar 13 2019 Nathan Scott <nathans> - 0-0.1.20190313gitecda9a5

You don't need this either:

%global shortcommit	%(c=%{commit}; echo ${c:0:7})

Comment 4 Nathan Scott 2019-03-14 05:44:40 UTC
(In reply to Robert-André Mauchin from comment #2)
> [...]
> should be snapshotdate i.e. the day you made the snapshot not the day of the
> commit. 
> Also it is automated by the macros, it is set to the date you upload the
> archive to dist-git.

Ah excellent - thanks for these and all the other clues.  I've fixed 'em
up, and will upload new spec/srpm files shortly.

> > (most OK but i686 and armv7hl builds failing tests with ENOMEM - needs some
> > further investigation)
> 
> Please fill a bug with upstream for failure on 32bits arches. Or patch the
> tests so it doesn't try to allocate 100,000×100,000 int.

I've opened https://github.com/yudai/golcs/issues/3

I also attempted some different settings in the test code, but have found
reducing the allocation sizes to also be problematic.  The array must be
of certain size that it takes >1sec to process for the test to function.

Reducing sizes results in some passes, but the test fails intermittently.
I'll patch just this one timing-sensitive test out of the build for now,
leaving other tests in place, and see if upstream can fix the issue for
the next release.

cheers.

Comment 6 Robert-André Mauchin 🐧 2019-03-14 13:44:14 UTC
LGTM, package approved.

Comment 7 Gwyn Ciesla 2019-03-14 19:49:01 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-yudai-golcs

Comment 8 Fedora Update System 2019-04-10 04:16:26 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-44c27f48cd

Comment 9 Fedora Update System 2019-04-10 04:16:49 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-41c3abf325

Comment 10 Fedora Update System 2019-04-12 02:46:47 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc30 has been pushed to the Fedora 30 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-2019-41c3abf325

Comment 11 Fedora Update System 2019-04-12 03:55:38 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc29 has been pushed to the Fedora 29 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-2019-44c27f48cd

Comment 12 Fedora Update System 2019-04-16 03:54:49 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc30 has been pushed to the Fedora 30 stable repository. If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2019-04-22 05:09:23 UTC
golang-github-yudai-golcs-0-0.1.20190315gitecda9a5.fc29 has been pushed to the Fedora 29 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.