On 2011/06/23 22:55, Daniel Dickman wrote:
> >> - it links with -lpthread instead of -pthread in several places
> 
> hints appreciated here. this might take me a bit of time to work out.

I didn't look at this but configure will probably be checking which
threading library to use, this will just be a case of working out which
check matches (should have line numbers in config.log) and replacing
the relevant -lpthread with -pthread (i.e. patch configure).

> > This diff [...] corrects license marker,
> 
> i also originally had GPLv2+ because I read answer #1 from the the R
> FAQ too quickly (http://cran.r-project.org/doc/FAQ/R-FAQ.html). Later
> I realized the "or any later version" applies to that document not to
> R itself. The actual project license details are summarized here
> http://www.r-project.org/Licenses/ (and also in the FAQ as Q 2.11) so
> I think the original GPLv2 that I had is more correct. I've added a
> link to the license page in the latest patch.

Ah - I checked a few source code files but didn't check every
single one. Link to the license page is a good idea.

> > ... and enables the shared lib.
> 
> So, enabling the shared lib give me the following which I haven't
> figured out how to fix yet --
> 
>     $ make update-plist
>     ...
>     WARNING: version mismatch for lib: R (212.2 vs. 0.0)

hmm, I thought I'd fixed that but obviously not - I'll take another
look at that.

Reply via email to