philippelatulippe / gitso

Automatically exported from code.google.com/p/gitso
1 stars 0 forks source link

Change color depth from Linux helper ? #9

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Gitso 0.5 ... first of all, congrads, you are doing excellent work :)

When in windows as a giver of help I can change the color depth to 8 bit
for faster screen updating through the GUI. How do I do this as a giver of
help from a linux box ?

Keep up the great work

Cheers

Dave

Original issue reported on code.google.com by dave6...@gmail.com on 27 Oct 2008 at 10:15

GoogleCodeExporter commented 8 years ago
Gitso was started on the premise that it's a simple little app. That's not to 
say
there shouldn't be any preferences involved. Unfortunately, we don't have the
resources to include this in the next release.  But in future versions, it'd
definitely be a nice thing to have as well as improved defaults. I think there 
are
many speed improvements that could be gained by configuring precisely these 
things.
Thanks for the input!

Original comment by gerbe...@gmail.com on 29 Oct 2008 at 6:06

GoogleCodeExporter commented 8 years ago
OK no problem. As a workaround I amended '/usr/share/gitso/Gitso.py' line 208 
to ...

   self.returnPID = os.spawnlp(os.P_NOWAIT, 'vncviewer', 'vncviewer', '-bgr233',
'-listen')

Colors a little off but uses 8 bits per pixel instead of 32 :)

Cheers

Dave

Original comment by dave6...@gmail.com on 29 Oct 2008 at 7:37

GoogleCodeExporter commented 8 years ago
Cheers for digging around in the code! We'll consider this flag (-bgr233) for 
the
next release. Alternatively, we could add a command-line option 
"--low-colors"...
This also would be a great help for giving support to OS X clients, which is
noticeably slower than Linux or Windows connections. There have been some 
complaints
about lower bpp settings in the past, so we'll also take that input into 
account. 
Ultimately, there could be an option to let the user decide.  Thanks again.

Aaron-

Original comment by gerbe...@gmail.com on 29 Oct 2008 at 8:30

GoogleCodeExporter commented 8 years ago
I love digging in code. If it was not for me trying to finish v2 of my own open
source project I would offer to give you a hand. Cheers

Dave

Original comment by dave6...@gmail.com on 29 Oct 2008 at 9:21

GoogleCodeExporter commented 8 years ago
Try hitting F8 after you're connected.  It might bring up a menu with options.  
If
not, maybe we'll switch out the vncviewer for linux.

Original comment by xburn...@gmail.com on 11 Nov 2008 at 5:09

GoogleCodeExporter commented 8 years ago
f8 does bring up a menu with some options but nothing to help compression, 
which you
get in the windows version :)

Cheers

Dave

Original comment by dave6...@gmail.com on 18 Nov 2008 at 6:57

Attachments:

GoogleCodeExporter commented 8 years ago
Hi. I modified the code to work with low colors (bgr233).
- command line option --low-colors
- checkbox on gui

This also affects issue 52.

Who can is send the patch to (or get permission for svn ci)?

~markus

Original comment by herr.bi...@gmail.com on 26 Nov 2010 at 11:36

Attachments:

GoogleCodeExporter commented 8 years ago
Does this work on all platforms or just linux? Also, to submit the patch, feel 
free to attach it to this support ticket.

Original comment by gerbe...@gmail.com on 29 Nov 2010 at 4:53

GoogleCodeExporter commented 8 years ago
I Just tried it under Linux (Ubuntu 10.10). See Attachment for the patch.

~markus

Original comment by herr.bi...@gmail.com on 9 Dec 2010 at 12:00

Attachments:

GoogleCodeExporter commented 8 years ago
Could you attach a patched version ? 

Original comment by t...@leasingborsen.dk on 8 Sep 2011 at 7:43

GoogleCodeExporter commented 8 years ago
I attached the patch to my last post. Save it into the gitso root dir, cd to it 
and type

patch -p0 < low-color-support.patch

~markus

Original comment by herr.bi...@gmail.com on 8 Sep 2011 at 8:15

GoogleCodeExporter commented 8 years ago
Fixed in r144.

Original comment by herr.bi...@gmail.com on 26 Sep 2012 at 8:53