>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:

Angus> Confusion? Senility? I don't have easy access to the source
Angus> code from my work machine and there's no "viewsvn" on the LyX
Angus> web site so you have to rely on my fading memory?

I definitely have an advantage over you here, because (this time at
least) I checked the code before claiming anything :)

Angus> Currently, the code appears to be broken on Windows, so can I
Angus> suggest you choose one of: * change the comment to reflect
Angus> working code * change broken to working code to reflect the
Angus> comment * change both code and comment

I think the comment are correct and Enrico's patch is the right one. I
just suggested a small improvement on it by removing the unneeded
"../Resources". If we can confirm that it works with windows and OSX,
I would therefore propose to apply the attached slight modification to
Enrico's patch.

JMarc

--- src/support/package.C.in.orig	2006-04-17 07:03:06.000000000 +0200
+++ src/support/package.C.in	2006-04-26 01:14:48.000000000 +0200
@@ -689,7 +689,11 @@ bool check_env_var_dir(string const & di
 // The locale directory relative to the LyX system directory.
 string const relative_locale_dir()
 {
+#if defined (USE_WINDOWS_PACKAGING) || defined (USE_MACOSX_PACKAGING)
+	return "locale/";
+#else
 	return "../locale/";
+#endif
 }
 
 

Reply via email to