[Bug 820544] Review Request: libguac-client-rdp - RDP support for guacd

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

 



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

Terje Røsten <terjeros@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |terjeros@xxxxxxxxxxxx

--- Comment #41 from Terje Røsten <terjeros@xxxxxxxxxxxx> ---
Hi guys,

I support Michael's words about the not-no-private-libs. 

Keep libs really private (apps might need patching) or let the default location
be ok. No using in moving for the sake of it.

The use of %{__isa} is solving a problem, however creating a new one. 

I have lots a real life experience with the wrong arch devel package is
installed issue, the archness of src package seems more a theoretical
problem. 

I have never used yum-builddep, mock/koji is better for such
operations any way?

Some comments are always good, when a stack has deps between packages
a common packager has no problem remembering how and why. However, Fedora
is large project and packages change maintainers all the time. Please
try to keep packages simple and easy to maintain for the next maintainer.


Btw: the package seems good outside this issues.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



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