Bug 1065058
Summary: | Review Request: textql - Execute SQL against structured text like CSV or TSV | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Fabio Alessandro Locati <fale> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | fale, i, msuchy, package-review |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2020-07-11 00:45:37 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: | 201449 |
Description
Fabio Alessandro Locati
2014-02-13 19:35:37 UTC
Some quick comments - You can drop * BuildRequires: gcc * BuildRequires: gcc-go You package builds for me in mock. Try building the binary rpm and check for errors using rpmlint. $ rpmlint textql.spec /var/lib/mock/fedora-rawhide-x86_64/result/textql-*rpm textql.spec: W: invalid-url Source0: textql-a7fbbe2.tar textql.src: W: spelling-error %description -l en_US sql -> sq, sol, sq l textql.src: W: invalid-url Source0: textql-a7fbbe2.tar textql.x86_64: W: spelling-error %description -l en_US sql -> sq, sol, sq l textql.x86_64: W: no-manual-page-for-binary textql textql-debuginfo.x86_64: E: debuginfo-without-sources 3 packages and 1 specfiles checked; 1 errors, 5 warnings. debuginfo-without-source is particularly a problem. - https://fedoraproject.org/wiki/Common_Rpmlint_issues#debuginfo-without-sources Hope this helps. Hi :), Thanks for the comment. I've ignored the error since another package is created (textql-debuginfo-0.0-1.gita7fbbe2.fc20.i686.rpm) which contains all debug files: /usr/lib/debug /usr/lib/debug/.build-id /usr/lib/debug/.build-id/30 /usr/lib/debug/.build-id/30/f803f80c1250509f25a3984b7c5084bfd5592f /usr/lib/debug/.build-id/30/f803f80c1250509f25a3984b7c5084bfd5592f.debug /usr/lib/debug/usr /usr/lib/debug/usr/bin /usr/lib/debug/usr/bin/textql.debug I'll move the debug part in the core package and drop the second one :). You'd better wait for a while, go packaging guideline is being developed: https://fedorahosted.org/fpc/ticket/382 (In reply to Fabio Alessandro Locati from comment #2) > Hi :), > > Thanks for the comment. I've ignored the error since another package is > created (textql-debuginfo-0.0-1.gita7fbbe2.fc20.i686.rpm) which contains all > debug files: > > /usr/lib/debug > /usr/lib/debug/.build-id > /usr/lib/debug/.build-id/30 > /usr/lib/debug/.build-id/30/f803f80c1250509f25a3984b7c5084bfd5592f > /usr/lib/debug/.build-id/30/f803f80c1250509f25a3984b7c5084bfd5592f.debug > /usr/lib/debug/usr > /usr/lib/debug/usr/bin > /usr/lib/debug/usr/bin/textql.debug > > I'll move the debug part in the core package and drop the second one :). Please nuke the debuginfo, it's nonsense to include them in the main package. golang packages should have no debuginfo. ping. What is the situation here? Do you want to continue or drop this one? In the mean time a lot of polices on GOLANG has been polished, and therefore I've re-created the SPEC from scratch. SPEC: https://fale.fedorapeople.org/rpms/golang-github-dinedal-textql.spec SRPM: https://fale.fedorapeople.org/rpms/golang-github-dinedal-textql-2.0.3-1.fc24.src.rpm KOJI: http://koji.fedoraproject.org/koji/taskinfo?taskID=15115062 Just found this https://bugzilla.redhat.com/show_bug.cgi?id=1062942#c25 and according to that this review should not block for FE-NEEDSPONSOR queue. That is true. I forgot to remove the blocking status from this repo when I got accepted as a packager. Good catch Parag :) and thanks! This is an automatic action taken by review-stats script. The submitter account doesn't exist anymore, therefore this ticket will be closed. |