[Bug 1065539] Review Request: subunit - C bindings for subunit

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

 



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



--- Comment #13 from Pádraig Brady <pbrady@xxxxxxxxxx> ---
BTW this loop looks very inefficient:

for filt in filters/*; do
  sed 's,/usr/bin/env ,/usr/bin/,' $filt > ${filt}.new
  chmod 0755 ${filt}.new
  touch -r $filt ${filt}.new
  mv -f ${filt}.new $filt
done

Could it be replaced with:

sed -i 's,/usr/bin/env ,/usr/bin/,; q' filters/*

or

find filters/ -exec sed -i 's,/usr/bin/env ,/usr/bin/,; q' {} +

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
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]