[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 #17 from Dominik 'Rathann' Mierzejewski <dominik@xxxxxxxxxxxxxx> ---
(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.

> APPROVED.

Is it still approved if I don't make the above change?

> 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]