Package: skippy
Version: 0.5.0-3
Severity: minor

Speculatively installed skippy from weekly news comment.
Haven't yet seen it work, but was waylaid but this output:

    [EMAIL PROTECTED]:~ >skippy
    WARNING: $HOME not set, not loading config.

$HOME is certainly set and in the environment, and skippy is not setuid.
Further, an ltrace shows the return value of getenv("HOME") as
"/home/jrodman", so I'm fairly certain this message is incorrect.

Also, I believe this is not accurate in terms of its statement of
intent, since the strace shows:

    open("/home/jrodman/.skippyrc", O_RDONLY) = 4
    fstat64(4, {st_mode=S_IFREG|0644, st_size=1107, ...}) = 0
    mmap2(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 
0xb7ffe000
    fstat64(4, {st_mode=S_IFREG|0644, st_size=1107, ...}) = 0
    _llseek(4, 0, [0], SEEK_SET)            = 0
    read(4, "# This is the Default Debian ski"..., 1107) = 1107
    _llseek(4, 1107, [1107], SEEK_SET)      = 0
    close(4)                                = 0
    munmap(0xb7ffe000, 4096)                = 0
    write(2, "WARNING: $HOME not set, not load"..., 44) = 44

So this message is written after it actually loads the file.
looking at the source logic, this appears to be a bug in
11_find_skippyrc_in_etc.patch

The logic, with patch applied, is:

    1) Check for HOME
      If HOME set, try to load config from $HOME/.skipopyrc

        2) If config is not yet successfully loaded try to load config from 
           /etc/X11/skippy/skippyrc

        3) Otherwise, if config WAS successfully loaded report an error about 
$HOME dir

So the error message about $HOME will only appear if the configuration was
actually successfully loaded out of $HOME.




Traditionally unix programs don't tend to actively complain about missing
config files, so maybe you should just remove the error.  Ideally I think
it would be a verbose form of output, but not by default.


-- System Information:
Debian Release: 3.1
  APT prefers testing
  APT policy: (990, 'testing')
Architecture: i386 (i686)
Kernel: Linux 2.6.10-jsr
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)

Versions of packages skippy depends on:
ii  libc6                    2.3.2.ds1-20    GNU C Library: Shared libraries an
ii  libfontconfig1           2.3.1-2         generic font configuration library
ii  libfreetype6             2.1.7-2.3       FreeType 2 font engine, shared lib
ii  libimlib2                1.2.0-2.1       powerful image loading and renderi
ii  libx11-6                 4.3.0.dfsg.1-10 X Window System protocol client li
ii  libxext6                 4.3.0.dfsg.1-10 X Window System miscellaneous exte
ii  libxft2                  2.1.2-6         FreeType-based font drawing librar
ii  libxrender1              0.8.3-7         X Rendering Extension client libra
ii  xlibs                    4.3.0.dfsg.1-10 X Keyboard Extension (XKB) configu
ii  zlib1g                   1:1.2.2-3       compression library - runtime

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to