Hi,

Am 01.10.2010 00:48, schrieb Cristian Morales Vega:
> 2010/9/30 Cristian Morales Vega<[email protected]>:
>> 2010/9/30 Cristian Morales Vega<[email protected]>:
>>> In the "Developers" forum I badly copy&pasted these patches. I
>>> resubmit them here to make them clearer since it seems some of them
>>> didn't hit the CVS.
>>
>> A new one to solve the "/usr/share/pixmaps/gmerlin-player.png ->
>> ../../src/packages/BUILDROOT/gmerlin-0.4.3-1.x86_64/usr/share/gmerlin/icons/player_icon.png"
>> problem explained in the forum.
>> Not perfect, but solves the problem and at least RPM removes the
>> unneeded "../" automatically.
>> It also install the icons in /usr/share/icons following the Icon Theme
>> Specification.
>
> And a new problem. There are multiple uses of GTK_WIDGET_SET_FLAGS,
> but it is deprecated since GTK 2.22. In its place, since GTK 2.18,
> there is gtk_widget_set_*().
> And since 2.21.6 GtkNotebookPage is also deprecated.

Ok, now it gets messy. My suggestion: Can you make this (and future) patches 
against CVS
instead of 0.4.3? Other than that, your fixes (bg_gtk_* wrappers) are perfectly 
correct.

I'm currently in a preparation phase for 1.0.0 and fixing stuff in the current 
code
is much more useful than repeating old fixes. Some of the gtk problems are 
already fixed
in CVS, and I don't want to go trough this more than once.

Burkhard

------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
Gmerlin-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gmerlin-general

Reply via email to