Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File::Spec->tmpdir broken when running as service #2098

Closed
p5pRT opened this issue Jun 16, 2000 · 2 comments
Closed

File::Spec->tmpdir broken when running as service #2098

p5pRT opened this issue Jun 16, 2000 · 2 comments

Comments

@p5pRT
Copy link

p5pRT commented Jun 16, 2000

Migrated from rt.perl.org#3389 (status was 'resolved')

Searchable as RT3389$

@p5pRT
Copy link
Author

p5pRT commented Jun 16, 2000

From matt@sergeant.org

Created by matt@sergeant.org

Please ignore the system spec below - this is a bug for Win32 (up
to and including perl-current).

File​::Spec uses the following for obtaining the tmpdir value​:

$ENV{TMPDIR} || $ENV{TEMP} || $ENV{TMP} || '/tmp'

(or code to that effect)

By default, Windows NT does not define the above environment
variables for the user account that runs services. So the default
of /tmp gets used. Since the current directory might be on any of
the available drives, /tmp could mean multiple things - causing bad
things to happen.

I suggest changing this selection to​:

$ENV{TMPDIR} || $ENV{TEMP} || $ENV{TMP} || $ENV{WINDIR} || 'C​:/temp'

Which is what MS Developer studio apparently uses (so I've heard).

Perl Info


Site configuration information for perl 5.00503:

Configured by root at Fri Dec 17 17:15:45 CET 1999.

Summary of my perl5 (5.0 patchlevel 5 subversion 3) configuration:
  Platform:
    osname=linux, osvers=2.2.14-1mdksmp, archname=i386-linux
    uname='linux jedi.mandrakesoft.com 2.2.14-1mdksmp #1 smp thu dec 2 01:02:03 cet 1999 i686 unknown '
    hint=recommended, useposix=true, d_sigaction=define
    usethreads=undef useperlio=undef d_sfio=undef
  Compiler:
    cc='cc', optimize='-O3 -fomit-frame-pointer -fno-exceptions -fno-rtti -pipe -s -mpentium -mcpu=pentium -march=pentium -ffast-math -fexpensive-optimizations', gccversion=2.95.2 19991024 (release)
    cppflags='-Dbool=char -DHAS_BOOL -I/usr/local/include'
    ccflags ='-Dbool=char -DHAS_BOOL -I/usr/local/include'
    stdchar='char', d_stdstdio=undef, usevfork=false
    intsize=4, longsize=4, ptrsize=4, doublesize=8
    d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=12
    alignbytes=4, usemymalloc=n, prototype=define
  Linker and Libraries:
    ld='cc', ldflags =' -L/usr/local/lib'
    libpth=/usr/local/lib /lib /usr/lib
    libs=-lnsl -ldl -lm -lc -lposix -lcrypt
    libc=, so=so, useshrplib=false, libperl=libperl.a
  Dynamic Linking:
    dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-rdynamic'
    cccdlflags='-fpic', lddlflags='-shared -L/usr/local/lib'

Locally applied patches:
    


@INC for perl 5.00503:
    /usr/lib/perl5/5.00503/i386-linux
    /usr/lib/perl5/5.00503
    /usr/lib/perl5/site_perl/5.005/i386-linux
    /usr/lib/perl5/site_perl/5.005
    .


Environment for perl 5.00503:
    HOME=/home/matt
    LANG=en
    LANGUAGE=en
    LC_ALL=en_GB
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=/usr/local/bin:/bin:/usr/bin::/opt/sybase/bin:/opt/ezmlm:/usr/X11R6/bin:/usr/games:/home/matt/bin:/opt/sybase/bin:/opt/ezmlm:/usr/X11R6/bin:/usr/games:/opt/sybase/bin:/opt/ezmlm:/usr/X11R6/bin:/usr/games:/opt/sybase/bin:/opt/ezmlm:/usr/X11R6/bin:/usr/games:/home/matt/bin:/home/matt/oldhome/bin
    PERL_BADLANG (unset)
    SHELL=/bin/bash


@p5pRT
Copy link
Author

p5pRT commented Jun 16, 2000

From @pudge

At 16.30 +0000 2000.06.16, matt@​sergeant.org wrote​:

Please ignore the system spec below - this is a bug for Win32 (up
to and including perl-current).

File​::Spec uses the following for obtaining the tmpdir value​:

$ENV{TMPDIR} || $ENV{TEMP} || $ENV{TMP} || '/tmp'

(or code to that effect)

By default, Windows NT does not define the above environment
variables for the user account that runs services. So the default
of /tmp gets used. Since the current directory might be on any of
the available drives, /tmp could mean multiple things - causing bad
things to happen.

I suggest changing this selection to​:

$ENV{TMPDIR} || $ENV{TEMP} || $ENV{TMP} || $ENV{WINDIR} || 'C​:/temp'

Which is what MS Developer studio apparently uses (so I've heard).

Since it also uses -d to see if the directory exists, you can just
add "C​:/temp" to the list of directories that are used. The code I
see is​:

  foreach (@​ENV{qw(TMPDIR TEMP TMP)}, qw(/tmp /)) {

So it could be​:

  foreach (@​ENV{qw(TMPDIR TEMP TMP)}, qw(/tmp C​:/temp /)) {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant