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



--- Comment #2 from Nathan Scott <[email protected]> ---
(In reply to Robert-André Mauchin from comment #1)
> Hello,
> 

Hi, thanks for reviewing!

>  - You *must* install the LICENSE file with the %license macro:
> 
> %files
> %license LICENSE
> 

Fixed.

>  - You should own /usr/lib64/redis and /usr/lib64/redis/modules or Requires
> a package that own these directories:
> 
> [!]: Package requires other packages for directories it uses.
>      Note: No known owner of /usr/lib64/redis, /usr/lib64/redis/modules
> [!]: Package must own all directories that it creates.
>      Note: Directories without known owners: /usr/lib64/redis/modules,
>      /usr/lib64/redis
> 
>    (Personally I think the Redis package sould be updated to own these
> directories)
> 

Fixed up as per BZ 1513291.

Updated packages at:
Spec URL: https://nathans.fedorapeople.org/rejson/rejson.spec
SRPM URL: https://nathans.fedorapeople.org/rejson/rejson-0.99.1-2.fc26.src.rpm

cheers.

-- 
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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to