Bug 443301

Summary:	Review Request: concordance - Logitech Harmony Remote Control Update Utility
Product:	[Fedora] Fedora	Reporter:	Nigel Jones <dev>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting, pertusus
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:	2008-04-21 21:37:27 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:

Description Nigel Jones 2008-04-20 13:04:38 UTC

Spec URL: 
SRPM URL: http://dev.nigelj.com/SRPMS/concordance-0.20-1.fc8.src.rpm
Description: 
Concordance enables users of the Logitech Harmony series of universal remote
controls to update their firmware and configuration information from within
linux.

rpmlint issues:
libconcord-devel.x86_64: W: no-documentation
libconcord-perl.x86_64: W: no-documentation
libconcord-python.x86_64: W: no-documentation

There is no suitable documentation...

libconcord-perl.x86_64: E: zero-length /usr/lib64/perl5/vendor_perl/5.8.8/x86_64-linux-thread-multi/auto/concord/concord.bs

I'm not 100% sure about this one, if someone can confirm it's not needed, then I'll %exclude in next revision

libconcord-perl.x86_64: W: perl-temp-file /usr/lib64/perl5/5.8.8/x86_64-linux-thread-multi/perllocal.pod

Ditto

libconcord-perl.x86_64: E: non-standard-executable-perm /usr/lib64/perl5/vendor_perl/5.8.8/x86_64-linux-thread-multi/auto/concord/concord.so 0555

Seems to match other similar files.

Nothing seems critical to me...

Just as a note though...
I've had a hell of a job getting the configure scripts to understand LDFLAGS/CFLAGS in places, hence the patches and sed's so per guidelines ( Adding to and overriding or filtering parts of these flags is permitted if there's a good reason to do so; the rationale for doing so should be reviewed and documented in the specfile especially in the override and filter cases.) I've done the patching...

Comment 1 Nigel Jones 2008-04-20 13:06:10 UTC

Opps...
Spec URL: http://dev.nigelj.com/SRPMS/concordance.spec
SRPM URL: http://dev.nigelj.com/SRPMS/concordance-0.20-1.fc8.src.rpm

Comment 2 Nigel Jones 2008-04-21 04:52:03 UTC

Okay, so I made a bit of a boo-boo...  new version...

Spec URL: http://dev.nigelj.com/SRPMS/concordance.spec
SRPM URL: http://dev.nigelj.com/SRPMS/concordance-0.20-2.fc8.src.rpm

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

Comment 3 Patrice Dumas 2008-04-21 19:11:00 UTC

It seems to be already submitted in
https://bugzilla.redhat.com/show_bug.cgi?id=328161

Comment 4 Nigel Jones 2008-04-21 21:37:27 UTC

Hmmm, I have *no* idea how I missed that okay, closing...

*** This bug has been marked as a duplicate of 328161 ***