[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: less https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: less https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|ASSIGNED Flag|fedora-review?, |fedora-review+ |needinfo?([EMAIL PROTECTED]) | --- Additional Comments From [EMAIL PROTECTED] 2007-04-17 22:37 EST --- My apologies for taking so long to get back to this! All issues are fixed in less-394-9.fc7. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: less https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added Status|MODIFIED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2007-04-05 04:08 EST --- Ed, could you please look at less-394-9.fc7 and approved this review request or if you see any reason why you wdon't want to aproved it here. Thanks -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: less
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983
--- Additional Comments From [EMAIL PROTECTED] 2007-02-19 06:34 EST ---
Thanks for your comment.
/etc/profile.d/less.{sh,csh} has the right permissions - rpmlint should not
display it as an error, the only problem was /usr/share/doc/less-394/LICENSE
permissions which are fixed in less-394-8.fc7.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: less
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983
--- Additional Comments From [EMAIL PROTECTED] 2007-02-18 17:40 EST ---
/etc/profile.d/less.{sh,csh} is 755, but should be 644:
E: less executable-sourced-script /etc/profile.d/less.sh 0755
E: less world-writable /usr/share/doc/less-394/LICENSE 0666
E: less executable-sourced-script /etc/profile.d/less.csh 0755
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: less https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Additional Comments From [EMAIL PROTECTED] 2007-02-08 04:03 EST --- All problems except of permission problem - which is reported by rpmlint are fixed in less-394-7.fc7. It is a problem to change permissions in cvs so the best solution I see is to leave them as they are. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: less https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225983] Merge Review: less
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: less
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983
--- Additional Comments From [EMAIL PROTECTED] 2007-02-03 13:37 EST ---
For build, rpmlint, etc. info please see:
http://linux.dell.com/files/fedora/FixBuildRequires/mock-results-core/i386/less-394-6.fc7.src.rpm/result/
good:
+ source matches upstream
+ license is correct
needswork or nits:
- please remove the trailing '.' from the summary
- rpmlint reports a few permissions errors:
http://linux.dell.com/files/fedora/FixBuildRequires/mock-results-core/i386/less-394-6.fc7.src.rpm/result/rpmlint.log
- please consider adding %{?dist} to the Release
- please include the license in %doc since upstream includes it
- please use the preferred BuildRoot
- please use %defattr(-,root,root,-)
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
