FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Kubuntu Development

 
 
LinkBack Thread Tools
 
Old 03-07-2011, 04:46 AM
Steve Langasek
 
Default lp:~goelkunal/kdelibs/debian into lp:~kubuntu-members/kdelibs/ubuntu

Review: Needs Information
The patch itself seems to be straightforward, but why are the glClearColor(), glClear() calls not needed here? Presumably they were added for a reason, and we should be sure that reason no longer applies before making such a change.

I see that these API calls are also part of GLES; so if these calls need to be kept, an alternative would be for plasma to select either GL or GLES as needed.
--
https://code.launchpad.net/~goelkunal/kdelibs/debian/+merge/51994
Your team Kubuntu Members is subscribed to branch lp:~kubuntu-members/kdelibs/ubuntu.

--
kubuntu-devel mailing list
kubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
 
Old 03-12-2011, 05:32 AM
Steve Langasek
 
Default lp:~goelkunal/kdelibs/debian into lp:~kubuntu-members/kdelibs/ubuntu

Review: Approve
As the upstream discussion shows, even for applets using the GLApplet interface (of which there appear to not be any), these calls should not be needed here. So this change is needed on ARM, and has no adverse affects elsewhere.
--
https://code.launchpad.net/~goelkunal/kdelibs/debian/+merge/51994
Your team Kubuntu Members is subscribed to branch lp:~kubuntu-members/kdelibs/ubuntu.

--
kubuntu-devel mailing list
kubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
 
Old 03-21-2011, 10:01 AM
Jonathan Riddell
 
Default lp:~goelkunal/kdelibs/debian into lp:~kubuntu-members/kdelibs/ubuntu

Review: Approve
Merged by Harald
--
https://code.launchpad.net/~goelkunal/kdelibs/debian/+merge/51994
Your team Kubuntu Members is subscribed to branch lp:~kubuntu-members/kdelibs/ubuntu.

--
kubuntu-devel mailing list
kubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
 
Old 03-21-2011, 10:01 AM
Jonathan Riddell
 
Default lp:~goelkunal/kdelibs/debian into lp:~kubuntu-members/kdelibs/ubuntu

The proposal to merge lp:~goelkunal/kdelibs/debian into lp:~kubuntu-members/kdelibs/ubuntu has been updated.

Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~goelkunal/kdelibs/debian/+merge/51994
--
https://code.launchpad.net/~goelkunal/kdelibs/debian/+merge/51994
Your team Kubuntu Members is subscribed to branch lp:~kubuntu-members/kdelibs/ubuntu.

--
kubuntu-devel mailing list
kubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
 

Thread Tools




All times are GMT. The time now is 08:20 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org