[Bug 754554] Review Request: presence - Bi-directional audio/video connections

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Fabian Deutsch <fabian.deutsch@xxxxxx> 2011-11-18 08:34:38 EST ---
Unbelievable :) Thanks for this quick review, Stanislav!

(In reply to comment #12)
> (In reply to comment #11)
> > (In reply to comment #10)
> > 
> > > Issues:
> > > [!]: MUST Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the
> > > beginning of %install. (EPEL5)
> > 
> > Fixed.
> 
> Well this was really needed only for EPEL5, but it doesn't hurt :-)

I'll take this into account and will have a look at other packages what is
needed.

> > I suppose this was because of globbing, I added the dir explicitly.
> 
> OK now. Though the "%{_datadir}/presence/*" entry is now redundant :-)

Hum, I thought that this just pulls the path, but not it's children.
Will be removed though.

> > > [!]: SHOULD Package functions as described.
> > > 
> > > While I don't have a webcam in my pc, I'd assume the application shows icons in
> > > bottom screen area. Those buttons work, but they have no icons so it's
> > > impossible to navigate. Perhaps some missing requires?
> > 
> > Yes, I suppose you ain't running GNOME? :) I added a dependency on
> > gnome-icon-theme and -symbolic this should solve this.
> > And you'll need to be creative to test the software on a machine without a
> > webcam :)
> 
> Well I can't seem to make it work in Xnest (segfaults), Xephyr (works, but it
> has the same problem with icons) even though I already had those icons
> installed. You are right I am not running Gnome, so my environment is a little
> non-standard. It shouldn't matter though. I should be able to run any
> application under any WM/DE. 

Crashes in Xnest as expected as it requires GL, which ain't supported by Xnest
(afaik). I am also seeing the icon problems on Fedora 15, but they are not
present on Fedora 16. Are you running F15?

> There was another weird thing when I tried to resize the window. It jumped to
> maximum width (as in maxiumu allowed by my graphics gard - wider than my
> monitor).

This is weird. I actually don't do much with the window size, just restricting
the aspect ratio and switching to fullscreen on demand. Maybe a result of the
WM and aspect-ratio-based-resizing.

> Normally I'd probably tell the packager to sort it out with upstream first, but
> since you are the upstream I think you are in a perfect position to fix this
> later on...That said:
>
> APPROVED

What a friday :) Thanks, fabian.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- 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]