[Bioperl-l] github -- pull request for adding bp_ to scripts
Chris Fields
cjfields at illinois.edu
Mon Jul 25 18:13:11 UTC 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