kicad-developers team mailing list archive

[Adam Wolf <adamwolf@xxxxxxxxxxxxxxxxxxxx>]

Hi Wayne,

This patch implements something Bernhard and Garth and I discussed and
agreed to on the list.  I haven't heard anyone reject it, but I also
haven't seen any agreement.

If you could take a look and see if you agree with the APPLE if defined
stuff, that would be good... Then I think it's ready.

Adam Wolf
On Jan 13, 2015 12:00 PM, "Wayne Stambaugh" <stambaughw@xxxxxxxxx> wrote:

> Adam,
>
> Is this patch ready to be committed?  I've got a backlog of OSX patches
> and I'm not sure what should be committed and what shouldn't.  Would the
> OSX devs please give me a "this is ready to commit" ping so nothing
> falls through the cracks.
>
> Thanks,
>
> Wayne
>
> On 1/2/2015 4:12 PM, Adam Wolf wrote:
> > Hi folks,
> >
> > I have written a patch that modifies the OS X help search path so it
> > looks inside of
> >
> > /Libraries/Application Support/kicad/help
> > <whatever.app>/Contents/SharedSupport/help
> >
> > per our previous discussion.
> >
> > I have wrapped the existing code in a ! __APPLE__ define, and wrapped
> > the OS X only code in a __APPLE__ define.
> >
> > I was thinking about removing the commented __linux__ and __WINDOWS__
> > ifdefs, but someone on the list recently mentioned both paths with
> > regards to Windows help files, so I am not sure if they are "canonical"
> > or not.  It will be a 2 second change to switch them from ! __APPLE__ to
> > __linux__ and __WINDOWS__, so no problem, just let me know if that is
> > preferable.
> >
> > I tried to follow all the coding conventions as well as make the code
> > blend with the existing code.
> >
> > I have tested it, and this works for both the kicad.app and the
> > symlinked apps as well, like pcbnew.app.
> >
> > Thanks everyone!  Let me know if there's anything I need to change.
> >
> > Adam Wolf
> > Cofounder and Engineer
> > W&L
> >
> >
> >
> > _______________________________________________
> > 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
>