Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Simple CPAN distribution.

Report information
The Basics
Id: 119859
Status: resolved
Priority: 0/
Queue: Test-Simple
People
Owner: Nobody in particular
Requestors: craigberry [...] mac.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Jan 14 18:28:23 2017 craigberry [...] mac.com - Ticket created
Subject: [PATCH] Use File::Spec->catfile() in Test2's Files.t test.
Date: Sat, 14 Jan 2017 16:27:51 -0600
To: bugs-Test-Simple [...] rt.cpan.org
From: "Craig A. Berry" <craigberry [...] mac.com>
Explicitly using forward slashes to concatenate filenames is not portable and we've already loaded File::Spec anyway. Also, don't try to remove the directory we're cleaning up unless we were successful in opening it. The patch is against Perl core, so apply with -p3. Show quoted text
________________________________________ Craig A. Berry "... getting out of a sonnet is much more difficult than getting in." Brad Leithauser

Message body is not shown because sender requested not to inline it.

  Sat Jan 14 21:57:33 2017 exodist7 [...] gmail.com - Correspondence added
Patch Applied: https://github.com/Test-More/test-more/commit/1b4514c567c8a5d1352fbfd74361b8dea8a5473b Thanks!
  Sat Jan 14 21:57:35 2017 The RT System itself - Status changed from 'new' to 'open'
  Sat Jan 14 21:57:40 2017 exodist7 [...] gmail.com - Status changed from 'open' to 'resolved'