On 27 June 2016 at 21:56, Marc Haber <[email protected]> wrote:
> On Sun, Jun 26, 2016 at 10:39:04PM +0200, Marc Haber wrote:
>> On Mon, Oct 26, 2015 at 10:10:35AM +0200, Riku Voipio wrote:
>> > ser2net uses : as field separator in ser2net.conf. Udev creates by-path and
>> > by-id paths to identify unique serial ports. These paths are highly
>> > useful as ttyUSB* might end up shuffling around.

>> I have forwarded this upstream.

> Here is upstream's answer:
> |Unfortunately, the patch provided is not really a very good idea. If there
> |is a quote later in the data (after the separating :), it would break things
> |up wrong.

Fair enough, it's a quick hack.

> |
> |To do this right would require more complicated parsing or some creative
> |handling.  I've come up with the following options:
> |
> |* Do complicated parsing
> |* Do backslash handling and use \x3a for the colon.
> |* Allow a name (like BANNER) to be created for a device.

option 4) use a standard format like yaml and get quoting/escaping
automatically. hand-crafted parsers are a bit risky anyways.

> |
> |Of the three, I prefer the last.  These device paths are long and fairly
> |meaningless, this would allow a meaningful name to be created for these
> |devices.  And it would probably be the simplest.
> |
> |-corey
>
> He is also now subscribed here. Welcome, corey.

Thanks for working on ser2net!

Riku

Reply via email to