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> <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: 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]" To: gEDA User Mailing List 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]" 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

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 p= ossibly a healthy dose of string escaping may also be required.

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'm not curre= ntly 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 mark= s or any whitespace characters should be done without a bit of consideratio= n. (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<= /a>) [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:
=C2=A0 gnetlist -g pcbpins -o Mcu.cmd Mcu.sch
=C2=A0 gnetlist -g PCB -o Mcu.net Mcu.sch
=C2=A0 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:
=C2=A0 Action: Netlist(Add,Encoder2Q1,U1-19)
=C2=A0 Action: Netlist(Add,Encoder1(Q1)
=C2=A0 unknown action `,U1-81)'
=C2=A0 Action: Netlist(Add,Encoder1(Q1)
=C2=A0 Segmentation fault


I intended to figure out where crash happen and hopefully solve it.


Nicklas Karlsson
--047d7b417f953b7ad7052be2959f--