From 226d56433c045cc7fbb6611c129532696a13762b Mon Sep 17 00:00:00 2001 From: Dick Hollenbeck Date: Fri, 27 May 2016 15:31:09 -0400 Subject: [PATCH] Remove support for in processing setting of ld library path. --- common/single_top.cpp | 40 ---------------------------------------- kicad/kicad.cpp | 37 ------------------------------------- 2 files changed, 77 deletions(-) diff --git a/common/single_top.cpp b/common/single_top.cpp index c3dbe05210..70165014cb 100644 --- a/common/single_top.cpp +++ b/common/single_top.cpp @@ -47,42 +47,6 @@ #include -// The functions we use will cause the program launcher to pull stuff in -// during linkage, keep the map file in mind to see what's going into it. - -/// Extend LIB_ENV_VAR list with the directory from which I came, prepending it. -static void set_lib_env_var( const wxString& aAbsoluteArgv0 ) -{ - // POLICY CHOICE 2: Keep same path, so that installer MAY put the - // "subsidiary DSOs" in the same directory as the kiway top process modules. - // A subsidiary shared library is one that is not a top level DSO, but rather - // some shared library that a top level DSO needs to even be loaded. It is - // a static link to a shared object from a top level DSO. - - // This directory POLICY CHOICE 2 is not the only dir in play, since LIB_ENV_VAR - // has numerous path options in it, as does DSO searching on linux, windows, and OSX. - // See "man ldconfig" on linux. What's being done here is for quick installs - // into a non-standard place, and especially for Windows users who may not - // know what the PATH environment variable is or how to set it. - - wxFileName fn( aAbsoluteArgv0 ); - - wxString ld_path( LIB_ENV_VAR ); - wxString my_path = fn.GetPath(); - wxString new_paths = PrePendPath( ld_path, my_path ); - - wxSetEnv( ld_path, new_paths ); - -#if defined(DEBUG) - { - wxString test; - wxGetEnv( ld_path, &test ); - printf( "LIB_ENV_VAR:'%s'\n", TO_UTF8( test ) ); - } -#endif -} - - // Only a single KIWAY is supported in this single_top top level component, // which is dedicated to loading only a single DSO. KIWAY Kiway( &Pgm(), KFCTL_STANDALONE ); @@ -223,10 +187,6 @@ bool PGM_SINGLE_TOP::OnPgmInit( wxApp* aWxApp ) return false; } - // Set LIB_ENV_VAR *before* loading the DSO, in case the top-level DSO holding the - // KIFACE has hard dependencies on subsidiary DSOs below it. - set_lib_env_var( absoluteArgv0 ); - if( !initPgm() ) return false; diff --git a/kicad/kicad.cpp b/kicad/kicad.cpp index 5bd4e3ab92..4a2d25a210 100644 --- a/kicad/kicad.cpp +++ b/kicad/kicad.cpp @@ -43,39 +43,6 @@ #include -/// Extend LIB_ENV_VAR list with the directory from which I came, prepending it. -static void set_lib_env_var( const wxString& aAbsoluteArgv0 ) -{ - // POLICY CHOICE 2: Keep same path, so that installer MAY put the - // "subsidiary DSOs" in the same directory as the kiway top process modules. - // A subsidiary shared library is one that is not a top level DSO, but rather - // some shared library that a top level DSO needs to even be loaded. It is - // a static link to a shared object from a top level DSO. - - // This directory POLICY CHOICE 2 is not the only dir in play, since LIB_ENV_VAR - // has numerous path options in it, as does DSO searching on linux, windows, and OSX. - // See "man ldconfig" on linux. What's being done here is for quick installs - // into a non-standard place, and especially for Windows users who may not - // know what the PATH environment variable is or how to set it. - - wxFileName fn( aAbsoluteArgv0 ); - - wxString ld_path( LIB_ENV_VAR ); - wxString my_path = fn.GetPath(); - wxString new_paths = PrePendPath( ld_path, my_path ); - - wxSetEnv( ld_path, new_paths ); - -#if defined(DEBUG) - { - wxString test; - wxGetEnv( ld_path, &test ); - printf( "LIB_ENV_VAR:'%s'\n", TO_UTF8( test ) ); - } -#endif -} - - // a dummy to quiet linking with EDA_BASE_FRAME::config(); #include KIFACE_I& Kiface() @@ -108,10 +75,6 @@ bool PGM_KICAD::OnPgmInit( wxApp* aWxApp ) return false; } - // Set LIB_ENV_VAR *before* loading the KIFACE DSOs, in case they have hard - // dependencies on subsidiary DSOs below it. - set_lib_env_var( absoluteArgv0 ); - if( !initPgm() ) return false;