[Bug 1603146] Review Request: cockpit-ostree - Cockpit user interface for rpm-ostree ( split out from existing cockpit.srpm)

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

--- Comment #2 from Igor Gnatenko  ---
Ugh, I was pretty sure I did the review. I'm very sorry.
> Source: cockpit-ostree-%{version}.tar.gz

Add some comment how to obtain it.
> %build
> # There is nothing to do, release tarballs already have dist/.

Remove this entirely.
> make install DESTDIR=%{buildroot}

%make_install
> find %{buildroot} -type f >> files.list
> sed -i "s|%{buildroot}find %{buildroot} -type f >> files.list

sed -i "s|%{buildroot}||" *.list||" *.list

Isn't there some better way? Also what about directory ownership?