[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#898861: Please replace non-free RJSONIO in rsconnect by jsonlite



Hi Jonathan,

in your package rsconnect you are using RJSONIO which has a non-free license
(BSD + evil clause, see here[1]).  I discussed this previously with the author
of googleVis who has managed to replace RJSONIO by jsonlite[2].

For the sake of testing I replaced RJSONIO by jsonlite in the
DESCRIPTION file[3], built the Debian package and was running test suite.
While this throws an error (which I would like to clarify as well) I do
not think that this is related to any JSON library:

BEGIN TEST testthat.R

R version 3.4.4 (2018-03-15) -- "Someone to Lean On"
Copyright (C) 2018 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

> library(testthat)
> library(rsconnect)
> 
> test_check("rsconnect")
-- 1. Failure: The linter identifies files not matching in case sensitivity (@te
filepath.capitalization$indices == 31 isn't true.

== testthat results  ===========================================================
OK: 178 SKIPPED: 6 FAILED: 1
1. Failure: The linter identifies files not matching in case sensitivity (@test-lint.R#50) 

Error: testthat unit tests failed
Execution halted


Can you please have a look?

Kind regards

        Andreas.


[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=712159#22
[2] https://alioth-lists.debian.net/pipermail/debian-science-maintainers/2016-July/041580.html
[3] https://salsa.debian.org/r-pkg-team/r-cran-rsconnect/blob/master/debian/patches/replace_non-free_rjsonio_by_jsonlite.patch

-- 
http://fam-tille.de


Reply to: