Bernhard's suggestions are good. Hold off on applying until I or someone else fixes it.
Thanks! On Jan 13, 2015 1:20 PM, "Bernhard Stegmaier" <[email protected]> wrote: > Hi, > > the patch is basically fine for me (didn’t try it, just reviewed the > diff), some notes on it: > * Instead of hard-coding "/Library/Application Support/kicad/“ you could > use the function “GetOSXKicadMachineDataDir()” I created some while back… > this probably would make things easier to find/change in future. > * Most of the other Apple specific stuff is wrapped into a __WXMAC__ > instead of __APPLE__. Should not make any difference, but again it might > make things easier when having only one define searching for OSX specific > things. > > > Regards, > Bernhard > > > On 13.01.2015, at 19:23, Adam Wolf <[email protected]> wrote: > > 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" <[email protected]> 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 : [email protected] >> > Unsubscribe : https://launchpad.net/~kicad-developers >> > More help : https://help.launchpad.net/ListHelp >> > >> >> >> _______________________________________________ >> Mailing list: https://launchpad.net/~kicad-developers >> Post to : [email protected] >> Unsubscribe : https://launchpad.net/~kicad-developers >> More help : https://help.launchpad.net/ListHelp >> > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : [email protected] > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > > >
_______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
