[Pkg-parrot-devel] Parrot 4.3.0 with FTBFS fix, should we package it?

Sat Apr 21 12:22:18 UTC 2012

On Thu, Apr 19, 2012 at 07:54:01PM +0200, Dominique Dumont wrote:
> Le Wednesday 18 April 2012 11:12:21, Alessandro Ghedini a écrit :
> > so parrot 4.3.0 has been released with the FTBFS fix and AFAICT is a stable
> > release, should we package it or just backport the patch to 4.0.0?
> 
> Next rakudo-star should be out fairly soon (site says this month). It will 
> likely require parrot 4.3.0. 

Yes, both nqp and rakudo have been released last week, and they require the new
parrot. rakudo-star should follow soon (though we are not really interested in
it, since we package the bare rakudo).

> So I think you should prepare parrot 4.3.0 and upload it as soon as next 
> rakudo-star is available.

I'm not much confident in doing a new upstream release. Though I've prepared a
new upload which fixes the FTBFS and adds an initial implementation of the
parrotapi machinery (basically only the Provides part).

 parrot (4.0.0-2) unstable; urgency=low
 .
   * Team upload
   * parrot-minimal Provides parrotapi-X.Y.Z
     - parrot-devel Depends on parrotapi-X.Y.Z
   * Add 01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch to fix FTBFS
     on ia64, sparc and mipsel

(also see attached debdiff)

I've added the "Provides: parrotapi-X.Y.Z" in parrot-minimal (this is IMO better
since parrot already strictly depends on it), a "Depends: parrotapi-X.Y.Z" in
parrot-devel (which should be moved to the respective library packages once the
splitting proposed by Allison is done) and finally the alignment patch taken
from the upstream git repository as is.

This has IMO a couple of nice side effects: 1) it lets us test the parrotapi
thing "on the wild", so that we can fix any problem with the next upload and 2)
if everything works fine, we can remove the Breaks on rakudo from the next
upload (which should hopefully bring the new upstream release too).

Allison (and everyone), are you ok with this upload? (also note that the FTBFS
is preventing nqp and rakudo from building on those architectures too, so it
would be nice if we could get this uploaded ASAP).

Cheers

-- 
perl -E '$_=q;$/= @{[@_]};and s;\S+;<inidehG ordnasselA>;eg;say~~reverse'
-------------- next part --------------
diff -Nru parrot-4.0.0/debian/changelog parrot-4.0.0/debian/changelog

--- parrot-4.0.0/debian/changelog	2012-04-06 21:00:47.000000000 +0200
+++ parrot-4.0.0/debian/changelog	2012-04-21 13:12:34.000000000 +0200
@@ -1,3 +1,13 @@
+parrot (4.0.0-2) unstable; urgency=low
+
+  * Team upload
+  * parrot-minimal Provides parrotapi-X.Y.Z
+    - parrot-devel Depends on parrotapi-X.Y.Z
+  * Add 01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch to fix FTBFS
+    on ia64, sparc and mipsel
+
+ -- Alessandro Ghedini <ghedo at debian.org>  Sat, 21 Apr 2012 13:11:47 +0200
+
 parrot (4.0.0-1) unstable; urgency=low
 
   [ Allison Randal ]
diff -Nru parrot-4.0.0/debian/control parrot-4.0.0/debian/control
--- parrot-4.0.0/debian/control	2012-04-21 13:27:48.000000000 +0200
+++ parrot-4.0.0/debian/control	2012-04-21 13:15:07.000000000 +0200
@@ -24,6 +24,7 @@
 Depends: ${shlibs:Depends}, ${misc:Depends}
 Conflicts: parrot ( << 1.0.0 )
 Replaces: parrot ( << 1.0.0 )
+Provides: parrotapi-4.0.0
 Description: minimal install of the Parrot VM
  This package contains the core Parrot virtual machine. It's all that you need
  to get up and running in the average use case, and is sufficient to run most
@@ -58,7 +59,7 @@
 Package: parrot-devel
 Architecture: any
 Section: devel
-Depends: ${shlibs:Depends}, ${misc:Depends}, parrot, libparrot-dev, perl
+Depends: ${shlibs:Depends}, ${misc:Depends}, parrot, parrotapi-4.0.0, libparrot-dev, perl
 Provides: parrot-pge, parrot-tge, parrot-pct, parrot-nqp
 Conflicts: parrot ( << 1.0.0 ), libparrot-dev ( << 1.0.0 )
 Replaces: parrot ( << 1.0.0 ), libparrot-dev ( << 1.0.0 )
diff -Nru parrot-4.0.0/debian/control.in parrot-4.0.0/debian/control.in
--- parrot-4.0.0/debian/control.in	2012-04-06 21:11:34.000000000 +0200
+++ parrot-4.0.0/debian/control.in	2012-04-21 13:03:50.000000000 +0200
@@ -24,6 +24,7 @@
 Depends: ${shlibs:Depends}, ${misc:Depends}
 Conflicts: parrot ( << 1.0.0 )
 Replaces: parrot ( << 1.0.0 )
+Provides: parrotapi- at SOVERSION@
 Description: minimal install of the Parrot VM
  This package contains the core Parrot virtual machine. It's all that you need
  to get up and running in the average use case, and is sufficient to run most
@@ -58,7 +59,7 @@
 Package: parrot-devel
 Architecture: any
 Section: devel
-Depends: ${shlibs:Depends}, ${misc:Depends}, parrot, libparrot-dev, perl
+Depends: ${shlibs:Depends}, ${misc:Depends}, parrot, parrotapi- at SOVERSION@, libparrot-dev, perl
 Provides: parrot-pge, parrot-tge, parrot-pct, parrot-nqp
 Conflicts: parrot ( << 1.0.0 ), libparrot-dev ( << 1.0.0 )
 Replaces: parrot ( << 1.0.0 ), libparrot-dev ( << 1.0.0 )
diff -Nru parrot-4.0.0/debian/patches/01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch parrot-4.0.0/debian/patches/01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch
--- parrot-4.0.0/debian/patches/01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch	1970-01-01 01:00:00.000000000 +0100
+++ parrot-4.0.0/debian/patches/01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch	2012-04-21 13:10:17.000000000 +0200
@@ -0,0 +1,31 @@
+From 76fd663e109223c5a383fb622a43f42d21ef624e Mon Sep 17 00:00:00 2001
+From: Christoph Otto <christoph_github at mksig.org>
+Date: Mon, 16 Apr 2012 10:54:58 -0700
+Subject: [PATCH] fix alignment issues on ia64, sparc and mipsel, courtesy of
+ Alessandro Ghedini
+
+---
+ src/gc/fixed_allocator.h |    3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/src/gc/fixed_allocator.h b/src/gc/fixed_allocator.h
+index 8c2f1b8..2a7e2b3 100644
+--- a/src/gc/fixed_allocator.h
++++ b/src/gc/fixed_allocator.h
+@@ -28,11 +28,12 @@ src/gc/fixed_allocator.h - implementation of allocator for small-size objects.
+ 
+ typedef struct Pool_Allocator_Free_List {
+     struct Pool_Allocator_Free_List * next;
+-
++    char *dummy; /* fix alignment on ia64, mipsel and sparc, similar to gh issue #603 */
+ } Pool_Allocator_Free_List;
+ 
+ typedef struct Pool_Allocator_Arena {
+     struct Pool_Allocator_Arena * next;
++    char *dummy; /* fix alignment on ia64, mipsel and sparc, similar to gh issue #603 */
+ } Pool_Allocator_Arena;
+ 
+ typedef struct Pool_Allocator {
+-- 
+1.7.10
+
diff -Nru parrot-4.0.0/debian/patches/series parrot-4.0.0/debian/patches/series
--- parrot-4.0.0/debian/patches/series	2012-04-06 20:58:35.000000000 +0200
+++ parrot-4.0.0/debian/patches/series	2012-04-21 13:11:07.000000000 +0200
@@ -1,2 +1,3 @@
+01_fix_alignment_issues_on_ia64_sparc_and_mipsel.patch
 02_fix_perl_interpreter_path.patch
 03_fix_nqp_man.patch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-parrot-devel/attachments/20120421/c29875bf/attachment.pgp>
