GMOD / GBrowse

the Generic Genome Browser

Home Page:http://gmod.org/wiki/GBrowse

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

errors in pod

osallou opened this issue · comments

Hi,
in v2.42, I found several errors in POD structure, preventing the pod2man transformation, plus a few typo errors.
Follows a patch to fix those errors.
I am not a pod expert, fixes remove errors but you may check what is best appropriate.

Patch to correct POD-generated manual pages - Olivier Sallou - February 2011
--- a/bin/gtf2gff3.pl
+++ b/bin/gtf2gff3.pl
@@ -1320,7 +1320,7 @@

=head1 NAME

-gtf2gff3
+gtf2gff3 - Converts GTF formatted files to valid GFF3 files

=head1 VERSION

@@ -1496,7 +1496,7 @@
=item C<< FATAL: Invalid value passed to strand: strand. >>

This may indicate that your GTF file does not indicate the strand for
-features that require it. Consider using the DEFAULT_STRAND paramater
+features that require it. Consider using the DEFAULT_STRAND parameter
in the config file. It may also indicate a software bug. Please
contact the author.

--- a/bin/wiggle2gff3.pl
+++ b/bin/wiggle2gff3.pl
@@ -2,7 +2,7 @@

=head1 NAME

-wiggle2gff3.pl
+wiggle2gff3.pl - Converts UCSC WIG format files into gff3 files

=head1 SYNOPSIS

--- a/lib/Bio/DB/GFF/Aggregator/match_gap.pm
+++ b/lib/Bio/DB/GFF/Aggregator/match_gap.pm
@@ -34,7 +34,6 @@
should be sufficient for simple nucleotide to nucleotide
alignments.

-################################################################################

=cut

--- a/lib/Bio/DB/GFF/Aggregator/reftranscript.pm
+++ b/lib/Bio/DB/GFF/Aggregator/reftranscript.pm
@@ -1,6 +1,6 @@
=head1 NAME

-package Bio::DB::GFF::Aggregator::reftranscript -- Aggregates references transcripts
+Bio::DB::GFF::Aggregator::reftranscript -- Aggregates references transcripts

=head1 SYNOPSIS

--- a/lib/Bio/Graphics/Browser2/AuthorizedFeatureFile.pm
+++ b/lib/Bio/Graphics/Browser2/AuthorizedFeatureFile.pm
@@ -23,7 +23,6 @@

=head2 METHODS

-=over 4

=cut

--- a/lib/Bio/Graphics/Browser2/DataBase.pm
+++ b/lib/Bio/Graphics/Browser2/DataBase.pm
@@ -63,12 +63,16 @@
$CACHE->delete($key);
}

+=over
+
=item Bio::Graphics::Browser2::DataBase->clone_databases()

Call this after a fork in the child process to make sure that all open
databases have had a chance to clone themselves if they need
to. Otherwise you will get random database failures.

+=back
+
=cut

sub clone_databases {
--- a/lib/Bio/Graphics/Browser2/Plugin.pm
+++ b/lib/Bio/Graphics/Browser2/Plugin.pm
@@ -238,7 +238,7 @@

The purpose of this methods is to suppress the 'Configure...'
or 'Find...' title that is printed at the top of the page when the
-plugin is loaded. It will return false unless overriden by a plugin where
+plugin is loaded. It will return false unless overridden by a plugin where
this behaviour is desired.

=item $type = $self->type()
--- a/lib/Bio/Graphics/Browser2/Region.pm
+++ b/lib/Bio/Graphics/Browser2/Region.pm
@@ -3,6 +3,12 @@

provide method for fetching and manipulating the current

region or regions.

+=head1 NAME
+
+Bio::Graphics::Browser2::Region -- Provide method for fetching and manipulating regions
+
+=cut
+
use strict;
use warnings;
use Bio::Graphics::Browser2::Shellwords;
--- a/lib/Bio/Graphics/Browser2/Render.pm
+++ b/lib/Bio/Graphics/Browser2/Render.pm
@@ -3,6 +3,12 @@
use strict;
use warnings;

+=head1 NAME
+
+Bio::Graphics::Browser2::Render -- Provide methods to render regions
+
+=cut
+
use JSON;
use Digest::MD5 'md5_hex';
use CGI qw(:standard param request_method header url iframe img span div br center url_param);
--- a/lib/Bio/Graphics/Browser2/RenderPanels.pm
+++ b/lib/Bio/Graphics/Browser2/RenderPanels.pm
@@ -2,6 +2,13 @@
use strict;
use warnings;

+=head1 NAME
+
+Bio::Graphics::Browser2::RenderPanels-- Provide methods to render a panel
+
+=cut
+
+
use GD 'gdTransparent','gdStyled';

use Bio::Graphics;
--- a/lib/Legacy/Graphics/Browser.pm
+++ b/lib/Legacy/Graphics/Browser.pm
@@ -4,6 +4,12 @@

This is an old version of Bio::Graphics::Browser retained for gbrowse_syn

It is on the path to deprecation

+=head1 NAME
+
+Legacy::Graphics::Browser-- Old version, deprecated
+
+=cut
+
=head1 METHODS

The remainder of this document describes the methods available to the
--- a/lib/Bio/Graphics/Browser2/DataSource.pm
+++ b/lib/Bio/Graphics/Browser2/DataSource.pm
@@ -31,6 +31,10 @@
}
}

+=head1 NAME
+
+Bio::Graphics::Browser2::DataSource -- DataSource to access data
+
=head1 SYNOPSIS

=head1 DESCRIPTION
@@ -149,6 +153,8 @@
delete $CONFIG_CACHE{$self->config_file};
}

+=back
=head2 userdata()

$path = $source->userdata(@path_components)
@@ -852,6 +858,8 @@
croak "Do not call make_link() on the DataSource. Call it on the Render object";
}

+=over
+
=item $db = $dsn->databases

Return all named databases from [name:database] tracks.
@@ -1086,6 +1094,8 @@
%DB_SETTINGS = ();
}

+=back
+
=head2 generate_image

($url,$path) = generate_image($gd);

A pull request via git would make it easiest to incorporate your changes. If not I expect someone can make these fixes.