[Bug 441072] Review Request: cwiid - Library and tools for comunicating with a wiimote

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cwiid - Library and tools for comunicating with a wiimote


https://bugzilla.redhat.com/show_bug.cgi?id=441072





------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx  2008-04-25 11:45 EST -------
Almost okay.
For 0.6.00-5:

* Directory permission issue
  - rpmlint shows:
----------------------------------------------------------
cwiid-wminput.i386: E: non-standard-dir-perm /etc/cwiid 0644
cwiid-wminput.i386: E: non-standard-dir-perm /etc/cwiid/wminput 0644
----------------------------------------------------------
    These directories should have 0755 permission.

    When you write
----------------------------------------------------------
    97  %files wminput
    98  %config(noreplace) %attr(0644,root,root) %{_sysconfdir}/cwiid
----------------------------------------------------------
    this actually sets all permissions of files/directories/etc
    under %_sysconfdir/cwiid ant itself to 0644.

    To avoid this, I recommend to write just:
----------------------------------------------------------
%config(noreplace) %{_sysconfdir}/cwiid/
----------------------------------------------------------
    (I usually add slash to the end intentionally if it is a
     directory) and change the permissions beforehand at %install like:
----------------------------------------------------------
find $RPM_BUILD_ROOT%{_sysconfdir} -type d -exec chmod 0644 {} ';'
----------------------------------------------------------

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]