|
From: Philippe W. <phi...@sk...> - 2016-11-04 22:24:30
|
On Thu, 2016-11-03 at 23:41 -0400, Rhys Kidd wrote: > #if defined(VGO_linux) > +#define PID_EXEPATH "/proc/%d/exe" > +#define SELF_EXEPATH "/proc/self/exe" > +#define SELF_EXEFD "/proc/self/fd/%d" > +#elif defined(VGO_solaris) > +#define PID_EXEPATH "/proc/%d/path/a.out" > +#define SELF_EXEPATH "/proc/self/path/a.out" > +#define SELF_EXEFD "/proc/self/path/%d" > +#endif > > > This changes the logic of the define guards. > > Previously #if defined(VGO_linux) would continue to guard this next > block until the #elif defined(VGO_solaris) took over. > > > The union of #if defined(VGO_linux) || defined(VGO_solaris) does not > cover all platforms that build this file - namely macOS. > > > I'll send a patch shortly to correct this logic. > Humph, completely forgot about macos. Thanks for the fix. Philippe |