← Back to team overview

kicad-developers team mailing list archive

Re: OSX: Pin Table crashes Library Editor

 

Your going to have to add a platform or all specifier to the patch name
because some patches are platform specific and some must be applied for
all platforms.

On 6/15/2015 5:04 PM, Bernhard Stegmaier wrote:
> From other build systems I know something like
>   <pkg>-<version>-<seq>-<name>.patch
> 
> So, something like
>   wxWidgets-3.0.2-001-wxDataViewCtrl.patch
> 
> Do we want to add more patches for open bugs in wxWidgets going forward
> to the stable release?
> At least for OS X there is one open issue in 3.0.2 that you cannot open
> other file formats than KiCad native in pcbnew, because the file type
> selector is broken. From a customer perspective it would be good,
> because they just don’t care if the bug is in KiCad or wxWidgets.
> This will increase maintenance effort, though… 
> 
> 
> Regards,
> Bernhard
> 
> 
>> On 15 Jun 2015, at 22:45, Adam Wolf <adamwolf@xxxxxxxxxxxxxxxxxxxx
>> <mailto:adamwolf@xxxxxxxxxxxxxxxxxxxx>> wrote:
>>
>> I'd gladly do *.patch, but not every patch in the patches directory is
>> for wx, and probably not all of them are for OSX...
>>
>> On Jun 15, 2015 3:43 PM, "Nick Østergaard" <oe.nick@xxxxxxxxx
>> <mailto:oe.nick@xxxxxxxxx>> wrote:
>>
>>     *.patch ?
>>
>>     2015-06-15 22:35 GMT+02:00 Adam Wolf
>>     <adamwolf@xxxxxxxxxxxxxxxxxxxx
>>     <mailto:adamwolf@xxxxxxxxxxxxxxxxxxxx>>:
>>     > I will make this update in the builder scripts before tonight's
>>     builds.
>>     >
>>     > I wonder if we can set up a filename convention for wx patches
>>     so they can
>>     > be automatically applied.  Anyone opposed to that idea?  I get
>>     that we're
>>     > going to be maintaining this out-of-tree for after the stable
>>     release, but
>>     > we can probably figure something out...
>>     >
>>     > Adam Wolf
>>     >
>>     > On Mon, Jun 15, 2015 at 3:17 PM, Wayne Stambaugh
>>     <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
>>     > wrote:
>>     >>
>>     >> Patch committed in product branch r5755.  Thanks Bernhard.
>>     >>
>>     >> I guess our OSX package builder script(s) will need to apply
>>     this patch
>>     >> to wxWidgets for our OSX bundles.
>>     >>
>>     >> On 6/14/2015 3:41 PM, Bernhard Stegmaier wrote:
>>     >> > Hi,
>>     >> >
>>     >> > FYI… I guess this is only OS X related.
>>     >> >
>>     >> > Yesterday I wanted to use the Pin Table for the first time
>>     and it always
>>     >> > crashes the Library Editor.
>>     >> > After some debugging I am pretty sure that this is a bug in
>>     the OS X
>>     >> > implementation of wxDataViewCtrl if using wx[V|H]SCROLL when
>>     creating
>>     >> > it.
>>     >> > Did the Pin Table ever work for somebody on OS X (I guess it
>>     never did)?
>>     >> >
>>     >> > I already filed a bug report for wxWidgets (still under
>>     moderation, so
>>     >> > it doesn’t show up yet).
>>     >> > I don’t know how long this will take, so attached a
>>     fix/workaround that
>>     >> > did it for me (for anyone interested, maybe Adam?).
>>     >> >
>>     >> > The only possible workaround in KiCad seems to be to remove the
>>     >> > wxVSCROLL from
>>     >> >
>>     >> >     m_Pins = newwxDataViewCtrl( this, wxID_ANY,
>>     wxDefaultPosition,
>>     >> >     wxSize( -1,-1),
>>     >> >
>>     >> >
>>     wxDV_HORIZ_RULES|wxDV_MULTIPLE|wxDV_ROW_LINES|wxDV_VERT_RULES|wxVSCROLL);
>>     >> >
>>     >> > In my tests OS X doesn’t seem to need this (scroll bars are
>>     dynamically
>>     >> > shown when needed even without this flag).
>>     >> > But, I don’t know about other platforms and this piece of
>>     code probably
>>     >> > can’t be #ifdef’d because this is generated code from
>>     wxFormBuilder?
>>     >> >
>>     >> >
>>     >> > Regards,
>>     >> > Bernhard
>>     >> >
>>     >> >
>>     >> >
>>     >> >
>>     >> >
>>     >> > _______________________________________________
>>     >> > 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
>>     <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
> 


References