Bug 1428437
Summary: | Review Request: golang-github-bkaradzic-go-lz4 - Port of LZ4 lossless compression algorithm to Go | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Fabio Valentini <decathorpe> |
Component: | Package Review | Assignee: | Jan Chaloupka <jchaloup> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | jchaloup, package-review |
Target Milestone: | --- | Flags: | jchaloup:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-03-16 21:18:38 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1427634 |
Description
Fabio Valentini
2017-03-02 15:42:53 UTC
Just a nit. I would suggest to move %{gopath}/src/%{import_path}/testdata line form %files unit-test-devel into %install section as echo "%%{gopath}/src/%%{import_path}/testdata" >> unit-test-devel.file-list right under cp -pvr testdata %{buildroot}/%{gopath}/src/%{import_path}/ So the directory ownership is as close to the command it created it as possible. Wrt. book copyrights, Tom, can you take a look at it? Good point, I moved the directory entry to the unit-test-devel.file-list, just like you suggested. That work is clearly and unambiguously in the Public Domain in the US as a result of "Klinger v Conan Doyle Estate, Ltd.": http://law.justia.com/cases/federal/appellate-courts/ca7/14-1128/14-1128-2014-06-16.html Note: Not _all_ of the Sherlock Holmes books are out of copyright, but this one is. jchaloup, can you please look at this again, now that the copyright issue is out of the way? Thanks Tom. Approved. Thanks for the review! Pkgdb request is sent. If you (jchaloup) need some packages reviewed at any point in time, just ping me - because it's been really nice to get those syncthing deps into fedora so quickly! Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/golang-github-bkaradzic-go-lz4 golang-github-bkaradzic-go-lz4-1.0.0-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-99747f14b8 golang-github-bkaradzic-go-lz4-1.0.0-1.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-71bcfd28d9 golang-github-bkaradzic-go-lz4-1.0.0-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-2e516c276d golang-github-bkaradzic-go-lz4-1.0.0-1.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-2017-2e516c276d golang-github-bkaradzic-go-lz4-1.0.0-1.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-2017-71bcfd28d9 golang-github-bkaradzic-go-lz4-1.0.0-1.fc26 has been pushed to the Fedora 26 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-2017-99747f14b8 golang-github-bkaradzic-go-lz4-1.0.0-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report. golang-github-bkaradzic-go-lz4-1.0.0-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report. golang-github-bkaradzic-go-lz4-1.0.0-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report. |