Re: [Monotone-devel] Commit error (Possible LUA problem)

[Philipp Gröschler]

Ahh, we're getting closer. Let's see ...

Stephen Leake schrieb:
> What are OS are you on, and what language settings?
Gentoo Linux on all machines, here on the workstation specifically:
> Linux 2.6.31-gentoo-r6 #1 SMP PREEMPT x86_64

Language setting is german, but Monotone was compiled without NLS.

> So it looks like the path to the executable is nil (should be 'vi', in
> this case). That is in turn specified by the environment variables
> "VISUAL" and "EDITOR" (see std_hooks edit_comment). If
> those env variables are null, it searches PATH for executables named
> 'editor' and 'vi'.
EDITOR is set to "/usr/bin/vim" as it should be, but VISUAL does not exist.

Thomas Keller schrieb:
> I can only guess here, but I'd propose you load a custom rc file which
> contains the current std_hooks.lua contents and debug your way through
> until you see the problem.
The strace should uncover those, right? I didn't create any custom hook
scripts, except our server, were everything works just fine. This is by
the way the only machine in our network which didn't get heavily updated
in the last month. As I stated before, I have been installing KDE4 and
my guess is, that this might have caused this situation.

I attached a text file with the (nearly complete) trace. (Only the
workspace files and some paths have been omitted)

> Philipp, try to open lua from command line and enter
> 
>> function foo(...) return unpack(arg) end
>> = foo(1,2,3)
> 
> Do you see
> 
> 1       2       3
> 
> as output then? If yes, then its not a lua problem.

No, in fact I see this:
> Lua 5.1.4  Copyright (C) 1994-2008 Lua.org, PUC-Rio
>> function foo(...) return unpack(arg) end         
>> = foo(1,2,3)                                     
> stdin:1: bad argument #1 to 'unpack' (table expected, got nil)
> stack traceback:                                              
>         [C]: in function 'unpack'                             
>         stdin:1: in function <stdin:1>                        
>         (tail call): ?                                        
>         [C]: ?      

I guess we have some clue now :-)

execve("/usr/bin/mtn", ["mtn", "ci"], [/* 69 vars */]) = 0
access("/etc/ld.so.preload", R_OK)      = -1 ENOENT (No such file or directory)
open("/etc/ld.so.cache", O_RDONLY)      = 3
open("/lib/libpcre.so.0", O_RDONLY)     = 3
open("/usr/lib/libbotan-1.8.2.so", O_RDONLY) = 3
open("/usr/lib/liblua.so.5", O_RDONLY)  = 3
open("/usr/lib/libsqlite3.so.0", O_RDONLY) = 3
open("/usr/lib/libidn.so.11", O_RDONLY) = 3
open("/lib/libz.so.1", O_RDONLY)        = 3
open("/usr/lib/gcc/x86_64-pc-linux-gnu/4.3.4/libstdc++.so.6", O_RDONLY) = 3
open("/lib/libm.so.6", O_RDONLY)        = 3
open("/lib/libgcc_s.so.1", O_RDONLY)    = 3
open("/lib/libc.so.6", O_RDONLY)        = 3
open("/lib/libpthread.so.0", O_RDONLY)  = 3
open("/lib/libbz2.so.1", O_RDONLY)      = 3
open("/usr/lib/libcrypto.so.0.9.8", O_RDONLY) = 3
open("/lib/librt.so.1", O_RDONLY)       = 3
open("/lib/libdl.so.2", O_RDONLY)       = 3
open("/usr/lib64/locale/locale-archive", O_RDONLY) = 3
open("/usr/lib64/gconv/gconv-modules.cache", O_RDONLY) = 3
getcwd("/####", 4096) = 59
stat("/####/_MTN", {st_mode=S_IFDIR|0755, st_size=144, ...}) = 0
stat("/####/_MTN", {st_mode=S_IFDIR|0755, st_size=144, ...}) = 0
stat("/####", {st_mode=S_IFDIR|0755, st_size=696, ...}) = 0
chdir("/####") = 0
stat("/####/.monotone/monotonerc", 0x7ffff455c580) = -1 ENOENT (No such file or 
directory)
stat("_MTN/monotonerc", 0x7ffff455c580) = -1 ENOENT (No such file or directory)
stat("_MTN/format", {st_mode=S_IFREG|0644, st_size=2, ...}) = 0
stat("_MTN/format", {st_mode=S_IFREG|0644, st_size=2, ...}) = 0
open("_MTN/format", O_RDONLY)           = 3
stat("_MTN/options", {st_mode=S_IFREG|0644, st_size=162, ...}) = 0
open("_MTN/options", O_RDONLY)          = 3
stat("_MTN/options", {st_mode=S_IFREG|0644, st_size=162, ...}) = 0
stat("_MTN/options", {st_mode=S_IFREG|0644, st_size=162, ...}) = 0
open("_MTN/options", O_RDONLY)          = 3
stat("/####/Develop/Test.mtn", {st_mode=S_IFREG|0644, st_size=573440, ...}) = 0
stat("/####/.monotone/keys", {st_mode=S_IFDIR|0755, st_size=128, ...}) = 0
stat("_MTN", {st_mode=S_IFDIR|0755, st_size=144, ...}) = 0
stat("_MTN/options", {st_mode=S_IFREG|0644, st_size=162, ...}) = 0
stat("_MTN", {st_mode=S_IFDIR|0755, st_size=144, ...}) = 0
open("_MTN/mt5z7qi7.tmp", O_RDWR|O_CREAT|O_EXCL, 0666) = 3
rename("_MTN/mt5z7qi7.tmp", "_MTN/options") = 0
stat("_MTN/revision", {st_mode=S_IFREG|0644, st_size=133, ...}) = 0
open("_MTN/revision", O_RDONLY)         = 3
stat("/####/Develop/Test.mtn", {st_mode=S_IFREG|0644, st_size=573440, ...}) = 0
stat("/####/Develop/Test.mtn", {st_mode=S_IFREG|0644, st_size=573440, ...}) = 0
open("/####/Develop/Test.mtn", O_RDWR|O_CREAT, 0644) = 3
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
access("/####/Develop/Test.mtn-journal", F_OK) = -1 ENOENT (No such file or 
directory)
stat("_MTN/revision", {st_mode=S_IFREG|0644, st_size=133, ...}) = 0
open("_MTN/revision", O_RDONLY)         = 4
stat("_MTN/inodeprints", 0x7ffff455b4a0) = -1 ENOENT (No such file or directory)
stat(".", {st_mode=S_IFDIR|0755, st_size=696, ...}) = 0
stat("img", {st_mode=S_IFDIR|0755, st_size=712, ...}) = 0
stat(".htaccess", {st_mode=S_IFREG|0644, st_size=642, ...}) = 0
stat(".htaccess", {st_mode=S_IFREG|0644, st_size=642, ...}) = 0
open(".htaccess", O_RDONLY)             = 4

....

mtn: beginning commit on branch '####'
stat("_MTN/log", {st_mode=S_IFREG|0644, st_size=0, ...}) = 0
stat("_MTN/log", {st_mode=S_IFREG|0644, st_size=0, ...}) = 0
open("_MTN/log", O_RDONLY)              = 4
open("/tmp/mtn.8KVQKC", O_RDWR|O_CREAT|O_EXCL, 0600) = 4
open("/tmp/mtn.8KVQKC", O_RDWR)         = 4
--- SIGCHLD (Child exited) @ 0 (0) ---
mtn: warning: [string "<std hooks>"]:33: bad argument #1 to 'unpack' (table 
expected, got nil)
mtn: misuse: edit of log message failed