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
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
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
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