X-Authentication-Warning: delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient: geda-user AT delorie DOT com
X-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=googlemail.com; s=20120113;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type;
        bh=JwLjnkenjSkJKWGkFNPXtpOJCpZqUOgOA9YZjWuGIx4=;
        b=pwkPf46ce0t4gD2sXFiCxkpOztxeQCnrtSWfXsriRHy17pmbfGVZnYRPYxWUkFmrlR
         qUbBVEhqGThy0MjzpvomvDKOlJ6iklNNGJ+oG+PqgkGa3E8bpOW/p9HLQw00aWyA/LWe
         FV+axOxOi/7o1xAZFhzibSzoAr6yQgLt3eGybjBFYE8b2HKne06xoOVzXOarLBABU3Jb
         VvaYUShmA1qg+qNerA5FznRx7+Nde2Nd2aLCgCx2PZMfbBpnOWKwoPl+33D8XyOMA5zz
         EbmMYWCgpecHgc9Kvyrs4dlLbIdDA/pFCsdqins+wPRYc1DYkDRCFUEeGkxbHJup2TJD
         aUYA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20130820;
        h=x-gm-message-state:mime-version:in-reply-to:references:date
         :message-id:subject:from:to:content-type;
        bh=JwLjnkenjSkJKWGkFNPXtpOJCpZqUOgOA9YZjWuGIx4=;
        b=Q/eXq3Fl638ExIrMv23BGUEzZbU4rtKq9qr/QY9oQ/m47jA+S6+7d/zUBzAk7LuMJZ
         71WUxP6RlDTG0I5qAe3QFENwsFEtcLw65CJk1H/GgQJ7J53I7PuqbGLqR4301dvNldNt
         IWM2//0xfn0lkjuz1jp8mUEtxt7XbKNU+z0lMO0Z0LMx/rqBINiaG3XTUmxwyTFho12A
         qjiwmr5nljV/o2NCxsmy96RQ9QRW++IA3hPcBCcH3D0nGP4Uh6hweYzbp0047tdHCEeE
         yqTWCFYCtTmftA0j/GgdGuQaz/N1nG2tMiJye4o8wmiLsSvz/P3ooZ0p+5Yrr6se4Ov8
         yJwA==
X-Gm-Message-State: AG10YOSBsizL7n5cySkc9V6Tz0JDXFsT1pI7M7B2d+UOZykbmZZ/pUD/HKz7pLU8lrFmfA2hZLqLp2SmVey1VQ==
MIME-Version: 1.0
X-Received: by 10.60.81.67 with SMTP id y3mr17776819oex.61.1455627049847; Tue,
 16 Feb 2016 04:50:49 -0800 (PST)
In-Reply-To: <20160216092912.7f7439f703b49175a21dbb1b@gmail.com>
References: <20160215215221 DOT fd472794e7b9446a243bfc40 AT gmail DOT com>
	<201602152055 DOT u1FKtM4K011038 AT envy DOT delorie DOT com>
	<20160215220938 DOT bbc7eaa59d827cd0b261ea97 AT gmail DOT com>
	<D3167F4D-4750-49F8-9125-BDC4937F5C69 AT noqsi DOT com>
	<201602152135 DOT u1FLZrw9012774 AT envy DOT delorie DOT com>
	<7F210DE7-0A0B-42F9-ABBE-2C2768621186 AT noqsi DOT com>
	<20160216081722 DOT 1065cbed6653d3da4ffc7498 AT gmail DOT com>
	<201602160724 DOT u1G7Ox26001785 AT envy DOT delorie DOT com>
	<20160216085628 DOT b70143c330cd4da98a4603d3 AT gmail DOT com>
	<201602160805 DOT u1G85d8c003148 AT envy DOT delorie DOT com>
	<20160216092912 DOT 7f7439f703b49175a21dbb1b AT gmail DOT com>
Date: Tue, 16 Feb 2016 12:50:49 +0000
Message-ID: <CAJXU7q_w5NunkojiCr36RHRTq0hJ+PZP1e0GumTRMoGXcvgRXQ@mail.gmail.com>
Subject: Re: [geda-user] pcb import schematic crash, parantheses in netname
From: "Peter Clifton (petercjclifton AT googlemail DOT com) [via geda-user AT delorie DOT com]" <geda-user AT delorie DOT com>
To: gEDA User Mailing List <geda-user AT delorie DOT com>
Content-Type: multipart/alternative; boundary=047d7b417f953b7ad7052be2959f
Reply-To: geda-user AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: geda-user AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

--047d7b417f953b7ad7052be2959f
Content-Type: text/plain; charset=UTF-8

Looks like some string quoting will be required...

Next thing of course, someone will insist that they NEED to use various
different types of quotation marks in their net names too, so possibly a
healthy dose of string escaping may also be required.

DJ... You wrote the action stuff, and the new import tool, what are your
thoughts on legal character set?  (Should it match whatever we can read in
.net files? Any idea what that allows? (I'm not currently at my computer).

Clearly an error message in invalid input would be good, but I'm not
convinced that allowing brackets (of any kind), quotation marks or any
whitespace characters should be done without a bit of consideration. (Might
be ok, just warrants thought)

Thoughts?

I'd be tempted to consider $ @ ? for exclusion too, but I'm not really sure
why.

Peter
On 16 Feb 2016 08:30, "Nicklas Karlsson (nicklas DOT karlsson17 AT gmail DOT com) [via
geda-user AT delorie DOT com]" <geda-user AT delorie DOT com> wrote:

> > You hack the Makefile :-)
>
> I tried "./configure --enable-debug" but I do not think it define "DEBUG".
>
> > Or you could just run "gnetlist -g pcbfwd foo.sch" which is all action.c
> is doing.
>
> Then I ran gsch2pcb it execute:
>   gnetlist -g pcbpins -o Mcu.cmd Mcu.sch
>   gnetlist -g PCB -o Mcu.net Mcu.sch
>   gnetlist -g gsch2pcb -o Mcu.new.pcb -m gnet-gsch2pcb-tmp.scm Mcu.sch
> and I could import Mcu.net into pcb without crash. Then I import schematic
> and there is parantheses in a net name import schematic in pcb crash.
>
> Where the crash happen:
>   Action: Netlist(Add,Encoder2Q1,U1-19)
>   Action: Netlist(Add,Encoder1(Q1)
>   unknown action `,U1-81)'
>   Action: Netlist(Add,Encoder1(Q1)
>   Segmentation fault
>
>
> I intended to figure out where crash happen and hopefully solve it.
>
>
> Nicklas Karlsson
>

--047d7b417f953b7ad7052be2959f
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p dir=3D"ltr">Looks like some string quoting will be required...</p>
<p dir=3D"ltr">Next thing of course, someone will insist that they NEED to =
use various different types of quotation marks in their net names too, so p=
ossibly a healthy dose of string escaping may also be required.</p>
<p dir=3D"ltr">DJ... You wrote the action stuff, and the new import tool, w=
hat are your thoughts on legal character set?=C2=A0 (Should it match whatev=
er we can read in .net files? Any idea what that allows? (I&#39;m not curre=
ntly at my computer).</p>
<p dir=3D"ltr">Clearly an error message in invalid input would be good, but=
 I&#39;m not convinced that allowing brackets (of any kind), quotation mark=
s or any whitespace characters should be done without a bit of consideratio=
n. (Might be ok, just warrants thought)</p>
<p dir=3D"ltr">Thoughts?</p>
<p dir=3D"ltr">I&#39;d be tempted to consider $ @ ? for exclusion too, but =
I&#39;m not really sure why.</p>
<p dir=3D"ltr">Peter</p>
<div class=3D"gmail_quote">On 16 Feb 2016 08:30, &quot;Nicklas Karlsson (<a=
 href=3D"mailto:nicklas DOT karlsson17 AT gmail DOT com">nicklas DOT karlsson17 AT gmail DOT com<=
/a>) [via <a href=3D"mailto:geda-user AT delorie DOT com">geda-user AT delorie DOT com</a=
>]&quot; &lt;<a href=3D"mailto:geda-user AT delorie DOT com">geda-user AT delorie DOT com=
</a>&gt; wrote:<br type=3D"attribution"><blockquote class=3D"gmail_quote" s=
tyle=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">&gt;=
 You hack the Makefile :-)<br>
<br>
I tried &quot;./configure --enable-debug&quot; but I do not think it define=
 &quot;DEBUG&quot;.<br>
<br>
&gt; Or you could just run &quot;gnetlist -g pcbfwd foo.sch&quot; which is =
all action.c is doing.<br>
<br>
Then I ran gsch2pcb it execute:<br>
=C2=A0 gnetlist -g pcbpins -o Mcu.cmd Mcu.sch<br>
=C2=A0 gnetlist -g PCB -o Mcu.net Mcu.sch<br>
=C2=A0 gnetlist -g gsch2pcb -o Mcu.new.pcb -m gnet-gsch2pcb-tmp.scm Mcu.sch=
<br>
and I could import Mcu.net into pcb without crash. Then I import schematic =
and there is parantheses in a net name import schematic in pcb crash.<br>
<br>
Where the crash happen:<br>
=C2=A0 Action: Netlist(Add,Encoder2Q1,U1-19)<br>
=C2=A0 Action: Netlist(Add,Encoder1(Q1)<br>
=C2=A0 unknown action `,U1-81)&#39;<br>
=C2=A0 Action: Netlist(Add,Encoder1(Q1)<br>
=C2=A0 Segmentation fault<br>
<br>
<br>
I intended to figure out where crash happen and hopefully solve it.<br>
<br>
<br>
Nicklas Karlsson<br>
</blockquote></div>

--047d7b417f953b7ad7052be2959f--