Re: testbot: Fix the ConfigLocal.pl include path.
Francois Gouget <fgouget(a)codeweavers.com> writes:
@@ -88,12 +88,11 @@ $JobArchiveDays = 0; if (!$::BuildEnv) { $::BuildEnv = 0; - eval 'require "$::RootDir/ConfigLocal.pl";'; + eval 'require "$::RootDir/lib/WineTestBot/ConfigLocal.pl"'; if ($@) { - print STDERR "Please create a valid $::RootDir/ConfigLocal.pl, " . - "use $::RootDir/lib/WineTestBot/ConfigLocalTemplate.pl as template\n"; - exit; + print STDERR "Please create a valid $::RootDir/lib/WineTestBot/ConfigLocal.pl file, use ConfigLocalTemplate.pl as template\n"; + exit(1);
I had changed it because we don't want to have local config files inside the git checkout. -- Alexandre Julliard julliard(a)winehq.org
On Tue, 11 Dec 2012, Alexandre Julliard wrote:
Francois Gouget <fgouget(a)codeweavers.com> writes:
@@ -88,12 +88,11 @@ $JobArchiveDays = 0; if (!$::BuildEnv) { $::BuildEnv = 0; - eval 'require "$::RootDir/ConfigLocal.pl";'; + eval 'require "$::RootDir/lib/WineTestBot/ConfigLocal.pl"'; [...] I had changed it because we don't want to have local config files inside the git checkout.
I see. It's for when the bin, lib and web directories are replaced with symbolic links to directories belonging to another user. Here I had the configuration file in that other user's account so that WineTestBot could not modify it. But I'm fine with either way. I'll update the instructions. -- Francois Gouget <fgouget(a)codeweavers.com>
participants (2)
-
Alexandre Julliard -
Francois Gouget