[Bug 467644] Review Request: sugar-memorize - Memorize for Sugar

[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.


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


Simon Schampijer <simon@xxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |simon@xxxxxxxxxxxxx




--- Comment #7 from Simon Schampijer <simon@xxxxxxxxxxxxx>  2008-11-19 03:45:22 EDT ---
Yup I did a full review. In the spec you posted above - the comment says that
this is removed but it is still present :)

{{{
#delete hidden files
find %{buildroot} -name '.*' -exec rm -f {} ';'
#wrong end-line-encoding
#sed -i 's/\r//'
%{buildroot}%{sugaractivitydir}Memorize.activity/images/stock-buddy.svg
#fix rpmlint non-executable-script error
#find %{buildroot}%{sugaractivitydir}Memorize.activity/*.py -type f | xargs
chmod a+x
#find %{buildroot}%{sugaractivitydir}Memorize.activity/csound/csoundserver.py
-type f | xargs chmod a+x
#remove exec bit
#find %{buildroot}%{sugaractivitydir}Memorize.activity/images/*.svg -type f
-name \* -exec chmod 644 {} \;
}}}

-- 
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.

_______________________________________________
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]