Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960771

--- Comment #3 from Christopher Meng <[email protected]> ---
(In reply to comment #2)
> (In reply to comment #1)
> > Maybe you can replace your source0 with 
> > 
> > https://fedorahosted.org/releases/p/o/%{name}/%{name}-%{version}.tar.xz
> 
> Thanks for the comment.  I was considering it; in general, the package name
> is implied all over the spec file anyway and not expected to change.   In
> particular the  /p/o/ part of the path also hardcodes a part of the package
> name, so using %{name} would misleadingly imply flexibility in an aspect
> that is actually hardcoded and not flexible.

In fact this is not important, so it doesn't matter~

ANd, you can simplify your files section:

%{_mandir}/man8/pkla-admin-identities.8*
%{_mandir}/man8/pkla-check-authorization.8*
%{_mandir}/man8/pklocalauthority.8*

with

%{_mandir}/man8/*.8*

This is optional, too.

-- 
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=yzHgmuXdEd&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to