Bug 2816 - cyclades-serial-client only supports 8 bit data size (cs8, not cs7 etc)
Summary: cyclades-serial-client only supports 8 bit data size (cs8, not cs7 etc)
Status: ASSIGNED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Serial port redirection (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: MediumPrio
Assignee: Peter Åstrand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-22 15:15 CEST by Peter Åstrand
Modified: 2013-08-22 14:30 CEST (History)
0 users

See Also:
Acceptance Criteria:

Attachments
Patch for sercd to switch to 8 bits at startup (509 bytes, patch)
2008-08-28 18:32 CEST, Peter Åstrand 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Description Peter Åstrand cendio 2008-06-22 15:15:39 CEST 
Our cyclades-serial-client only supports emulating a serial port with 8 bits per byte (cs8). If the physical port is configured for something else when sercd starts, c-s-c will print an error:

cyclades-ser-cli /var/opt/thinlinc/sessions/astrand/2/dev/ttyS0: NOTICE: Opening /dev/pts/5 pseudo-tty
cyclades-ser-cli /var/opt/thinlinc/sessions/astrand/2/dev/ttyS0: ERR: Can't set termios : Invalid argument

However, if using the LD_PRELOAD library with an application that changes to cs8, things works. 

For completeness, it might be nice to support cs7 etc someday. Currently, this is not possible due to limitations of ptys. See http://marc.info/?l=linux-serial&m=120612342110347&w=2.
Comment 3 Peter Åstrand cendio 2008-08-28 18:42:26 CEST 
Followup to comment #1: I was using the JavaComm library comm3.0_u1_linux.zip. One good thing is that several demo programs are included. For example, the "BlackBox" application looks interesting. 

On the stupid side, it's necessary to enter all serial port paths into javax.comm.properties. Ie, this file must be personal, meaning that all users also needs to have a personal comm.jar, meaning that you have to fiddle with CLASSPATH...
Note You need to log in before you can comment on or make changes to this bug.