[Bioperl-l] github -- pull request for adding bp_ to scripts

Chris Fields cjfields at illinois.edu
Mon Jul 25 14:13:11 EDT 2011


On Jul 25, 2011, at 12:59 PM, Carnë Draug wrote:

> 2011/7/25 Chris Fields <cjfields at illinois.edu>:
>> (Lincoln, maybe you can add to the below?)
>> 
>> On Jul 25, 2011, at 12:21 PM, Carnë Draug wrote:
>> 
>>> 2011/7/25 Chris Fields <cjfields at illinois.edu>:
>>>> I responded to that:
>>>> 
>>>> http://lists.open-bio.org/pipermail/bioperl-l/2011-July/035425.html
>>>> 
>>>> I haven't heard any arguments against it, will merge in today.  We will need to ensure Build.PL is set up so the scripts are installed correctly.
>>> 
>>> Oh, you're right. I somehow missed it. Thank you.
>>> 
>>> Another thing that I found weird is the extension. Why the .PLS
>>> extension? It's the first time I see it for a perl script and actually
>>> confuses my desktop.
>> 
>> I recall there being an argument for this at some point; there is an old post here:
>> 
>> http://thread.gmane.org/gmane.comp.lang.perl.bio.general/880
>> http://thread.gmane.org/gmane.comp.lang.perl.bio.general/1150/focus=1216
>> 
>> The way I read that, I think the reasoning was to indicate the scripts are templates and the proper perl version would be affixed during build/installation.  Maybe this is now outdated; we have since moved on from ExtUtils::MakeMakes to Module::Build; IIRC the tempting system mentioned in the latter thread still works regardless of the file extension.
> 
> At least the script 'bp_das_server' has the .pl extension so I'm
> guessing it no longer makes any difference.
> 
> Carnë

I think it'll be okay, but maybe we can submit that as a separate pull request just in case.  I would like a a bit more feedback from the GMOD side.

chris


More information about the Bioperl-l mailing list