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

Bug#694322: unblock: src:rgl/0.92.798-1



Hi Julien, hi Laszlo,

Probably not of much help, maybe for deciding, but I stumbled over
this looking at current issues for wheezy:

On Sun, Nov 25, 2012 at 09:52:03PM +0100, Julien Cristau wrote:
> On Sun, Nov 25, 2012 at 15:10:15 +0100, Laszlo Kajan wrote:
> 
> > Package: release.debian.org
> > Severity: normal
> > User: release.debian.org@packages.debian.org
> > Usertags: unblock
> > 
> > Please unblock package src:rgl
> > 
> > Dear Release Team!
> > 
> > This is Laszlo from BSP Munich addressing bug #684810.
> > 
> > Maintainer of rgl would have liked to have the fixed version (rgl/0.92.892-1) or the current (0.92.894-1) in unstable unblocked, but has forgotten to send the unblock request.
> > The changes are quite extensive (diffs are attached for both versions), but the maintainer says that 'Just push the updated package. These interfaces are stable -- the rgl package only does one thing -- and there should not be side effect.'
> > 
> > I have prepared a fix that addresses the bug (and does no more) - diff attached.
> > 
> > I see no versioned << <= == dependency on r-cran-rgl (binary pack build from rgl).
> > 
> > Please tell us: which diff should we - if any - turn into an upload to testing-proposed-updates?
> > 
> > rgl_0.92.798-1_0.92.798-1+deb7u1.diff
> > rgl_0.92.798-1_0.92.892-1.diff
> > rgl_0.92.798-1_0.92.894-1.diff
> > 
> Only the first of those might be acceptable.  Although the addition of
> debian/compat makes me kind of suspicious, as does the lintian override.
> Also, please don't send insanely big diffs by mail, otherwise mails are
> dropped by the list software (I was getting ready to remove this package
> from testing when I was pointed at this bug).

I tried to rebuild the package prepared by Laszlo without the added
debian/compat, so resulting debdiff attached. At first glance it looks
okay to me.

The lintian override probably was added, as the embedded-library tag
is listed in the autorejects list for ftp-masters[1].

 [1]: http://ftp-master.debian.org/static/lintian.tags

binary debdiff shows that a css file get's added to the rebuilded
package.

----cut---------cut---------cut---------cut---------cut---------cut-----
debdiff binary-r-cran-rgl/r-cran-rgl_0.92.798-1_amd64.deb r-cran-rgl_0.92.798-1+deb7u1_amd64.deb 
[The following lists of changes regard files as different if they have
different names, permissions or owners.]

Files in second .deb but not in first
-------------------------------------
-rw-r--r--  root/root   /usr/lib/R/site-library/rgl/html/R.css

Control files: lines which differ (wdiff format)
------------------------------------------------
Depends: libc6 (>= 2.2.5), libgcc1 (>= 1:4.1.1), libgl1-mesa-glx | libgl1, libglu1-mesa | libglu1, libpng12-0 (>= 1.2.13-4), libstdc++6 (>= [-4.1.1),-] {+4.6),+} libx11-6, r-base-core (>= 2.12.1)
Installed-Size: [-3224-] {+3146+}
Version: [-0.92.798-1-] {+0.92.798-1+deb7u1+}
----cut---------cut---------cut---------cut---------cut---------cut-----

Maintainer replied also [2] that if this does not "mesh with the rest
of
testing/stable, I'd say let us remove it".

 [2]: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=684810#36

----cut---------cut---------cut---------cut---------cut---------cut-----
$ dak rm -R -n -s testing rgl
Working... done.
Will remove the following packages from testing:

r-cran-rgl | 0.92.798-1 | amd64, armel, armhf, i386, ia64, kfreebsd-amd64, kfreebsd-i386, mips, mipsel, powerpc, s390, s390x, sparc
       rgl | 0.92.798-1 | source

Maintainer: Dirk Eddelbuettel <edd@debian.org>

------------------- Reason -------------------

----------------------------------------------

Checking reverse dependencies...
# Broken Depends:
misc3d: r-cran-misc3d
rcmdr: r-cran-rcmdr

Dependency problem found.
----cut---------cut---------cut---------cut---------cut---------cut-----

So if removal of rgl should happen, also some other packages needs to
be removed.

Regards,
Salvatore
diff -u rgl-0.92.798/debian/changelog rgl-0.92.798/debian/changelog
--- rgl-0.92.798/debian/changelog
+++ rgl-0.92.798/debian/changelog
@@ -1,3 +1,12 @@
+rgl (0.92.798-1+deb7u1) wheezy; urgency=low
+
+  * Non-maintainer upload.
+  * FTBFS: types.h:98:5: error: 'copy' was not declared in this scope, and no
+    declarations were found by argument-dependent lookup at the point of
+    instantiation [-fpermissive] (Closes: #684810).
+
+ -- Laszlo Kajan <lkajan@rostlab.org>  Sun, 25 Nov 2012 13:08:19 +0000
+
 rgl (0.92.798-1) unstable; urgency=low
 
   * New upstream release
only in patch2:
unchanged:
--- rgl-0.92.798.orig/src/types.h
+++ rgl-0.92.798/src/types.h
@@ -6,7 +6,7 @@
 // C++ header file
 // This file is part of RGL
 //
-// $Id: types.h 494 2006-08-01 17:25:31Z dmurdoch $
+// $Id: types.h 880 2012-05-08 11:54:12Z murdoch $
 
 
 #include <cstring>
@@ -80,6 +80,16 @@
 };
 
 //
+// mem copy
+//
+
+template<class A, class B>
+inline void copy(A* from, B* to, int size)
+{
+  memcpy( (void*) to, (const void*) from, size*sizeof(A) );
+       }
+
+//
 // TEMPLATE
 //   ARRAY
 //
@@ -105,16 +115,6 @@
 };
 
 //
-// mem copy
-//
-
-template<class A, class B>
-inline void copy(A* from, B* to, int size)
-{
-  memcpy( (void*) to, (const void*) from, size*sizeof(A) );
-}
-
-//
 // cast-copy doubles to floats
 //
 

Attachment: signature.asc
Description: Digital signature


Reply to: