habacker added a comment.

  In D14779#311511 <https://phabricator.kde.org/D14779#311511>, @aacid wrote:
  
  > Can you please replace your setLocale call in initTestCase for
  >
  >   void initLocale()
  >   {
  >     setenv("LC_ALL", "en_US.utf-8", 1);
  
  
  Just to inform you: this will not work with msvc 
https://stackoverflow.com/questions/17258029/c-setenv-undefined-identifier-in-visual-studio
 and adds an additional 'kdewin' dependency to windows builds
  
  >   }
  >   Q_CONSTRUCTOR_FUNCTION(initLocale)
  >    
  > 
  > This should make it work (or at least it does here)
  
  Just recognized  general issues in kcoreaddons translation, which need to be 
fixed before - please accept D14940 <https://phabricator.kde.org/D14940>.
  
  Then I will rebase this patch and check your suggestion.

REPOSITORY
  R244 KCoreAddons

REVISION DETAIL
  https://phabricator.kde.org/D14779

To: habacker, #frameworks
Cc: aacid, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to