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

Bug#681110: unblock: sdl-stretch/0.3.1-3



2012/7/17 Julien Cristau <jcristau@debian.org>:
>> There's an unblock already for sdl-stretch/0.3.1-2 because it was in
>> unstable before the freeze, but -2 failed to build in kfreebsd-i386
>> and thus it never migrated to testing:
>>
>>   https://buildd.debian.org/status/logs.php?pkg=sdl-stretch&arch=kfreebsd-i386
>>
>> I produced -3 and asked the package to be removed from
>> "Packages-arch-specific" (see #680275) so it would build in
>> kfreebsd-amd64 and amd64 [1], but we were asked to restrict the
>> architectures ourselves first (any-amd64 any-386) in a new
>> revision/upload of the package before granting the removal.
>>
>> The package doesn't have rdepends, so this unblock shoudln't cause much impact:
>>
> The package doesn't have rdepends, so the lack of an unblock shouldn't
> cause much impact.  Are there users requesting this package on kbsd64,
> or another reason why this is particularly important to add now?

The situation is a bit messy, I hope that I can explain myself in the
next few paragraphs.

I figured out that if -2 was to be allowed, -3 could be as well; and
it's not only kfreebsd-amd64 but amd64 itself, which is probably the
most important architecture by now.

The thing is, sdl-stretch is a relatively unimportant package in the
SDL-team realm, so we attended other packages first (most were
abandoned by the end of 2011).  Recently I went to look why
sdl-stretch wasn't migrating, and I discovered that it had a
restriction in "packages-arch-specific" that was restricting it for
386 only.  I checked with upstream and the package is actually
amd64-capable (it contains assembler routines, so it's not available
for any actitectures), but it was restricted to 386 in Debian since
2005 or so.

The amd64 package is actually present because we built it with
pbuilder in amd64 arch and uploaded it, but if the package was to be
rebuilt when the P-A-S restriction was in place, it wouldn't build in
amd64 in the buildds.

OTOH, -2 was not restricted in debian/control to amd64 and i386
architectures (which is the thing that I fixed in -3), so if it's
rebuilt now, it will try to be built in all architectures, and in some
of them it will probably fail, thus causing a FTBFS later in the
freeze period or beyond (once released).

So I think that the best thing to do is to allow -3 to go in, the only
actual change it's the restriction on architectures (see debdiff
attached), and it was requested before buildd team accepted to remove
it from the P-A-S file:

  http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=680275


Cheers.
-- 
Manuel A. Fernandez Montecelo <manuel.montezelo@gmail.com>
diff -Nru sdl-stretch-0.3.1/debian/changelog sdl-stretch-0.3.1/debian/changelog
--- sdl-stretch-0.3.1/debian/changelog	2012-04-26 15:51:26.000000000 +0100
+++ sdl-stretch-0.3.1/debian/changelog	2012-07-04 20:25:34.000000000 +0100
@@ -1,3 +1,22 @@
+sdl-stretch (0.3.1-3) unstable; urgency=low
+
+  * Change "Architecture: any-amd64 any-i386" for both binary packages.
+
+    This package is a bit special because it contains optimised routines in
+    assembler for some architectures, so even if it can be built successfully in
+    others, it doesn't really make sense to provide the package.  According to
+    upstream, current models of both i386 and amd64 are supported, and while
+    it's desired to have support in other platforms (e.g. ARM), it's not there
+    yet.
+
+    Before 0.3.1-1, debian/control was set only to build on i386; and in 0.3.1-1
+    it was set to "any", but the file
+    https://buildd.debian.org/quinn-diff/sid/Packages-arch-specific restricted
+    the architectures effectively to "i386 kfreebsd-i386 hurd-i386" (see
+    #680275).
+
+ -- Manuel A. Fernandez Montecelo <manuel.montezelo@gmail.com>  Wed, 04 Jul 2012 20:12:03 +0100
+
 sdl-stretch (0.3.1-2) unstable; urgency=low
 
   * Bump Standards-Version to 3.9.3 (no changes needed)
diff -Nru sdl-stretch-0.3.1/debian/control sdl-stretch-0.3.1/debian/control
--- sdl-stretch-0.3.1/debian/control	2012-04-26 15:51:35.000000000 +0100
+++ sdl-stretch-0.3.1/debian/control	2012-07-04 20:07:19.000000000 +0100
@@ -17,7 +17,7 @@
 Homepage: http://sdl-stretch.sourceforge.net
 
 Package: libsdl-stretch-0-3
-Architecture: any
+Architecture: any-amd64 any-i386
 Multi-Arch: same
 Pre-Depends: ${misc:Pre-Depends}
 Depends: ${shlibs:Depends},
@@ -33,7 +33,7 @@
 
 Package: libsdl-stretch-dev
 Section: libdevel
-Architecture: any
+Architecture: any-amd64 any-i386
 Depends: ${misc:Depends},
          libsdl-stretch-0-3 (= ${binary:Version}),
          libsdl1.2-dev(>= 1.2.14~)

Reply to: