Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=905255 --- Comment #49 from Simone Caronni <negativo17@xxxxxxxxx> --- Oops, you're right, the rm line should be in the %build or %install section. --- open-vm-tools.spec.old 2013-04-12 08:46:27.422858942 +0200 +++ open-vm-tools.spec 2013-04-12 10:29:41.633744622 +0200 @@ -107,6 +107,7 @@ # Remove unnecessary files from packaging find %{buildroot}/%{_libdir} -name '*.la' -delete rm -fr %{buildroot}/%{_defaultdocdir} +rm -f docs/api/build/FreeSans.ttf %if 0%{?fedora} || 0%{?rhel} >= 7 @@ -129,7 +130,7 @@ /usr/sbin/ldconfig if [ $1 -eq 1 ] ; then # Initial installation - /bin/systemctl daemon-reload >/dev/null 2>&1 || : + /bin/systemctl enable %{toolsdaemon}.service >/dev/null 2>&1 || : fi %preun btw Fedora already has that font by default in package gnu-free-sans-fonts. Should we add gnu-free-sans-fonts as BuildRequires? Koji does not give any error when building the current package. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2iAxEllYkp&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review