[Bioperl-l] github -- pull request for adding bp_ to scripts
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:
>>>> 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:
>> 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.
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.
More information about the Bioperl-l