Bug#754715: Debian claims source missing for rgl

2014-07-21 Thread Duncan Murdoch
On 21/07/2014, 6:21 AM, Dirk Eddelbuettel wrote:
 
 On 19 July 2014 at 22:28, Dirk Eddelbuettel wrote:
 | 
 | On 19 July 2014 at 20:43, Duncan Murdoch wrote:
 | | On 19/07/2014, 7:18 PM, Dirk Eddelbuettel wrote:
 | |  
 | |  On 19 July 2014 at 17:48, Duncan Murdoch wrote:
 | |  | On 19/07/2014, 2:52 PM, Bastien ROUCARIES wrote:
 | |  |  On Sat, Jul 19, 2014 at 4:11 PM, Dirk Eddelbuettel 
 e...@debian.org wrote:
 | |  |  Duncan,
 | |  | 
 | |  |  The Debian package linter claims that the included file
 | |  | 
 | |  |  inst/WebGL/CanvasMatrix.js
 | |  | 
 | |  |  in your rgl package lacks source code.
 | |  | 
 | |  |  I know next to nothing about Javascript, and to me Javascript 
 always appeared
 | |  |  to _be_ source code.
 | |  |  No ti is minified javascript, not source code. You lose comments and
 | |  |  variable name.
 | |  | 
 | |  | It's a compiled Javascript file -- minified, as Bastien says.  The
 | |  | docs describe where to get the original source, and have a copy of the
 | |  | copyright notice.  It's a very commonly included file in WebGL pages.
 | |  | 
 | |  | I'd rather not include the source, as it is redundant, and easily
 | |  | available elsewhere.
 | |  
 | |  The powers-that-be and sticklers within Debian may then throw it out of 
 the archive.
 | | 
 | | I don't really care too much about including the source, but I
 | | definitely can't remove the compiled version.  Is it sufficient to
 | | have both?
 | 
 | Yes, I am asked to provide the source in the sources we use for building.  
 | 
 | Sorry if I was unclear, I was _not_ asking you to replace the minimized
 | source with the full source, but the source for everything Policy wants 
 the
 | source along with the version you deploy :-/
 
 That said, the problem is of course more between me and the Debian packaging
 (unless CRAN eventually follows suit and also asks for source on .js; I am
 not suggesting they do).
 
 Do you have a canonical URL for a source of that file CanvasMatrix.js ?

I've added the source file to the package on R-forge.  Unfortunately,
it's currently in an unstable state (a segfault on OSX), so I can't push
this change to CRAN yet.  Once I track down that bug and I'm happy it's
stable again, I'll let you know.

If you want to put in a link to the file before that, this one should work:

https://r-forge.r-project.org/scm/viewvc.php/pkg/rgl/inst/WebGL/CanvasMatrix.src.js?root=rglview=log

I can't guarantee that R-forge won't change its location, but the
rgl/inst/WebGL/CanvasMatrix.src.js part should be stable.

Duncan


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#754715: Debian claims source missing for rgl

2014-07-19 Thread Duncan Murdoch
On 19/07/2014, 2:52 PM, Bastien ROUCARIES wrote:
 On Sat, Jul 19, 2014 at 4:11 PM, Dirk Eddelbuettel e...@debian.org wrote:
 Duncan,

 The Debian package linter claims that the included file

 inst/WebGL/CanvasMatrix.js

 in your rgl package lacks source code.

 I know next to nothing about Javascript, and to me Javascript always appeared
 to _be_ source code.
 No ti is minified javascript, not source code. You lose comments and
 variable name.

It's a compiled Javascript file -- minified, as Bastien says.  The
docs describe where to get the original source, and have a copy of the
copyright notice.  It's a very commonly included file in WebGL pages.

I'd rather not include the source, as it is redundant, and easily
available elsewhere.

Duncan Murdoch
 The GNU file tool seems to agree

 edd@max:~/src/debian/CRAN/rgl-0.93.1098$ file inst/WebGL/CanvasMatrix.js
 inst/WebGL/CanvasMatrix.js: ASCII text, with very long lines
 edd@max:~/src/debian/CRAN/rgl-0.93.1098$

 Do you have a good answer?

 I am CCing the (to me at least somewhat overzealous) Debian colleague who
 filed this; maybe he has added insights. I am also ccing the bug report for
 process; records are at http://bugs.debian.org/754715

 Dirk

 --
 http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#754715: Debian claims source missing for rgl

2014-07-19 Thread Duncan Murdoch
On 19/07/2014, 7:18 PM, Dirk Eddelbuettel wrote:
 
 On 19 July 2014 at 17:48, Duncan Murdoch wrote:
 | On 19/07/2014, 2:52 PM, Bastien ROUCARIES wrote:
 |  On Sat, Jul 19, 2014 at 4:11 PM, Dirk Eddelbuettel e...@debian.org 
 wrote:
 |  Duncan,
 | 
 |  The Debian package linter claims that the included file
 | 
 |  inst/WebGL/CanvasMatrix.js
 | 
 |  in your rgl package lacks source code.
 | 
 |  I know next to nothing about Javascript, and to me Javascript always 
 appeared
 |  to _be_ source code.
 |  No ti is minified javascript, not source code. You lose comments and
 |  variable name.
 | 
 | It's a compiled Javascript file -- minified, as Bastien says.  The
 | docs describe where to get the original source, and have a copy of the
 | copyright notice.  It's a very commonly included file in WebGL pages.
 | 
 | I'd rather not include the source, as it is redundant, and easily
 | available elsewhere.
 
 The powers-that-be and sticklers within Debian may then throw it out of the 
 archive.

I don't really care too much about including the source, but I
definitely can't remove the compiled version.  Is it sufficient to
have both?

Duncan

 
 Dirk
 
 | 
 | Duncan Murdoch
 |  The GNU file tool seems to agree
 | 
 |  edd@max:~/src/debian/CRAN/rgl-0.93.1098$ file 
 inst/WebGL/CanvasMatrix.js
 |  inst/WebGL/CanvasMatrix.js: ASCII text, with very long lines
 |  edd@max:~/src/debian/CRAN/rgl-0.93.1098$
 | 
 |  Do you have a good answer?
 | 
 |  I am CCing the (to me at least somewhat overzealous) Debian colleague who
 |  filed this; maybe he has added insights. I am also ccing the bug report 
 for
 |  process; records are at http://bugs.debian.org/754715
 | 
 |  Dirk
 | 
 |  --
 |  http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org
 | 
 


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#361968: [Rd] Bug#361968: Wrong name in manpage

2006-04-11 Thread Duncan Murdoch

On 4/11/2006 7:45 PM, Dirk Eddelbuettel wrote:
R-devel'ers, 


On 11 April 2006 at 10:24, Dirk Eddelbuettel wrote:
| 
| Kanru,
| 
| Thanks for the bugreport.
| 
| On 11 April 2006 at 22:03, Kanru Chen wrote:

| | Package: r-base-core
| | Version: 2.2.1.svn37668-1
| | Severity: minor
| | 
| | In manpage of /usr/bin/R, the first, fourth and last line shows `VERSION'

| | instead of `R'.

I haven't seen any follow-up yet -- here is what it looks like (cut and
pasted from Emacs man page viewer) and note the 'Version' in place of R:

VERSION(1)FSF   VERSION(1)

NAME
   Version - a language for data analysis and graphics

SYNOPSIS
   R [options] [ infile] [ outfile]
   R CMD command [arguments]

DESCRIPTION
   Start  R,  a  system for statistical computation and graphics, with the
   specified options, or invoke an R tool via the 'R CMD' interface.
[...]


| | I believe it is a typo.
| 
| More likely something is wrong with how R.1 is autogenerated using help2man.
| 
| Incidentally, that `R --version' now starts its ouput with 'Version' rather

| than R had bit us in the RPy builds where the version number was regexp'ed
| out of the result, and was still expecting the line to start with R just like
| help2man seems to expect the program name first. 


It seems to stem from src/main/version.c:

void attribute_hidden PrintVersionString(char *s)
{
if(strcmp(R_SVN_REVISION, unknown)==0)
{
sprintf(s, Version %s.%s %s (%s-%s-%s),
R_MAJOR, R_MINOR, R_STATUS, R_YEAR, R_MONTH, R_DAY);
}
else{
if(strlen(R_STATUS)==0){
sprintf(s, Version %s.%s (%s-%s-%s),
R_MAJOR, R_MINOR, R_YEAR, R_MONTH, R_DAY);
}
else{
sprintf(s, Version %s.%s %s (%s-%s-%s r%s),
R_MAJOR, R_MINOR, R_STATUS, R_YEAR, R_MONTH, R_DAY,
R_SVN_REVISION);
}
}
}

Would replacing 'Version ...' with 'R (Version ...)' be an acceptable
alternative ?


I think the problem is that PrintVersion (a few lines up from there) 
used to put the R in front; PrintVersionString doesn't include the R. 
PrintVersionString is called from other places where the R would not be 
appropriate, but PrintVersion is only called when acting on `R 
--version' or synonyms.


Fritz, I think this was your change in r36923 a few months ago.  Do you 
have time to deal with it?


Duncan



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]