On 16 December 2015 at 07:36, Hadley Wickham wrote:
| I'd be happy to accept help to do this conditionally on linux, but it
| needs to be bundled for windows and mac.
I'll take you up on that. It didn't take Jelmer and me all that many
trials. If you're happy to support this, I can work on some configure logic
to use the system rapidxml if found, and revert to locally building it as OS
X and Windows will do anyway.
Dirk
| Hadley
|
| On Wed, Dec 16, 2015 at 5:20 AM, Dirk Eddelbuettel <edd@debian.org> wrote:
| >
| > Hi Andrew,
| >
| > On 16 December 2015 at 11:41, Andrew Shadura wrote:
| > | Package: r-cran-readxl
| > | Version: 0.1.0-1
| > | Severity: normal
| > |
| > | Dear Maintainer,
| > |
| > | I have packaged rapidxml as a separate package. Please use that
| > | instead of bundling its code with your package.
| >
| > I agree with you on a theoretical level ("hey, wouldn't it be nice") -- but
| > will not do this from a practial, software-engineering standpoint as it is
| > more important to have R packages built reliably and identically on all
| > platforms.
| >
| > So I will not 'fork' this for Debian and build it differently just here.
| > This is an upstream issue of _optionally_ relying on system libraries where
| > present. This can be done, and I helped e.g. Jelmer do just that by using
| > system libnlopt if it is found. Hence I am passing this on to upstream, and
| > marking it forwarded. It would be great if you could help, I am afraid I may
| > not have time for it.
| >
| > Cheers, Dirk
| >
| >
| > | --
| > | Cheers,
| > | Andrew
| >
| > --
| > http://dirk.eddelbuettel.com | @eddelbuettel | edd@debian.org
|
|
|
| --
| http://had.co.nz/