2014-10-04 0:40 GMT+02:00 David Sanders <da...@sandersweb.net>:
>>-----Original Message-----
>>From: Agustin Martin [mailto:agmar...@debian.org]
>>Sent: Friday, October 03, 2014 12:54 PM
>>To: David Sanders; 763...@bugs.debian.org
>>Subject: Re: Bug#763892: aspell-en needs to be Multi-Arch: foreign
>>
>>The problem I see above is in aspell-no, which does not use
> aspell-autobuildhash, and >is an arch:any package that might not work with
> multiarch. In this particular case, pulled >aspell-no:i386 will use 32 bit
> sizes and is usable with libaspell15:amd64, but aspell->no:amd64 will not
> work with current aspell because it was built with 64 bit numbers
>>
>>IIRC, Debian aspell maintainer, Brian Nelson, set as policy that all aspell
> dictionaries >must be automatically built at postinst stage, but do not have
> the reference here.
>>
>>I am cloning this bug report to aspell-no. I think it should use
> aspell-autobuildhash. As a >lesser evil, should be rebuilt with new aspell
> to make sure 32 bit numbers are always >used in hash creation and set
> appropriate versioned dependency on aspell, although >might show some exotic
> behavior during installation, like the one seen above.
>>
>>We might later conclude that all aspell:all dicts need to be set as
>>"Multi-Arch: foreign", but we should discard aspell-no influence first.
>>
>
> But back to aspell-en.  I stand behind my theory that aspell-en and the
> other aspell-xx packages need to be Multi-Arch: foreign.  I should have
> stated in my original bug report that this was my THEORY.  I guess I didn't
> have my morning coffee.

Hi, David,

I stand corrected, aspell-en must be declared "Multi-Arch: foreign"
for jessie, and it is better that other aspell-xx packages also are,
but not so important

Noticed that I wrongly tested with

# apt-get install aspell:i386 libaspell15 --install-recommends

instead of with

# apt-get install aspell:i386 --install-recommends

as I should have done. So, aspell-en is pulled in the first case, but
not in the second.

That means that aspell-en should really be "multiarch: foreign."  For
other dicts is better if they are also set foreign, for instance, if I
run, with a "multiarch: foreign" aspell-en available (built one for
the test),

# apt-get install aspell:i386 aspell-es --install-recommends

aspell-en will also be pullled together with aspell-es, althouth
aspell-es alone should have matched the depends. A bit weird, but not
that grave for jessie release. Anyway, better if addressed too.

Brian, if you do not object I plan to upload next Monday an aspell-en
NMU to DELAYED/2 with attached patch. I am aware that this is a very
short time frame, but I think we should deal with this ASAP.

> You have identified an additional problem that I did not see.  That is that
> aspell-no is not compatible with multi-arch as it stands right now.  This
> problem also applies to aspell-da which has the same problem.  Bugs with
> "important" priority need to be filled against these two packages to they
> can be fixed before the next release.

I already cloned this bug report to aspell-no (normal severity by
now). I will also file a bug against aspell-da about this. I was
pretty sure that Brian stated as policy that all aspell dicts must use
aspell-autobuildhash, but searching now I can only find that
aspell-autobuildhash was tagged as the preferred form for aspell
dicts.

Brian, we recently made some changes to dictionaries-common-dev (see
http://bugs.debian.org/737515) regarding a very simple debhelper
structure to handle aspell dictionaries directly based on upstream
files. I used aspell-en as one of the test boxes, so I will provide
you with a diff and a wishlist bug for your consideration.

Thanks a lot for your feedback, David,

Regards,

-- 
Agustin
diff --git a/debian/changelog b/debian/changelog
index 307d4ee..735a169 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+aspell-en (7.1-0-1.1) unstable; urgency=low
+
+  * Non-Maintainer Upload.
+  * Make aspell-en "Multiarch: Foreign" (Closes: #763892).
+
+ -- Agustin Martin Domingo <agmar...@debian.org>  Sat, 04 Oct 2014 20:39:19 
+0200
+
 aspell-en (7.1-0-1) unstable; urgency=low
 
   * New upstream release (Closes: #632023)
diff --git a/debian/control b/debian/control
index bc8b4fb..c35c19b 100644
--- a/debian/control
+++ b/debian/control
@@ -8,6 +8,7 @@ Homepage: http://aspell.net/
 
 Package: aspell-en
 Architecture: all
+Multi-Arch: foreign
 Depends: ${misc:Depends}, aspell (>= 0.60.3-2), dictionaries-common (>= 0.49.2)
 Provides: aspell-dictionary
 Description: English dictionary for GNU Aspell

Reply via email to