[Bug 866901] Review Request: gogui - GUI to play game of Go

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

 



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

--- Comment #16 from Mohamed El Morabity <pikachu.2014@xxxxxxxxx> ---
> 4) DON'T pipe sed through cat, it's useless, unreadable and considered bad
> practice. Simply use sed only instead:
> 
> for FILE in bin/*; do
> 	if [ -f $FILE -a -x $FILE ]; then
> 		sed -e "s;^GOGUI_LIB=.*;GOGUI_LIB=\"%{_javadir}/%{name}\";" \
> 		-e "s;^JAVA_DEFAULT=.*;JAVA_DEFAULT=\"$JAVA_DEFAULT\";" \
> 		> $RPM_BUILD_ROOT%{_prefix}/$FILE
> 		chmod a+x $RPM_BUILD_ROOT%{_prefix}/$FILE
> 	fi
> done
Mistake, use this instead:

 for FILE in bin/*; do
     if [ -f $FILE -a -x $FILE ]; then
         sed -e "s;^GOGUI_LIB=.*;GOGUI_LIB=\"%{_javadir}/%{name}\";" \
         -e "s;^JAVA_DEFAULT=.*;JAVA_DEFAULT=\"$JAVA_DEFAULT\";" $FILE \
         > $RPM_BUILD_ROOT%{_prefix}/$FILE
         chmod a+x $RPM_BUILD_ROOT%{_prefix}/$FILE
     fi
 done

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



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