Bug 226107 – Merge Review: lslk

Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.

Bug 226107 - Merge Review: lslk

Summary:	Merge Review: lslk

Status:	CLOSED NEXTRELEASE

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	Package Review (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Parag AN(पराग)
QA Contact:	Fedora Package Reviews List
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2007-01-31 14:35 EST by Nobody's working on this, feel free to take it
Modified:	2010-10-06 05:50 EDT (History)
CC List:	2 users (show)

See Also:
Fixed In Version:	lslk-1.29-24.fc15
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2010-10-06 05:50:16 EDT
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Flags:	panemade: fedora‑review+

Attachments	(Terms of Use)
spec cleanup (1.72 KB, patch) 2010-10-05 11:48 EDT, Parag AN(पराग)	no flags	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Nobody's working on this, feel free to take it 2007-01-31 14:35:27 EST

Fedora Merge Review: lslk

http://cvs.fedora.redhat.com/viewcvs/devel/lslk/
Initial Owner: kzak@redhat.com

Comment 1 Parag AN(पराग) 2010-10-05 11:48:56 EDT

Created attachment 451705 [details]
spec cleanup

This git patch clean this package for this review. Please commit this patch or allow to commit.

Comment 2 Karel Zak 2010-10-06 04:28:22 EDT

Committed, pushed to rawhide. THANKS!

Comment 3 Parag AN(पराग) 2010-10-06 04:47:58 EDT

oops! looks like my patch just added %doc but when I locally built this new package, missed to add them actually and thus rpmlint was clean.

With the new build rpmlint is complaining

lslk.x86_64: E: non-standard-executable-perm /usr/share/doc/lslk-1.29/Inventory 0555L
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.

lslk.x86_64: W: spurious-executable-perm /usr/share/doc/lslk-1.29/Inventory
The file is installed with executable permissions, but was identified as one
that probably should not be executable.  Verify if the executable bits are
desired, and remove if not.

This can be fixed by adding following line in end of %prep section
chmod 644 Inventory

Can you please fix this?

Comment 4 Karel Zak 2010-10-06 05:42:55 EDT

(In reply to comment #3)
> This can be fixed by adding following line in end of %prep section
> chmod 644 Inventory
>
> Can you please fix this?

 Fixed.

Comment 5 Parag AN(पराग) 2010-10-06 05:50:16 EDT

Thanks for quick fix.

APPROVED this review.

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