[Debian-med-packaging] r4656 - trunk/packages/R/r-cran-epitools/trunk/debian

[Andreas Tille]

Hi Charles,

thanks for updating R-EpiTools.  Some changes probably need to be done
to all R packages.

On Tue, Mar 09, 2010 at 01:44:55PM +0000, Charles Plessy wrote:
> Modified: trunk/packages/R/r-cran-epitools/trunk/debian/compat
> ===================================================================
> --- trunk/packages/R/r-cran-epitools/trunk/debian/compat	2010-03-09 12:58:57 UTC (rev 4655)
> +++ trunk/packages/R/r-cran-epitools/trunk/debian/compat	2010-03-09 13:44:55 UTC (rev 4656)
> @@ -1 +1 @@
> -5
> +7

I wonder whether we generally should use 7 in all debian/compat files (I
was not aware that I used compat=5 in any of my packages) except if
something really speaks against this upgrade (no idea what this should
be).
 
>  Package: r-cran-epitools
>  Architecture: all
> -Depends: ${shlibs:Depends}, r-base-core
> +Depends: ${shlibs:Depends}, ${R:Depends}

Probably this is needed for all R related packages.

> ===================================================================
> --- trunk/packages/R/r-cran-epitools/trunk/debian/copyright	2010-03-09 12:58:57 UTC (rev 4655)
> +++ trunk/packages/R/r-cran-epitools/trunk/debian/copyright	2010-03-09 13:44:55 UTC (rev 4656)
> @@ -2,7 +2,7 @@
>  
>  Name: EpiTools
>  Contact: Tomas Aragon <aragon at berkeley.edu>
> -Source: http://cran.r-project.org/src/contrib/epitools_0.5-3.tar.gz
> +Source: http://cran.r-project.org/src/contrib/epitools_0.5-5.tar.gz

I wonder whether we should replace this by 

   Source: http://cran.r-project.org/src/contrib/

IMHO chances are really high to forget updating the debian/copyright
file at each new upstream version.

>  include /usr/share/R/debian/r-cran.mk
> +
> +# Require a number equal or superior than the R version the package was built with.
> +install/r-$(debRreposname)-$(cranName)::
> +	echo "R:Depends=r-base-core (>= $(shell R --version | head -n1 | perl -ne 'print / +([0-9]\.[0-9]+\.[0-9])/'))" >> debian/r-$(debRreposname)-$(cranName).substvars

I wonder whether this could be changed with a flag inside

    /usr/share/R/debian/r-cran.mk

which would come more handy than changing any R related rules file with this
code snippet.  If you agree we should report this to r-base-dev package.

Kind regards and thanks once more for the update

    Andreas.

-- 
http://fam-tille.de