]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Feed.pm
workaround Mail::Thread memory leak
[public-inbox.git] / lib / PublicInbox / Feed.pm
index ad058395ce6c19294d832fc91ee1a162ebce9611..a507cda12350a7fda8bbd67a7b727b45f0f36638 100644 (file)
@@ -13,8 +13,6 @@ use constant {
        MAX_PER_PAGE => 25,
 };
 
-# FIXME: workaround https://rt.cpan.org/Public/Bug/Display.html?id=22817
-
 # main function
 sub generate {
        my ($class, $args) = @_;
@@ -23,7 +21,6 @@ sub generate {
        require POSIX;
        my $max = $args->{max} || MAX_PER_PAGE;
 
-       local $ENV{GIT_DIR} = $args->{git_dir};
        my $feed_opts = get_feedopts($args);
        my $addr = $feed_opts->{address};
        $addr = $addr->[0] if ref($addr);
@@ -44,15 +41,16 @@ sub generate {
                my ($add) = @_;
                add_to_feed($feed_opts, $feed, $add, $git);
        });
+       $git = undef; # destroy pipes
+       Email::Address->purge_cache;
        $feed->as_string;
 }
 
 sub generate_html_index {
        my ($class, $args) = @_;
-       require Mail::Thread;
+       require PublicInbox::Thread;
 
        my $max = $args->{max} || MAX_PER_PAGE;
-       local $ENV{GIT_DIR} = $args->{git_dir};
        my $feed_opts = get_feedopts($args);
 
        my $title = $feed_opts->{description} || '';
@@ -70,8 +68,9 @@ sub generate_html_index {
                push @messages, $mime;
                1;
        });
+       $git = undef; # destroy pipes.
 
-       my $th = Mail::Thread->new(@messages);
+       my $th = PublicInbox::Thread->new(@messages);
        $th->thread;
        my $html = "<html><head><title>$title</title>" .
                '<link rel="alternate" title="Atom feed" href="' .
@@ -87,6 +86,8 @@ sub generate_html_index {
        });
        dump_html_line($_, 0, \$html) for $th->rootset;
 
+       Email::Address->purge_cache;
+
        my $footer = nav_footer($args->{cgi}, $last);
        $footer = "<hr /><pre>$footer</pre>" if $footer;
        $html . "</pre>$footer</html>";
@@ -130,7 +131,8 @@ sub each_recent_blob {
        # get recent messages
        # we could use git log -z, but, we already know ssoma will not
        # leave us with filenames with spaces in them..
-       my @cmd = qw/git log --no-notes --no-color --raw -r/;
+       my @cmd = ('git', "--git-dir=$args->{git_dir}",
+                       qw/log --no-notes --no-color --raw -r/);
        push @cmd, $range;
 
        my $pid = open(my $log, '-|', @cmd) or