Re: [Kicad-developers] Another performance patch....

2018-02-10 Thread Seth Hillbrand
As a stop-gap for 5, could we create and delete a dummy file when saving
footprints?  That should trigger the UNIX time update for directories.  Or
would that approach be too hack-y?

-S

2018-02-10 17:49 GMT-08:00 Wayne Stambaugh :

> Jeff,
>
> This patch wont work.  It ignores changes to the footprint files already
> in the path.  You still have to check for a library path change and each
> individual file in the library path for changes.  The patch I just
> committed only fixed the issue when adding a new footprint file to an
> existing footprint library path.
>
> As for using wxFileSystemWatcher, I am not opposed to this but not for
> v5.  This will require a lot of testing on all platforms so I'm going to
> defer this for v6.
>
> Cheers,
>
> Wayne
>
>
> On 02/10/2018 05:36 PM, Jeff Young wrote:
>
>>
>>
>> On 10 Feb 2018, at 22:34, Jeff Young  wrote:
>>>
>>> Hi Wayne,
>>>
>>> Yep, just a minute.  (I suggested you back your other change out first
>>> in the bug report, but don’t do that now as my rebase will be on top of
>>> your merge.)
>>>
>>> Cheers,
>>> Jeff.
>>>
>>>
>>> On 10 Feb 2018, at 22:28, Wayne Stambaugh  wrote:

 Jeff,

 Would you please rebase the performance patch so I can use `git am` to
 merge it.

 Thanks,

 Wayne

 On 02/08/2018 07:35 PM, Jeff Young wrote:

> Ping.
> Any thoughts on patching wxWidgets for other platforms?
>
>> On 6 Feb 2018, at 14:08, Jeff Young  j...@rokeby.ie>> wrote:
>>
>> I accidentally dropped the dev list off the last few replies.  They
>> included an update which fixed the issue Seth hypothesised.
>>
>> But I’m updating it one more time because, well, I made it faster
>> again.
>>
>> There are two patches this time: the updated Kicad patch which knocks
>> yet another 1/4 second off both first- and subsequent-times, and a
>> wxWidgets patch which when combined with the Kicad patch brings the
>> subsequent-time to near-instantaneous.
>>
>> Since we maintain our own Mac wxWidgets there’s no reason not to get
>> the full benefit there.  I’ll leave it up to others to decide whether to
>> include the wxWidgets patch for other platforms.
>>
>> (Note: while the two patches are dependent on each other /for the
>> final /performance gain, they are /not/ dependent on each other for
>> compiling/linking/running.)
>>
>> Cheers,
>> Jeff.
>>
>>
>>
>> On 5 Feb 2018, at 22:48, Jeff Young > j...@rokeby.ie>> wrote:
>>>
>>> I think I must be having a thick moment, because I’m still not
>>> following you.
>>>
>>> If I create a new library in the Footprint Editor and save it to a
>>> new location, then those components won’t be immediately available in 
>>> Place
>>> Footprint until the library location is added to the Footprint Library
>>> Table, right?
>>>
>>> Or am I completely missing something?
>>>
>>> Cheers,
>>> Jeff.
>>>
>>> On 5 Feb 2018, at 22:17, Wayne Stambaugh > wrote:

 On 02/05/2018 05:08 PM, Jeff Young wrote:

> Hi Wayne,
> Do you mean if I use a text editor to modify a module file while
> KiCad is running, will KiCad notice and reload it?  The answer is 
> yes.  The
> checksum is generated as a hash of the disk directory last-modified 
> dates;
> the lib-table only tells it what the current libraries are.
>

 A text editor is one use case but the footprint library editor can
 also save an entire library file without going through the 
 fp-lib-table so
 it's important that we don't break this behavior.  I may have missed 
 this
 as I am sitting in Montreal airport after missing my connection with no
 sleep, a nasty case of jet lag, and an inbox that is sprialing out of
 control so I may have just overlooked it.

 Nice talk at FOSDEM, by the way.  Do we all get t-shirts? ;)
>

 Maybe one of these days I'll get around to some KiCad apparel of
 some type to give out to the dev team.  Although I don't think you 
 should
 depend on my lack of graphics skills to design anything. :)

 Cheers,

 Wayne


 Cheers,
> Jeff.
> PS: updated patch attached to fix the issue Seth discovered
> (hypothesised?).
>
>> On 5 Feb 2018, at 21:59, Wayne Stambaugh > > wrote:
>>
>> What happens when footprint library file is modified outside the
>> fp-lib-table?  At one point you could change the footprint 

[Kicad-developers] Improving Eagle Import netlist matching

2018-02-10 Thread Russell Oliver
Hi All,

I've discovered the cause of a problem when importing Eagle Projects and
getting the schematic and boards synced.

Currently when importing an Eagle schematic, labels for nets that are only
found one Eagle sheet are imported as local KiCad labels. This preserves
the visual design of the schematic some what. For eagle schematics with
more than 1 sheet, where hierarchical sheets are created in Kicad, global
labels are created to tie the nets together across the sheets the same as
Eagle due to its lack of scopes for net names.

The problem is that the imported PCB will have net names that are global
e.g "VBUS" while the imported schematic may end up with local netname for
the root sheet e.g "/VBUS". This will cause errors for boards with zones
and named vias with the original/global netname e.g."VBUS"

My proposal to fix this is to create another pass in the netlist generation
code that would remove the forward slash '/' for unique local nets in the
root sheet provided it does not clash with an existing net name.

Kind Regards
Russell


P.S During debugging this issue, I discovered that a local label and global
label of the same name on the same sheet are connected regardless of any
wires. Which if there is a hierarchical sheet can lead to the same net for
2 wire segments on separate sheets connected only to local labels, if the
identical global label is somewhere else on both sheets. Is this the
expected behaviour? or just a side effect?
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


[Kicad-developers] [PATCH] Eagle schematic import, label placement fix.

2018-02-10 Thread Russell Oliver
This fixes an issue i found during some testing. Small labels are added to
connect segments sharing the same net name in the eagle schematic. These
were placed in the middle of a wire, but other wires may cross at this
point without being connected by a junction. When the netlist is created
the label will connect these wires anyway leading to a incorrect netlist.
These labels are now added at a wire end where existing connection rules
apply.

Russell


0001-Eagle-Schematic-Import-Label-Placement-fix.patch
Description: Binary data
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] Another performance patch....

2018-02-10 Thread Wayne Stambaugh

Jeff,

This patch wont work.  It ignores changes to the footprint files already 
in the path.  You still have to check for a library path change and each 
individual file in the library path for changes.  The patch I just 
committed only fixed the issue when adding a new footprint file to an 
existing footprint library path.


As for using wxFileSystemWatcher, I am not opposed to this but not for 
v5.  This will require a lot of testing on all platforms so I'm going to 
defer this for v6.


Cheers,

Wayne

On 02/10/2018 05:36 PM, Jeff Young wrote:




On 10 Feb 2018, at 22:34, Jeff Young  wrote:

Hi Wayne,

Yep, just a minute.  (I suggested you back your other change out first in the 
bug report, but don’t do that now as my rebase will be on top of your merge.)

Cheers,
Jeff.



On 10 Feb 2018, at 22:28, Wayne Stambaugh  wrote:

Jeff,

Would you please rebase the performance patch so I can use `git am` to merge it.

Thanks,

Wayne

On 02/08/2018 07:35 PM, Jeff Young wrote:

Ping.
Any thoughts on patching wxWidgets for other platforms?

On 6 Feb 2018, at 14:08, Jeff Young > 
wrote:

I accidentally dropped the dev list off the last few replies.  They included an 
update which fixed the issue Seth hypothesised.

But I’m updating it one more time because, well, I made it faster again.

There are two patches this time: the updated Kicad patch which knocks yet 
another 1/4 second off both first- and subsequent-times, and a wxWidgets patch 
which when combined with the Kicad patch brings the subsequent-time to 
near-instantaneous.

Since we maintain our own Mac wxWidgets there’s no reason not to get the full 
benefit there.  I’ll leave it up to others to decide whether to include the 
wxWidgets patch for other platforms.

(Note: while the two patches are dependent on each other /for the final 
/performance gain, they are /not/ dependent on each other for 
compiling/linking/running.)

Cheers,
Jeff.




On 5 Feb 2018, at 22:48, Jeff Young > 
wrote:

I think I must be having a thick moment, because I’m still not following you.

If I create a new library in the Footprint Editor and save it to a new 
location, then those components won’t be immediately available in Place 
Footprint until the library location is added to the Footprint Library Table, 
right?

Or am I completely missing something?

Cheers,
Jeff.


On 5 Feb 2018, at 22:17, Wayne Stambaugh > wrote:

On 02/05/2018 05:08 PM, Jeff Young wrote:

Hi Wayne,
Do you mean if I use a text editor to modify a module file while KiCad is 
running, will KiCad notice and reload it?  The answer is yes.  The checksum is 
generated as a hash of the disk directory last-modified dates; the lib-table 
only tells it what the current libraries are.


A text editor is one use case but the footprint library editor can also save an 
entire library file without going through the fp-lib-table so it's important 
that we don't break this behavior.  I may have missed this as I am sitting in 
Montreal airport after missing my connection with no sleep, a nasty case of jet 
lag, and an inbox that is sprialing out of control so I may have just 
overlooked it.


Nice talk at FOSDEM, by the way.  Do we all get t-shirts? ;)


Maybe one of these days I'll get around to some KiCad apparel of some type to 
give out to the dev team.  Although I don't think you should depend on my lack 
of graphics skills to design anything. :)

Cheers,

Wayne



Cheers,
Jeff.
PS: updated patch attached to fix the issue Seth discovered (hypothesised?).

On 5 Feb 2018, at 21:59, Wayne Stambaugh > wrote:

What happens when footprint library file is modified outside the fp-lib-table?  
At one point you could change the footprint library file without performing the 
file write through the fp-lib-table and the next time you accessed the library, 
it would recognize the file was modified and reload the cache.  Please make 
sure this behavior is not broken.

On 02/05/2018 04:51 PM, Jeff Young wrote:

wxWidgets should return Now() which will make the checksums not match and 
trigger a reload.
Of course what actually happens is that wxWidgets asserts. ;)
New patch on the way….
Cheers,
Jeff.

On 5 Feb 2018, at 21:42, Seth Hillbrand  > wrote:

Jeff-

Will removing a footprint library trigger a refresh?  Or will that mess up the 
checksum calculation?

-S

2018-02-05 13:04 GMT-08:00 Jeff Young  
>:

  This one for Place Footprint (specifically the List All dialog
  found therein).

  On my machine it knocks the first-use time from 4s to 3s and the
  subsequent-use time from 2.5s to 1s.

  Cheers,
  Jeff.


  ___
  

[Kicad-developers] Command Window for Scripting

2018-02-10 Thread Andrey Kuznetsov
Hi,

I was wondering if there were ideas to add command window for scripting
inside schematic and pcbnew for v6?

For example, in Allegro you can create functions/aliases that when typed
into command line while inside layout tool would execute an for example
change grid type from 0.1mm to 1mm or turn on a predetermined set of
layers. This is for professionals who automate a lot of their actions to as
few actions as possible while working on large projects and need the
ability to reduce a dozen common steps into one.

What kind of scripting interface would it be?

-- 
Remember The Past, Live The Present, Change The Future
Those who look only to the past or the present are certain to miss the
future [JFK]

kandre...@gmail.com
Live Long and Prosper,
Andrey
___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] [PATCH] handle LoadBoard IOError exception in python scripts

2018-02-10 Thread Wayne Stambaugh

Hi Damien,

I merged your patch into the development branch.  Thank you for your 
contribution to KiCad.


Cheers,

Wayne

On 02/10/2018 07:49 AM, Damien Espitallier wrote:

Hi,

Loading a wrong file or an non-existing file with the LoadBoard python 
function kill the program with the following message:


"""
terminate called after throwing an instance of 'IO_ERROR'
Abandon (core dumped)
"""

The attached patch enable exception handling for the LoadBoard function.

Damien Espitallier


___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp



___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] Another performance patch....

2018-02-10 Thread Jeff Young
Hi Wayne,

Yep, just a minute.  (I suggested you back your other change out first in the 
bug report, but don’t do that now as my rebase will be on top of your merge.)

Cheers,
Jeff.


> On 10 Feb 2018, at 22:28, Wayne Stambaugh  wrote:
> 
> Jeff,
> 
> Would you please rebase the performance patch so I can use `git am` to merge 
> it.
> 
> Thanks,
> 
> Wayne
> 
> On 02/08/2018 07:35 PM, Jeff Young wrote:
>> Ping.
>> Any thoughts on patching wxWidgets for other platforms?
>>> On 6 Feb 2018, at 14:08, Jeff Young >> > wrote:
>>> 
>>> I accidentally dropped the dev list off the last few replies.  They 
>>> included an update which fixed the issue Seth hypothesised.
>>> 
>>> But I’m updating it one more time because, well, I made it faster again.
>>> 
>>> There are two patches this time: the updated Kicad patch which knocks yet 
>>> another 1/4 second off both first- and subsequent-times, and a wxWidgets 
>>> patch which when combined with the Kicad patch brings the subsequent-time 
>>> to near-instantaneous.
>>> 
>>> Since we maintain our own Mac wxWidgets there’s no reason not to get the 
>>> full benefit there.  I’ll leave it up to others to decide whether to 
>>> include the wxWidgets patch for other platforms.
>>> 
>>> (Note: while the two patches are dependent on each other /for the final 
>>> /performance gain, they are /not/ dependent on each other for 
>>> compiling/linking/running.)
>>> 
>>> Cheers,
>>> Jeff.
>>> 
>>> 
>>> 
 On 5 Feb 2018, at 22:48, Jeff Young > wrote:
 
 I think I must be having a thick moment, because I’m still not following 
 you.
 
 If I create a new library in the Footprint Editor and save it to a new 
 location, then those components won’t be immediately available in Place 
 Footprint until the library location is added to the Footprint Library 
 Table, right?
 
 Or am I completely missing something?
 
 Cheers,
 Jeff.
 
> On 5 Feb 2018, at 22:17, Wayne Stambaugh  > wrote:
> 
> On 02/05/2018 05:08 PM, Jeff Young wrote:
>> Hi Wayne,
>> Do you mean if I use a text editor to modify a module file while KiCad 
>> is running, will KiCad notice and reload it?  The answer is yes.  The 
>> checksum is generated as a hash of the disk directory last-modified 
>> dates; the lib-table only tells it what the current libraries are.
> 
> A text editor is one use case but the footprint library editor can also 
> save an entire library file without going through the fp-lib-table so 
> it's important that we don't break this behavior.  I may have missed this 
> as I am sitting in Montreal airport after missing my connection with no 
> sleep, a nasty case of jet lag, and an inbox that is sprialing out of 
> control so I may have just overlooked it.
> 
>> Nice talk at FOSDEM, by the way.  Do we all get t-shirts? ;)
> 
> Maybe one of these days I'll get around to some KiCad apparel of some 
> type to give out to the dev team.  Although I don't think you should 
> depend on my lack of graphics skills to design anything. :)
> 
> Cheers,
> 
> Wayne
> 
> 
>> Cheers,
>> Jeff.
>> PS: updated patch attached to fix the issue Seth discovered 
>> (hypothesised?).
>>> On 5 Feb 2018, at 21:59, Wayne Stambaugh >> > wrote:
>>> 
>>> What happens when footprint library file is modified outside the 
>>> fp-lib-table?  At one point you could change the footprint library file 
>>> without performing the file write through the fp-lib-table and the next 
>>> time you accessed the library, it would recognize the file was modified 
>>> and reload the cache.  Please make sure this behavior is not broken.
>>> 
>>> On 02/05/2018 04:51 PM, Jeff Young wrote:
 wxWidgets should return Now() which will make the checksums not match 
 and trigger a reload.
 Of course what actually happens is that wxWidgets asserts. ;)
 New patch on the way….
 Cheers,
 Jeff.
> On 5 Feb 2018, at 21:42, Seth Hillbrand   > 
> wrote:
> 
> Jeff-
> 
> Will removing a footprint library trigger a refresh?  Or will that 
> mess up the checksum calculation?
> 
> -S
> 
> 2018-02-05 13:04 GMT-08:00 Jeff Young   >:
> 
>   This one for Place Footprint (specifically the List All dialog
>   found therein).
> 
>   On my machine it knocks the first-use time from 4s to 3s and the
>   

Re: [Kicad-developers] Another performance patch....

2018-02-10 Thread Wayne Stambaugh

Jeff,

Would you please rebase the performance patch so I can use `git am` to 
merge it.


Thanks,

Wayne

On 02/08/2018 07:35 PM, Jeff Young wrote:

Ping.

Any thoughts on patching wxWidgets for other platforms?

On 6 Feb 2018, at 14:08, Jeff Young > wrote:


I accidentally dropped the dev list off the last few replies.  They 
included an update which fixed the issue Seth hypothesised.


But I’m updating it one more time because, well, I made it faster again.

There are two patches this time: the updated Kicad patch which knocks 
yet another 1/4 second off both first- and subsequent-times, and a 
wxWidgets patch which when combined with the Kicad patch brings the 
subsequent-time to near-instantaneous.


Since we maintain our own Mac wxWidgets there’s no reason not to get 
the full benefit there.  I’ll leave it up to others to decide whether 
to include the wxWidgets patch for other platforms.


(Note: while the two patches are dependent on each other /for the 
final /performance gain, they are /not/ dependent on each other for 
compiling/linking/running.)


Cheers,
Jeff.









On 5 Feb 2018, at 22:48, Jeff Young > wrote:


I think I must be having a thick moment, because I’m still not 
following you.


If I create a new library in the Footprint Editor and save it to a 
new location, then those components won’t be immediately available in 
Place Footprint until the library location is added to the Footprint 
Library Table, right?


Or am I completely missing something?

Cheers,
Jeff.

On 5 Feb 2018, at 22:17, Wayne Stambaugh > wrote:


On 02/05/2018 05:08 PM, Jeff Young wrote:

Hi Wayne,
Do you mean if I use a text editor to modify a module file while 
KiCad is running, will KiCad notice and reload it?  The answer is 
yes.  The checksum is generated as a hash of the disk directory 
last-modified dates; the lib-table only tells it what the current 
libraries are.


A text editor is one use case but the footprint library editor can 
also save an entire library file without going through the 
fp-lib-table so it's important that we don't break this behavior.  I 
may have missed this as I am sitting in Montreal airport after 
missing my connection with no sleep, a nasty case of jet lag, and an 
inbox that is sprialing out of control so I may have just overlooked it.



Nice talk at FOSDEM, by the way.  Do we all get t-shirts? ;)


Maybe one of these days I'll get around to some KiCad apparel of 
some type to give out to the dev team.  Although I don't think you 
should depend on my lack of graphics skills to design anything. :)


Cheers,

Wayne



Cheers,
Jeff.
PS: updated patch attached to fix the issue Seth discovered 
(hypothesised?).
On 5 Feb 2018, at 21:59, Wayne Stambaugh > wrote:


What happens when footprint library file is modified outside the 
fp-lib-table?  At one point you could change the footprint library 
file without performing the file write through the fp-lib-table 
and the next time you accessed the library, it would recognize the 
file was modified and reload the cache.  Please make sure this 
behavior is not broken.


On 02/05/2018 04:51 PM, Jeff Young wrote:
wxWidgets should return Now() which will make the checksums not 
match and trigger a reload.

Of course what actually happens is that wxWidgets asserts. ;)
New patch on the way….
Cheers,
Jeff.
On 5 Feb 2018, at 21:42, Seth Hillbrand 
 
> wrote:


Jeff-

Will removing a footprint library trigger a refresh?  Or will 
that mess up the checksum calculation?


-S

2018-02-05 13:04 GMT-08:00 Jeff Young  >:


  This one for Place Footprint (specifically the List All dialog
  found therein).

  On my machine it knocks the first-use time from 4s to 3s and the
  subsequent-use time from 2.5s to 1s.

  Cheers,
  Jeff.


  ___
  Mailing list: https://launchpad.net/~kicad-developers
  
  Post to : kicad-developers@lists.launchpad.net 


  
  Unsubscribe : https://launchpad.net/~kicad-developers
  
  More help   : https://help.launchpad.net/ListHelp
  


___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net 
 


Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

___
Mailing 

Re: [Kicad-developers] [PATCH] Add cross-probing messages to zoom in, zoom out and zoom to one of the preset zoom levels.

2018-02-10 Thread Wayne Stambaugh

Tom,

Whet is the purpose of this patch?  I don't see where you are sending 
the zoom messages from the schematic editor or are you doing this from 
you own application?


Cheers,

Wayne

On 02/08/2018 10:51 AM, Tom Cook wrote:
This is something I find useful when integrating pcbnew into my own 
workflows; it means that as well as focusing on a component using KiWay 
commands, you can set the zoom level to something other than the default 
this-component-only level.


Regards,
Tom


___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp



___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] [PATCH] Allow OpenCASCADE standard edition

2018-02-10 Thread Seth Hillbrand
Nick was having an unknown issue but I haven't heard back from him on
whether the issue was resolved by doing a clean cmake.  If it isn't then
I'll need to address that.

Nick did you have a chance to test that?

-S

2018-02-10 6:18 GMT-08:00 Wayne Stambaugh :

> Seth,
>
> What is the current status of this patch?  I would like to get it merged
> before rc1 so the package devs can test it.
>
> Cheers,
>
> Wayne
>
> On 01/30/2018 02:12 PM, Seth Hillbrand wrote:
> > Nick-
> >
> > Thanks for the test.  I'm attaching revised patch that allows multiple
> > OpenCASCADE installations on a single machine.
> >
> > I've tested with a few different OpenCASCADE versions down to 6.8 with
> > no issues installed alongside OCE 0.17.  You can choose which you want
> > to link by using either "-DKICAD_USE_OCE" or "-DKICAD_USE_OCC".  If both
> > are specified, "-DKICAD_USE_OCC" will override.
> >
> > I corrected the cmake message display to be less doubled and correctly
> > show the library location.
> >
> > I've also added the OCC version and type to the about window version
> info.
> >
> > The "Based on" line was taken from
> > (https://github.com/FreeCAD/FreeCAD/blob/master/src/FCConfig.h) when
> > trying to determine how FreeCAD likes to refer to themselves.  The
> > actual FindOpenCascade.cmake file did not have a copyright header
> > attached but falls under the license from
> > (https://github.com/FreeCAD/FreeCAD/blob/master/COPYING).  I note that I
> > had it written as "FreeCAD CADx development system".  I've corrected
> > this to read "FreeCAD CAx development system".
> >
> > I'm not sure to what the "CheckSymbolExists" line is referring.  I don't
> > see it on my machine.
> >
> > -Seth
> >
> > 2018-01-29 14:30 GMT-08:00 Nick Østergaard  > >:
> >
> > Hi Seth,
> >
> > I just took the patch for a testrun and will state some comments
> below.
> >
> > This looks a bit strange:
> >
> > -- Boost version: 1.66.0
> > -- -- OpenCASCADE Community Edition has been found.
> > -- -- Found OCE/OpenCASCADE version: 6.8.0
> > -- -- OCE/OpenCASCADE include directory:
> > /opt/oce/lib/oce-0.17/../../include/oce
> > -- -- OCE/OpenCASCADE shared libraries directory:
> > -- Check for installed Python Interpreter -- found
> >
> > The messages are with double -- and the shared libs.
> >
> > But an improvement with your patch over what is currently in kicad
> > is that it found OCE on my system without explicitly specifind
> > OCE_DIR. But how do I make it use OCCT when I also have OCE
> installed?
> >
> > What does CADx mean in that header?
> >
> > What is this about?
> > -- Looking for
> > /home/amazingdude/kicad-source-mirror/build_seths_
> occt_patch_occt/CMakeFiles/CMakeTmp/CheckSymbolExists.cxx
> > -- Looking for
> > /home/amazingdude/kicad-source-mirror/build_seths_
> occt_patch_occt/CMakeFiles/CMakeTmp/CheckSymbolExists.cxx
> > - not found
> >
> > I tried to test it with removing oce and just have occt installed
> > and got something like this:
> > -- Found OCC: /opt/opencascade/inc (found version "6.9.1")
> > -- -- Found OCE/OpenCASCADE version: 6.9.1
> > -- -- OCE/OpenCASCADE include directory: /opt/opencascade/inc
> > -- -- OCE/OpenCASCADE shared libraries directory:
> /opt/opencascade/lib
> >
> > It did not build against community/opencascade 6.9.1-7. I got the
> > following error in a clean build dir.
> >
> > $ make kicad2step -j1
> > [  0%] Linking CXX executable kicad2step
> > /usr/bin/ld: cannot find -lTKMesh
> > /usr/bin/ld: cannot find -lTKernel
> > /usr/bin/ld: cannot find -lTKG2d
> > /usr/bin/ld: cannot find -lTKG3d
> > /usr/bin/ld: cannot find -lTKMath
> > /usr/bin/ld: cannot find -lTKIGES
> > /usr/bin/ld: cannot find -lTKSTL
> > /usr/bin/ld: cannot find -lTKXSBase
> > /usr/bin/ld: cannot find -lTKBin
> > /usr/bin/ld: cannot find -lTKBO
> > /usr/bin/ld: cannot find -lTKCDF
> > /usr/bin/ld: cannot find -lTKBRep
> > /usr/bin/ld: cannot find -lTKTopAlgo
> > /usr/bin/ld: cannot find -lTKGeomAlgo
> > /usr/bin/ld: cannot find -lTKGeomBase
> > /usr/bin/ld: cannot find -lTKPrim
> > /usr/bin/ld: cannot find -lTKSTEP
> > /usr/bin/ld: cannot find -lTKSTEPBase
> > /usr/bin/ld: cannot find -lTKSTEPAttr
> > /usr/bin/ld: cannot find -lTKFeat
> > /usr/bin/ld: cannot find -lTKCAF
> > /usr/bin/ld: cannot find -lTKXCAF
> > /usr/bin/ld: cannot find -lTKLCAF
> > /usr/bin/ld: cannot find -lTKXDESTEP
> > /usr/bin/ld: cannot find -lTKXDEIGES
> > collect2: error: ld returned 1 exit status
> > make[3]: ***
> > [utils/kicad2step/CMakeFiles/kicad2step.dir/build.make:355:
> > utils/kicad2step/kicad2step] Error 1
> > make[2]: *** [CMakeFiles/Makefile2:3007:
> > utils/kicad2step/CMakeFiles/kicad2step.dir/all] Error 2
> > make[1]: 

Re: [Kicad-developers] [PATCH] Allow OpenCASCADE standard edition

2018-02-10 Thread Wayne Stambaugh
Seth,

What is the current status of this patch?  I would like to get it merged
before rc1 so the package devs can test it.

Cheers,

Wayne

On 01/30/2018 02:12 PM, Seth Hillbrand wrote:
> Nick-
> 
> Thanks for the test.  I'm attaching revised patch that allows multiple
> OpenCASCADE installations on a single machine.
> 
> I've tested with a few different OpenCASCADE versions down to 6.8 with
> no issues installed alongside OCE 0.17.  You can choose which you want
> to link by using either "-DKICAD_USE_OCE" or "-DKICAD_USE_OCC".  If both
> are specified, "-DKICAD_USE_OCC" will override.  
> 
> I corrected the cmake message display to be less doubled and correctly
> show the library location.
> 
> I've also added the OCC version and type to the about window version info.
> 
> The "Based on" line was taken from
> (https://github.com/FreeCAD/FreeCAD/blob/master/src/FCConfig.h) when
> trying to determine how FreeCAD likes to refer to themselves.  The
> actual FindOpenCascade.cmake file did not have a copyright header
> attached but falls under the license from
> (https://github.com/FreeCAD/FreeCAD/blob/master/COPYING).  I note that I
> had it written as "FreeCAD CADx development system".  I've corrected
> this to read "FreeCAD CAx development system".
> 
> I'm not sure to what the "CheckSymbolExists" line is referring.  I don't
> see it on my machine.
> 
> -Seth
> 
> 2018-01-29 14:30 GMT-08:00 Nick Østergaard  >:
> 
> Hi Seth,
> 
> I just took the patch for a testrun and will state some comments below.
> 
> This looks a bit strange:
> 
> -- Boost version: 1.66.0
> -- -- OpenCASCADE Community Edition has been found.
> -- -- Found OCE/OpenCASCADE version: 6.8.0
> -- -- OCE/OpenCASCADE include directory:
> /opt/oce/lib/oce-0.17/../../include/oce
> -- -- OCE/OpenCASCADE shared libraries directory:
> -- Check for installed Python Interpreter -- found
> 
> The messages are with double -- and the shared libs.
> 
> But an improvement with your patch over what is currently in kicad
> is that it found OCE on my system without explicitly specifind
> OCE_DIR. But how do I make it use OCCT when I also have OCE installed?
> 
> What does CADx mean in that header?
> 
> What is this about?
> -- Looking for
> 
> /home/amazingdude/kicad-source-mirror/build_seths_occt_patch_occt/CMakeFiles/CMakeTmp/CheckSymbolExists.cxx
> -- Looking for
> 
> /home/amazingdude/kicad-source-mirror/build_seths_occt_patch_occt/CMakeFiles/CMakeTmp/CheckSymbolExists.cxx
> - not found
> 
> I tried to test it with removing oce and just have occt installed
> and got something like this:
> -- Found OCC: /opt/opencascade/inc (found version "6.9.1")
> -- -- Found OCE/OpenCASCADE version: 6.9.1
> -- -- OCE/OpenCASCADE include directory: /opt/opencascade/inc
> -- -- OCE/OpenCASCADE shared libraries directory: /opt/opencascade/lib
> 
> It did not build against community/opencascade 6.9.1-7. I got the
> following error in a clean build dir.
> 
> $ make kicad2step -j1
> [  0%] Linking CXX executable kicad2step
> /usr/bin/ld: cannot find -lTKMesh
> /usr/bin/ld: cannot find -lTKernel
> /usr/bin/ld: cannot find -lTKG2d
> /usr/bin/ld: cannot find -lTKG3d
> /usr/bin/ld: cannot find -lTKMath
> /usr/bin/ld: cannot find -lTKIGES
> /usr/bin/ld: cannot find -lTKSTL
> /usr/bin/ld: cannot find -lTKXSBase
> /usr/bin/ld: cannot find -lTKBin
> /usr/bin/ld: cannot find -lTKBO
> /usr/bin/ld: cannot find -lTKCDF
> /usr/bin/ld: cannot find -lTKBRep
> /usr/bin/ld: cannot find -lTKTopAlgo
> /usr/bin/ld: cannot find -lTKGeomAlgo
> /usr/bin/ld: cannot find -lTKGeomBase
> /usr/bin/ld: cannot find -lTKPrim
> /usr/bin/ld: cannot find -lTKSTEP
> /usr/bin/ld: cannot find -lTKSTEPBase
> /usr/bin/ld: cannot find -lTKSTEPAttr
> /usr/bin/ld: cannot find -lTKFeat
> /usr/bin/ld: cannot find -lTKCAF
> /usr/bin/ld: cannot find -lTKXCAF
> /usr/bin/ld: cannot find -lTKLCAF
> /usr/bin/ld: cannot find -lTKXDESTEP
> /usr/bin/ld: cannot find -lTKXDEIGES
> collect2: error: ld returned 1 exit status
> make[3]: ***
> [utils/kicad2step/CMakeFiles/kicad2step.dir/build.make:355:
> utils/kicad2step/kicad2step] Error 1
> make[2]: *** [CMakeFiles/Makefile2:3007:
> utils/kicad2step/CMakeFiles/kicad2step.dir/all] Error 2
> make[1]: *** [CMakeFiles/Makefile2:3019:
> utils/kicad2step/CMakeFiles/kicad2step.dir/rule] Error 2
> make: *** [Makefile:979: kicad2step] Error 2
> 
> But those libs do existm, searched for the last one;
> $ yaourt  -Ql opencascade | grep TKXDEIGES
> opencascade /opt/opencascade/lib/libTKXDEIGES.so
> opencascade /opt/opencascade/lib/libTKXDEIGES.so.0
> opencascade /opt/opencascade/lib/libTKXDEIGES.so.0.0.0
> 
> I got these variables set in the CMakeCache
> 
> cat 

[Kicad-developers] [PATCH] handle LoadBoard IOError exception in python scripts

2018-02-10 Thread Damien Espitallier
Hi,

Loading a wrong file or an non-existing file with the LoadBoard python
function kill the program with the following message:

"""
terminate called after throwing an instance of 'IO_ERROR'
Abandon (core dumped)
"""

The attached patch enable exception handling for the LoadBoard function.

Damien Espitallier
From cc6ca8af7f681c043ae6e60f2ef27dabe131ce4c Mon Sep 17 00:00:00 2001
From: Damien Espitallier 
Date: Sat, 10 Feb 2018 13:46:54 +0100
Subject: [PATCH] Enable exception handler for python LoadBoard function
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="2.16.1"

This is a multi-part message in MIME format.
--2.16.1
Content-Type: text/plain; charset=UTF-8; format=fixed
Content-Transfer-Encoding: 8bit

---
 pcbnew/swig/pcbnew.i | 1 +
 1 file changed, 1 insertion(+)


--2.16.1
Content-Type: text/x-patch; name="0001-Enable-exception-handler-for-python-LoadBoard-functi.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment; filename="0001-Enable-exception-handler-for-python-LoadBoard-functi.patch"

diff --git a/pcbnew/swig/pcbnew.i b/pcbnew/swig/pcbnew.i
index 53c089168..f88b5b9bf 100644
--- a/pcbnew/swig/pcbnew.i
+++ b/pcbnew/swig/pcbnew.i
@@ -115,6 +115,7 @@ HANDLE_EXCEPTIONS(PLUGIN::FootprintDelete)
 %include 
 %include 
 
+HANDLE_EXCEPTIONS(LoadBoard)
 %include 
 
 

--2.16.1--


___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] New OpenGL debug stuff crashes pcbnew on startup

2018-02-10 Thread jp charras
Le 10/02/2018 à 10:54, jp charras a écrit :
> Le 09/02/2018 à 20:08, Wayne Stambaugh a écrit :
>> Linux and windows are find for me.
>>
>> On 02/09/2018 01:14 PM, Jeff Young wrote:
>>> Mac-only?
> 
> Mac-only?: No. For me, it crashes on Windows (W7 32 bits) and works only on 
> Linux.
> 
> The crash happens both in debug and release mode, and in debug mode I have 
> the same trace as on OSX.

Hi Orson,

Could you have a look into my commit 5ecfd326545c0bc34c853174e4de371a8bc0ac82

It allows call to enableGlDebug only in Debug Build ( __WXDEBUG__ is defined 
even in Kicad release
mode) and only on Linux.

For some reasons, on my W7 32 bits install, glEnable( GL_DEBUG_OUTPUT ) crash 
without message,
and glDisable( GL_DEBUG_OUTPUT ) crash but after displaying a error message.

Thanks,

-- 
Jean-Pierre CHARRAS

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] New OpenGL debug stuff crashes pcbnew on startup

2018-02-10 Thread jp charras
Le 09/02/2018 à 20:08, Wayne Stambaugh a écrit :
> Linux and windows are find for me.
> 
> On 02/09/2018 01:14 PM, Jeff Young wrote:
>> Mac-only?

Mac-only?: No. For me, it crashes on Windows (W7 32 bits) and works only on 
Linux.

The crash happens both in debug and release mode, and in debug mode I have the 
same trace as on OSX.


-- 
Jean-Pierre CHARRAS

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp