Skip Menu |
 

This queue is for tickets about the File-Path CPAN distribution.

Report information
The Basics
Id: 120645
Status: resolved
Priority: 0/
Queue: File-Path

People
Owner: jkeenan [...] cpan.org
Requestors: jkeenan [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in:
  • 2.12
  • 2.12_001
  • 2.12_002
Fixed in: (no value)



Subject: t/Path_root.t fails to remove temporary directories
Download (untitled) / with headers
text/plain 1.4k
When testing File-Path as root, we run t/Path_root.t. This file manually creates several directories beneath the current working directory. However, the file does not clean these up before exiting. This leaves the working directory looking something like this: ##### root@localhost:/home/jkeenan/gitwork/File-Path # ls -l total 68 drwxr-xr-x 8 jkeenan wheel 512 Mar 17 09:01 .git -rw-r--r-- 1 jkeenan wheel 200 Mar 8 11:00 .gitignore -rw-r--r-- 1 jkeenan wheel 443 Mar 8 11:00 .travis.yml -rw-r--r-- 1 jkeenan wheel 9428 Mar 14 08:44 Changes -rw-r--r-- 1 jkeenan wheel 207 Mar 8 11:00 MANIFEST -rw-r--r-- 1 jkeenan wheel 352 Mar 8 11:00 MANIFEST.SKIP -rw-r--r-- 1 jkeenan wheel 600 Mar 8 11:00 META.yml -rw-r--r-- 1 jkeenan wheel 1140 Mar 8 11:00 Makefile.PL -rw-r--r-- 1 jkeenan wheel 1878 Mar 14 08:44 README lrwxr-xr-x 1 jkeenan wheel 6 Mar 8 11:00 README.md -> README drwxr-xr-x 8 root wheel 512 Mar 17 08:55 blib drwxr-xr-x 3 jkeenan wheel 512 Mar 8 11:00 lib drwxr-xr-x 2 jkeenan wheel 512 Mar 17 09:02 t drwxr-xr-x 5 root wheel 512 Mar 17 08:55 test-58cbdcd1-441-39ba drwxr-xr-x 5 root wheel 512 Mar 17 08:57 test-58cbdd2c-450-8694 drwxr-xr-x 2 jkeenan wheel 512 Mar 8 11:00 xt ##### Should you then switch back to running as a regular user, certain tests in t/Path.t will fail. A pull request will be created to address this. Thank you very much. Jim Keenan
Download (untitled) / with headers
text/plain 1.5k
On Fri Mar 17 10:48:41 2017, JKEENAN wrote: Show quoted text
> When testing File-Path as root, we run t/Path_root.t. This file > manually creates several directories beneath the current working > directory. However, the file does not clean these up before exiting. > This leaves the working directory looking something like this: > > ##### > root@localhost:/home/jkeenan/gitwork/File-Path # ls -l > total 68 > drwxr-xr-x 8 jkeenan wheel 512 Mar 17 09:01 .git > -rw-r--r-- 1 jkeenan wheel 200 Mar 8 11:00 .gitignore > -rw-r--r-- 1 jkeenan wheel 443 Mar 8 11:00 .travis.yml > -rw-r--r-- 1 jkeenan wheel 9428 Mar 14 08:44 Changes > -rw-r--r-- 1 jkeenan wheel 207 Mar 8 11:00 MANIFEST > -rw-r--r-- 1 jkeenan wheel 352 Mar 8 11:00 MANIFEST.SKIP > -rw-r--r-- 1 jkeenan wheel 600 Mar 8 11:00 META.yml > -rw-r--r-- 1 jkeenan wheel 1140 Mar 8 11:00 Makefile.PL > -rw-r--r-- 1 jkeenan wheel 1878 Mar 14 08:44 README > lrwxr-xr-x 1 jkeenan wheel 6 Mar 8 11:00 README.md -> README > drwxr-xr-x 8 root wheel 512 Mar 17 08:55 blib > drwxr-xr-x 3 jkeenan wheel 512 Mar 8 11:00 lib > drwxr-xr-x 2 jkeenan wheel 512 Mar 17 09:02 t > drwxr-xr-x 5 root wheel 512 Mar 17 08:55 test-58cbdcd1-441-39ba > drwxr-xr-x 5 root wheel 512 Mar 17 08:57 test-58cbdd2c-450-8694 > drwxr-xr-x 2 jkeenan wheel 512 Mar 8 11:00 xt > ##### > > Should you then switch back to running as a regular user, certain > tests in t/Path.t will fail. > > A pull request will be created to address this. > > Thank you very much. > Jim Keenan
https://github.com/rpcme/File-Path/pull/45
Download (untitled) / with headers
text/plain 1.8k
On Fri Mar 17 10:52:21 2017, JKEENAN wrote: Show quoted text
> On Fri Mar 17 10:48:41 2017, JKEENAN wrote:
> > When testing File-Path as root, we run t/Path_root.t. This file > > manually creates several directories beneath the current working > > directory. However, the file does not clean these up before exiting. > > This leaves the working directory looking something like this: > > > > ##### > > root@localhost:/home/jkeenan/gitwork/File-Path # ls -l > > total 68 > > drwxr-xr-x 8 jkeenan wheel 512 Mar 17 09:01 .git > > -rw-r--r-- 1 jkeenan wheel 200 Mar 8 11:00 .gitignore > > -rw-r--r-- 1 jkeenan wheel 443 Mar 8 11:00 .travis.yml > > -rw-r--r-- 1 jkeenan wheel 9428 Mar 14 08:44 Changes > > -rw-r--r-- 1 jkeenan wheel 207 Mar 8 11:00 MANIFEST > > -rw-r--r-- 1 jkeenan wheel 352 Mar 8 11:00 MANIFEST.SKIP > > -rw-r--r-- 1 jkeenan wheel 600 Mar 8 11:00 META.yml > > -rw-r--r-- 1 jkeenan wheel 1140 Mar 8 11:00 Makefile.PL > > -rw-r--r-- 1 jkeenan wheel 1878 Mar 14 08:44 README > > lrwxr-xr-x 1 jkeenan wheel 6 Mar 8 11:00 README.md -> README > > drwxr-xr-x 8 root wheel 512 Mar 17 08:55 blib > > drwxr-xr-x 3 jkeenan wheel 512 Mar 8 11:00 lib > > drwxr-xr-x 2 jkeenan wheel 512 Mar 17 09:02 t > > drwxr-xr-x 5 root wheel 512 Mar 17 08:55 test-58cbdcd1-441-39ba > > drwxr-xr-x 5 root wheel 512 Mar 17 08:57 test-58cbdd2c-450-8694 > > drwxr-xr-x 2 jkeenan wheel 512 Mar 8 11:00 xt > > ##### > > > > Should you then switch back to running as a regular user, certain > > tests in t/Path.t will fail. > > > > A pull request will be created to address this. > > > > Thank you very much. > > Jim Keenan
> > https://github.com/rpcme/File-Path/pull/45
p.r. 45 was merged into master in March. File-Path version 2.13 was released to CPAN today (May 31 2017). Marking ticket Resolved. Thank you very much. Jim Keenan


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.