[Phylobase-devl] New phylobase build approach using static libncl (Was: Rcpp and OS X compiliation)

François Michonneau francois.michonneau at gmail.com
Sat Feb 27 20:55:29 CET 2010


If I remember correctly, none of the nexus files that are included in
the package work (pkg/inst/nexusfiles). The only file that I can get
to work is the example file that's on the NCL website
(http://hydrodictyon.eeb.uconn.edu/ncl/#readerdata).

  -- François


On Sat, Feb 27, 2010 at 14:51, Peter Cowan <pdc at berkeley.edu> wrote:
>
> On Feb 27, 2010, at 11:39 AM, François Michonneau wrote:
>
>> It would be really good if we could fix the bugs I reported before
>> releasing this updated version to CRAN (in particular bug #812).
>> Indeed, we can't import most nexus trees in phylobase. It looks that
>> the NCL interface needs to be updated but I don't know C++ and it's
>> hard for me to investigate the problem. I'm willing to help though and
>> write the RUnit tests associated with readNexus once the problem is
>> fixed.
>
> Agreed, we should fix this.  I can take a look at it, but I'm not a C++ expert either, do you have an example file that doesn't work for us to test with?
>
> Unit tests would be great, and I was just writing an email suggesting that, when I got this.
>
> I'm willing to set aside some time to do the release once we are satisfied things are working properly.  It'd be nice to do this soonish, because it does make the package much more usable.
>
> Peter
>
>
>
>>  Cheers,
>>  -- François
>>
>>
>> On Sat, Feb 27, 2010 at 14:29, Dirk Eddelbuettel <edd at debian.org> wrote:
>>>
>>> On 27 February 2010 at 14:08, Ben Bolker wrote:
>>> |   fine with me, phylobase passes R CMD check on my system without it.
>>> | We can always restore it from SVN if it breaks something, right?
>>>
>>> Correct. Plus there is a full copy in SVN at the top-level.
>>>
>>> So removed as rev741.  Do you think you want to wrap this up as 0.5.1 or
>>> 0.5.5 or ... for CRAN?
>>>
>>> Dirk
>>>
>>>
>>> | Dirk Eddelbuettel wrote:
>>> | > Hi phylobase'ers,
>>> | >
>>> | > On 8 February 2010 at 17:54, Dirk Eddelbuettel wrote:
>>> | > | On 8 February 2010 at 15:40, Peter Cowan wrote:
>>> | > | | It appears to be building fine with Mac OS X (i386 & x86_64).  I agreed the
>>> | > | | commingled approach is the way to go.  I think this will suit the needs of
>>> | > | | our users best.  In fact, I tried it months ago but didn't have success, so I
>>> | > | | thank you for your efforts!
>>> | > |
>>> | > | Cool! I didn't know that, so thanks for the thumbs up!
>>> | > |
>>> | > | | From my understanding, we don't need all of the source files for our NCL commands.
>>> | > |
>>> | > | Correct!  But I wanted to be careful here. But in fact I think now that move
>>> | > | the old
>>> | > |
>>> | > |   src/ncl/
>>> | > |
>>> | > | into
>>> | > |
>>> | > |         src/ncl.original/
>>> | > |
>>> | > | and moved the files we need out of it, we can in fact clean up via, say,
>>> | > |
>>> | > |         $ svn rm pkg/src/ncl.original
>>> | > |         $ svn ci pkg -m'remove unused parts of ncl library'
>>> | > |
>>> | > | but I think I should leave that to one of the actual phylobase maintainers.
>>> | >
>>> | > I think we can do
>>> | >
>>> | >           $ svn rm pkg/src/ncl.original
>>> | >
>>> | > to clean up a little more.  Comments, anyone?
>>> | >
>>> | > Dirk
>>> | >
>>> | > | | Down the road, we might be able to clean things up a little.  For example
>>> | > | | the nxsdistancedatum.cpp file is empty
>>> | > |
>>> | > | Hadn't noticed.
>>> | > |
>>> | > | | and nxsemptyblock.cpp appears not to be used either.
>>> | > |
>>> | > | That one actually blocks building which is why I #ifdef'ed it out.
>>> | > |
>>> | > | Oh, the sweet sound of progress :)
>>> | > |
>>> | > | Dirk
>>> | > |
>>> | > | --
>>> | > |   Registration is open for the 2nd International conference R / Finance 2010
>>> | > |   See http://www.RinFinance.com for details, and see you in Chicago in April!
>>> | >
>>> |
>>> |
>>> | --
>>> | Ben Bolker
>>> | Associate professor, Biology Dep't, Univ. of Florida
>>> | bolker at ufl.edu / people.biology.ufl.edu/bolker
>>> | GPG key: people.biology.ufl.edu/bolker/benbolker-publickey.asc
>>> |
>>> |
>>> | [GNUPG:] ERRSIG 7395291A3C337A73 17 2 01 1267297727 9
>>> | [GNUPG:] NO_PUBKEY 7395291A3C337A73
>>>
>>> --
>>>  Registration is open for the 2nd International conference R / Finance 2010
>>>  See http://www.RinFinance.com for details, and see you in Chicago in April!
>>> _______________________________________________
>>> Phylobase-devl mailing list
>>> Phylobase-devl at lists.r-forge.r-project.org
>>> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/phylobase-devl
>>>
>> _______________________________________________
>> Phylobase-devl mailing list
>> Phylobase-devl at lists.r-forge.r-project.org
>> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/phylobase-devl
>
>


More information about the Phylobase-devl mailing list