kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #33886
Re: [PATCH] Add empty eeschema page layout for Eagle schematic import. Bug #1729722
I prefer adding an empty.kicad_wks file be added to the project and then
set the board and schematic worksheets to this file. This would negate
the file format rev bumps.
On 2/12/2018 10:48 AM, Maciej Sumiński wrote:
> Hi Russell,
>
> You are right, it adds a special case for project file format, so
> technically it is a version bump. Alternatively one could simply create
> "empty.kicad_wks" and store there the contents of emptyPageLayout[]. It
> seems safer, but on the other hand I would like to be able to specify
> that design uses an empty worksheet layout.
>
> Wayne, what do you think? Is it acceptable that we add a reserved
> keyword "empty" to indicate an empty worksheet layout for a project? If
> so, we need to implement it both for pcbnew and eeschema.
>
> Regards,
> Orson
>
> On 02/11/2018 12:24 AM, Russell Oliver wrote:
>> Hi Orson and Wayne
>>
>> I have updated the patch from your changes Orson, so that the change is
>> saved using the project file settings, by setting the schematic layout file
>> path as "empty" eg
>> PageLayoutDescrFile=empty
>>
>> It is technically a file format change, but it also provides the option to
>> those that do not want a border to specify it for a schematic.
>>
>> Kind Regards
>> Russell
>>
>>
>> On Thu, Feb 8, 2018 at 10:25 PM Maciej Sumiński <maciej.suminski@xxxxxxx>
>> wrote:
>>
>>> There is still one problem to be solved here: worksheet layout is not
>>> saved in schematic file, so the default worksheet is restored when an
>>> imported project is saved and reloaded.
>>>
>>> I have nothing against the patch, it gives a nicer first impression.
>>>
>>> Cheers,
>>> Orson
>>>
>>> On 02/07/2018 04:44 PM, Wayne Stambaugh wrote:
>>>> Thanks for testing this. I know I'm being paranoid but we've been bit
>>>> by this before. Maybe someday our unit testing will actually get
>>>> implemented.
>>>>
>>>> Wayne
>>>>
>>>> On 2/7/2018 8:21 AM, Russell Oliver wrote:
>>>>> I just tested printing then and it worked fine. plus one person's
>>>>> unhandled edge case is another's unit test.
>>>>>
>>>>>
>>>>>
>>>>> On Thu, Feb 8, 2018 at 12:05 AM Wayne Stambaugh <stambaughw@xxxxxxxxx
>>>>> <mailto:stambaughw@xxxxxxxxx>> wrote:
>>>>>
>>>>> Be careful with zero length line segments. They have been known to
>>>>> cause issues in the past. We recently fixed a print bug where a
>>> zero
>>>>> diameter circle was causing pages not to print.
>>>>>
>>>>> On 2/7/2018 7:45 AM, Russell Oliver wrote:
>>>>> > Hi Orson,
>>>>> >
>>>>> > I'm completely fine with any simplifications and style changes.
>>>>> >
>>>>> > With regards to the zero length line, it appears on line 110 of
>>> your
>>>>> > patch file.
>>>>> > 110: + "(line (name segm1:Line) (start 0 0) (end 0 0))\n"
>>>>> >
>>>>> > JP mentions in a comment to the bug report that there is a legacy
>>>>> > compatibility requirement to have at least one item in the page
>>>>> layout,
>>>>> > otherwise the default layout it used. This was for old schematics
>>> that
>>>>> > do not have a page layout specified.
>>>>> >
>>>>> > Kind Regards
>>>>> > Russell
>>>>> >
>>>>> >
>>>>> > On Wed, Feb 7, 2018 at 12:13 AM Maciej Sumiński
>>>>> <maciej.suminski@xxxxxxx <mailto:maciej.suminski@xxxxxxx>
>>>>> > <mailto:maciej.suminski@xxxxxxx <mailto:maciej.suminski@xxxxxxx
>>>>>>
>>>>> wrote:
>>>>> >
>>>>> > Hi Russell,
>>>>> >
>>>>> > Thank you very much for the patch. It works as expected and I
>>>>> would like
>>>>> > to merge it, but there are two things.
>>>>> >
>>>>> > I have simplified the patch a bit (moved the empty layout to
>>>>> an existing
>>>>> > file, minor code formatting fixes), so please confirm you are
>>>>> ok with
>>>>> > committing it under your name.
>>>>> >
>>>>> > Another question is about "there is a 0 length line to fool
>>>>> something
>>>>> > somewhere." comment for const char emptyLayout[]. Could you
>>> say
>>>>> > something more about it? I could not spot a 0 length line in
>>>>> the layout
>>>>> > description, so perhaps we can remove it to avoid confusion.
>>>>> >
>>>>> > Regards,
>>>>> > Orson
>>>>> >
>>>>> > On 02/03/2018 01:27 AM, Russell Oliver wrote:
>>>>> > > Attached is a patch that adds an empty layout using the same
>>>>> > method as the
>>>>> > > SetDefaultLayout function, which is then called by the Eagle
>>>>> schematic
>>>>> > > plugin to leave only the imported frame visible.
>>>>> > >
>>>>> > > https://bugs.launchpad.net/kicad/+bug/1729722
>>>>> > >
>>>>> > > Kind Regards
>>>>> > > Russell
>>>>> > >
>>>>> > >
>>>>> > >
>>>>> > > _______________________________________________
>>>>> > > Mailing list: https://launchpad.net/~kicad-developers
>>>>> > > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>>>>> > > Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> > > More help : https://help.launchpad.net/ListHelp
>>>>> > >
>>>>> >
>>>>> > _______________________________________________
>>>>> > Mailing list: https://launchpad.net/~kicad-developers
>>>>> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>>>>> > Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> > More help : https://help.launchpad.net/ListHelp
>>>>> >
>>>>> >
>>>>> >
>>>>> > _______________________________________________
>>>>> > Mailing list: https://launchpad.net/~kicad-developers
>>>>> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>> > Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> > More help : https://help.launchpad.net/ListHelp
>>>>> >
>>>>>
>>>>> _______________________________________________
>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> More help : https://help.launchpad.net/ListHelp
>>>>>
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help : https://help.launchpad.net/ListHelp
>>>>
>>>
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help : https://help.launchpad.net/Lis
>>> <https://help.launchpad.net/ListHelp>t Help n
>>>
>>
>
>
Follow ups
References