From c13272432ad28adb506faf6fb9121569cf5ec710 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Mon, 21 Dec 2020 07:51:22 +0000 Subject: extsearch*: drop unnecessary path canonicalization Unlike inboxdir, the canonical-ness of -extindex paths is not relevant at the moment, and may never be relevant at all. So don't mislead others into thinking these paths being canonicalized matters. --- lib/PublicInbox/ExtSearch.pm | 2 -- lib/PublicInbox/ExtSearchIdx.pm | 2 -- 2 files changed, 4 deletions(-) (limited to 'lib') diff --git a/lib/PublicInbox/ExtSearch.pm b/lib/PublicInbox/ExtSearch.pm index 2a560935..a2b97798 100644 --- a/lib/PublicInbox/ExtSearch.pm +++ b/lib/PublicInbox/ExtSearch.pm @@ -9,7 +9,6 @@ use strict; use v5.10.1; use PublicInbox::Over; use PublicInbox::Inbox; -use File::Spec (); use PublicInbox::MiscSearch; use DBI qw(:sql_types); # SQL_BLOB @@ -18,7 +17,6 @@ use parent qw(PublicInbox::Search); sub new { my (undef, $topdir) = @_; - $topdir = File::Spec->canonpath($topdir); bless { topdir => $topdir, # xpfx => 'ei15' diff --git a/lib/PublicInbox/ExtSearchIdx.pm b/lib/PublicInbox/ExtSearchIdx.pm index c4b429df..f04e0443 100644 --- a/lib/PublicInbox/ExtSearchIdx.pm +++ b/lib/PublicInbox/ExtSearchIdx.pm @@ -30,13 +30,11 @@ use PublicInbox::V2Writable; use PublicInbox::InboxWritable; use PublicInbox::ContentHash qw(content_hash); use PublicInbox::Eml; -use File::Spec; use PublicInbox::DS qw(now); use DBI qw(:sql_types); # SQL_BLOB sub new { my (undef, $dir, $opt) = @_; - $dir = File::Spec->canonpath($dir); my $l = $opt->{indexlevel} // 'full'; $l !~ $PublicInbox::SearchIdx::INDEXLEVELS and die "invalid indexlevel=$l\n"; -- cgit v1.2.3-24-ge0c7