[Bioperl-l] Bio::Root::Version functionality is in.

[Brian Osborne]

Aaron,

>Well, that's really two separate questions:

>1) Should that module be using Bio::Root::RootI anyway, regardless of
>$VERSION issues?

Ah. But I did not say that I was asking just one simple question.      ;-)

In fact the first question occurred to me just browsing around, looking at
who was using Root and who was using RootI. It appears that "base" modules
for a given set of modules will use either (and "newer" sets of modules with
interface files seem to always use RootI, makes sense). I guess that we've
arrived at a fairly stable state with respect to Root and RootI that this is
not an issue, but, hypothetically, anyone considering a change to Root/RootI
should be aware of all these modules reading from Root directly. But,
separate and more complex issue than $VERSION, yes.

Brian O.



-----Original Message-----
From: bioperl-l-bounces at bioperl.org [mailto:bioperl-l-bounces at bioperl.org]On
Behalf Of Aaron J Mackey
Sent: Sunday, March 09, 2003 2:33 PM
To: Brian Osborne
Cc: Bioperl
Subject: RE: [Bioperl-l] Bio::Root::Version functionality is in.


On Sun, 9 Mar 2003, Brian Osborne wrote:

> In a module like that would you do "use Bio::Root::RootI" or the more
direct
> "use Bio::Root::Version qw($VERSION);"?

Well, that's really two separate questions:

1) Should that module be using Bio::Root::RootI anyway, regardless of
$VERSION issues?

2) use Bio::Root::Version or Bio::Root::RootI depending on answer to #1.

the qw($VERSION) isn't necessary (in fact, it's currently not supported,
but that could be easy to fix: change @EXPORT to @EXPORT_OK in Version.pm,
then use Bio::Root::Version qw($VERSION) in RootI.pm)

There is one case in which the "viral" $VERSION won't exactly work:

use Bio::SeqIO;
use Bio::SeqIO::fasta 1.2;

__END__
Bio::SeqIO::fasta does not define $Bio::SeqIO::fasta::VERSION--version check
failed at -e line 1.
BEGIN failed--compilation aborted

This doesn't work because RootI only gets compiled once; I'll think about
how to fix this, but we should probably document that if you require a
particular version of BioPerl, then you should declare that need sooner
rather than later (and then only once).

-Aaron

_______________________________________________
Bioperl-l mailing list
Bioperl-l at bioperl.org
http://bioperl.org/mailman/listinfo/bioperl-l