[Bug 1324784] Review Request: pseudo - Advanced tool for simulating superuser privileges

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

 



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



--- Comment #18 from Igor Gnatenko <ignatenko@xxxxxxxxxx> ---
(In reply to Dominik 'Rathann' Mierzejewski from comment #17)
> (In reply to Igor Gnatenko from comment #15)
> > if getfattr --help >/dev/null 2>&1; then
> >     xattr_runs=true
> > 
> > 
> > hmm, then don't drop it.
> 
> As I said, it's only a warning.
> 
> (In reply to Igor Gnatenko from comment #16)
> > Just checked all bits. On thing MUST be fixed:
> > %{_libdir}/pseudo/libpseudo.so -> %{_libdir}/%{name}/
>      
> Why? %{name} is not shorter than "pseudo" and I'd argue that readability is
> better with the actual name instead of a macro.
Point was about owning /usr/lib64/pseudo/. But you was owning file without dir.
> 
> > APPROVED.
> 
> Is it still approved if I don't make the above change?
Sure!
> 
> > Dominik, can you add me as co-maintainer in PkgDB when you will request
> > package? I would be happy to help with it (as it could potentially become
> > replacement for fakechroot for RPM test suide).
> 
> Yes.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]