← Back to team overview

kicad-developers team mailing list archive

Re: 3D refactor

 

2016-02-25 22:41 GMT+01:00 Cirilo Bernardo <cirilo.bernardo@xxxxxxxxx>:
>
>
> On Fri, Feb 26, 2016 at 1:07 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx>
> wrote:
>>
>> I found KISYS3DMOD was set to the path where KiCad was installed but I
>> don't think I set it.  Does the windows installer set the KISYSMOD and
>> KISYS3DMOD environment variables for all users?  I deleted the
>> KISYS3DMOD environment variable and Pcbnew did not crash.  However, I
>> still got the warning dialog about the missing model file when Pcbnew
>> was launched.  @Cirilo, would you please change this so that the missing
>> 3D model warning dialog is displayed when the 3D viewer is launched
>> rather than when Pcbnew is launched.  I'm guessing this will annoy users.
>>
>
> Hi Wayne,  which warning dialog do you mean? The only ones which
> should pop up are related to search paths (duplicate search paths etc)
> and are there to alert the user that they may need to make changes
> to the 3D search path configuration. In the case of the dialog you posted
> (3d-refactor-win64-install-1.png) r6531 includes a modification to prevent
> the automatic inclusion of the Current Working DIrectory in the search
> path so that specific warning should no longer appear on MSWin.
>
>>
>> What is most concerning is that when I ran kicad from gdb, it did not
>> crash so there is something at play here that makes me uncomfortable.  I
>> ran it from the mingw64 version of gdb as well as the mingw64 version of
>> QT Creator and it ran fine in both cases.  The bzr version on the
>> installer says r6528 even though the version string says no-bzr.
>>
>
> I'll do a careful check of the top level CMakeLists.txt file as nickoe
> suggests.

FWIW, I am using bzr directly for the 3d_initial_merge branch, where
as I am using git for the normal nightlies. But I wonder why the
create from bzr revno seems broken. I don't think it has been changed
recently, but I could be wrong. I also just tried to compare your
versions of the stuff with the latest form product and it looks the
same, except for the expected changes for the new plugins. So maybe it
is just a broken CMakeModules/CreateBzrVersionHeader.cmake?

I don't really like to use bzr, so I try to avoid that. It was just
easier to use bzr directly in the PKGBUILD I use to build the testing
version.

>
> - Cirilo
>
>>
>> On 2/25/2016 4:46 AM, Nick Østergaard wrote:
>> > Hi Wayne
>> >
>> > It does not crash for me. Does this also happen if the KISYS3DMOD env
>> > var is not set on your system?
>> >
>> > The plugins seems to be loaded correctly because I can see the 3D
>> > models in the preview.
>> >
>> > Also, the build is a debug build, you should be able to run the
>> > pcbnew.exe you installed form the the gdb in msys2, if you can
>> > reproduce the issue when running form msys2.
>> >
>> > 2016-02-24 21:33 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
>> >> The installed windows version is crashing for me.  When I launch Pcbnew
>> >> from KiCad I get a warning dialog about KISYS3DMOD
>> >> (3d-refactor-win64-install-1.png).  When I dismiss this dialog I get a
>> >> windows system warning dialog (3d-refactor-win64-install-2.png.  When I
>> >> dismiss the windows system warning dialog, pcbnew crashes taking kicad
>> >> with it.  I didn't look but is it possible that Pcbnew is not finding
>> >> the 3d plugin library.  Cirilo, would you please verify the windows
>> >> library load and install paths?
>> >>
>> >> On 2/24/2016 1:12 PM, Nick Østergaard wrote:
>> >>> See http://downloads.kicad-pcb.org/windows/testing/3d_initial_merge/
>> >>>
>> >>> I have not tested it recently, it might have some issue, please test
>> >>> it.
>> >>>
>> >>> 2016-02-24 18:51 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
>> >>>> Thanks Bernhard,
>> >>>>
>> >>>> Before I committed the 3D refactor, I was going to ask our package
>> >>>> devs
>> >>>> to take a look at it due to the new 3D model plugins.  I would like
>> >>>> to
>> >>>> get this correct before I commit it.  What's the status on windows.
>> >>>> I've built and installed from source and that works find but I'm
>> >>>> guessing our installers will need to be changed to accommodate this.
>> >>>> Once we have the install issues sorted out, I'll commit it.
>> >>>>
>> >>>> Cheers,
>> >>>>
>> >>>> Wayne
>> >>>>
>> >>>> On 2/24/2016 12:26 PM, Bernhard Stegmaier wrote:
>> >>>>> Hi,
>> >>>>>
>> >>>>> I just tried on OS X and plugin handling doesn’t seem to be OK.
>> >>>>> At least, “make install” does directly (try to) install them to
>> >>>>> “/Applications/…” which is completely wrong (and also maybe the case
>> >>>>> why
>> >>>>> my built binary doesn’t find them?). Also, the path itself under
>> >>>>> SharedSupport I guess isn’t right.
>> >>>>>
>> >>>>> It also install this here
>> >>>>> bstegmaier$ ls -al /Applications/kicad.app/Contents/Macos/
>> >>>>> total 464
>> >>>>> drwxr-xr-x  4 bstegmaier  admin     136 24 Feb 18:17 .
>> >>>>> drwxr-xr-x  3 bstegmaier  admin     102 24 Feb 18:17 ..
>> >>>>> -rwxr-xr-x  1 bstegmaier  admin  231560 24 Feb 18:17
>> >>>>> libkicad_3dsg..0.0.dylib
>> >>>>> lrwxr-xr-x  1 bstegmaier  admin      24 24 Feb 18:17
>> >>>>> libkicad_3dsg.dylib
>> >>>>> -> libkicad_3dsg..0.0.dylib
>> >>>>> which doesn’t look right, what is it good for?
>> >>>>>
>> >>>>> If you like, I can have a look and try to fix this.
>> >>>>>
>> >>>>>
>> >>>>> Regards,
>> >>>>> Bernhard
>> >>>>>
>> >>>>>> On 23.02.2016, at 22:21, Cirilo Bernardo <cirilo.bernardo@xxxxxxxxx
>> >>>>>> <mailto:cirilo.bernardo@xxxxxxxxx>> wrote:
>> >>>>>>
>> >>>>>>
>> >>>>>>
>> >>>>>> On Tue, Feb 23, 2016 at 7:37 PM, jp charras <jp.charras@xxxxxxxxxx
>> >>>>>> <mailto:jp.charras@xxxxxxxxxx>> wrote:
>> >>>>>>
>> >>>>>>     Le 23/02/2016 08:56, Cirilo Bernardo a écrit :
>> >>>>>>     > On Tue, Feb 23, 2016 at 3:00 AM, Wayne Stambaugh
>> >>>>>> <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
>> >>>>>>     > wrote:
>> >>>>>>     >
>> >>>>>>     >> Cirilo,
>> >>>>>>     >>
>> >>>>>>     >> I have one last change request that I missed in my previous
>> >>>>>> review.
>> >>>>>>     >> Please change your debugging output method.  Windows apps
>> >>>>>> have no
>> >>>>>>     >> concept of stdout even when run from the command line.
>> >>>>>> wxLogTrace is
>> >>>>>>     >> the preferred method for debugging output.  It solves two
>> >>>>>> problems.  The
>> >>>>>>     >> debugging strings are redirected to the correct output on
>> >>>>>> all supported
>> >>>>>>     >> platforms and the debugging output is turned off by default.
>> >>>>>> Turning on
>> >>>>>>     >> the debugging output is as simple as:
>> >>>>>>     >>
>> >>>>>>     >> export WXTRACE="SOME_DEBUGGING_OUTPUT_TO_ENABLE"
>> >>>>>>     >>
>> >>>>>>     >> The is handy so I don't have to sift through everyone's
>> >>>>>> debugging output
>> >>>>>>     >> to find what I'm looking for.  You just need to pick a
>> >>>>>> unique string not
>> >>>>>>     >> used by any other kicad wxLogTrace calls to enable tracing
>> >>>>>> of your
>> >>>>>>     >> debugging output.
>> >>>>>>     >>
>> >>>>>>     >>
>> >>>>>>     > The 3d_initial_merge branch has been updated to use
>> >>>>>> wxLogTrace;
>> >>>>>>     > it has also been synchronized to the main branch r6527.  The
>> >>>>>> latest
>> >>>>>>     > revision for the 3d_initial_merge branch is r6527. The branch
>> >>>>>> builds
>> >>>>>>     > successfully and I have tested the previewer with various
>> >>>>>> VRML1/2 and
>> >>>>>>     > X3D models.
>> >>>>>>     >
>> >>>>>>     > - Cirilo
>> >>>>>>
>> >>>>>>     Thanks, Cirilo.
>> >>>>>>     Very good work.
>> >>>>>>
>> >>>>>>     At least on Windows, 3d_plugin_manager.cpp is missing the line:
>> >>>>>>     #include <wx/filename.h>
>> >>>>>>
>> >>>>>>     Without this include, it does not compile.
>> >>>>>>
>> >>>>>>     --
>> >>>>>>     Jean-Pierre CHARRAS
>> >>>>>>
>> >>>>>>     _______________________________________________
>> >>>>>>     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
>> >>>>>>
>> >>>>>>
>> >>>>>> Thanks Jean-Pierre,
>> >>>>>>
>> >>>>>>  I have added the header and checked that it builds on Windows
>> >>>>>> (W7-64
>> >>>>>> bit).
>> >>>>>>
>> >>>>>> - Cirilo
>> >>>>>>
>> >>>>>> _______________________________________________
>> >>>>>> 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/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
>
>


References