apt-mirror does not support armhf arch

[Fumihito YOSHIDA]

[Impact]
Cannot be used to mirror armhf, one of the supported architectures on some releases and especially LTS releases.

[Test Case]
# Configure apt-mirror to mirror armhf
$ sudo sed -i 's/archive.ubuntu.com\/ubuntu/ports.ubuntu.com\/ubuntu-ports/' /etc/apt/mirror.list
$ sudo sed -i 's/^deb /deb-armhf /' /etc/apt/mirror.list
# Change to root
$ sudo -i
# Attempt to start apt-mirror
$ su - apt-mirror -c apt-mirror

On failure you'll see:

Downloading 48 index files using 20 threads...
Begin time: Wed Aug 15 09:37:21 2012
[20]... [19]... [18]... [17]... [16]... [15]... [14]... [13]... [12]... [11]... [10]... [9]... [8]... [7]... [6]... [5]... [4]... [3]... [2]... [1]... [0]...
End time: Wed Aug 15 09:37:23 2012

Proceed indexes: [Psh: 1: cannot open ports.ubuntu.com/ubuntu-ports//dists/precise/main/binary-arm/Packages.gz: No such file
apt-mirror: can't open index in proceed_index_gz at /usr/bin/apt-mirror line 445.

[Regression Potential]
The most obvious risk with the upstream fix is that we break the regex for other architectures. I'd recommend making sure that apt-mirror still successfully mirrors i386/amd64 archs.

[Other Info]
This SRU request for the precise target.
The package to be built and released is from raring.

[dann frazier (dannf)]

This is still broken as of precise, but fixed in quantal.

Note that the fix applied for http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=644630 was insufficient, you also need the fix applied for http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=682395.

[dann frazier (dannf)]

Verified as fix-released in quantal as of 0.4.8-6. Dropping the "[lucid]" tag, as this affects everything up to precise.

[Philip Wyett (philip-wyett-deactivatedaccount)]

If you would like to test the fixed package rebuilt for precise and report the results. You can obtain it from my temporary PPA.

  https://launchpad.net/~philip-wyett/+archive/temporary

[Philip Wyett (philip-wyett-deactivatedaccount)]

Local testing shows the updated package in my PPA is working.

If we can get other confirmation. I will attempt to push the package as a Stable Release Update (SRU).

[Benjamin Drung (bdrung)]

Go for a SRU!

[Benjamin Drung (bdrung)]

I uploaded apt-mirror 0.4.8-5ubuntu0.1 to precise-proposed (patch attached).

[Dave Walker (davewalker)]

Hello Fumihito, or anyone else affected,

Accepted apt-mirror into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/apt-mirror/0.4.8-5ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Ubuntu Foundations Team Bug Bot (crichton)]

The fix for this bug has been awaiting testing feedback in the -proposed repository for precise for more than 90 days. Please test this fix and update the bug appropriately with the results. In the event that the fix for this bug is still not verified 15 days from now, the package will be removed from the -proposed repository.

[Brian Murray (brian-murray)]

Hello Fumihito, or anyone else affected,

Accepted apt-mirror into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/apt-mirror/0.4.8-5ubuntu0.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Fumihito YOSHIDA (hito)]

# First, sorry for my lazy work...

It seems still broken, verification failed.

Actual results:
--------------------------------------------------------------
$ apt-cache policy apt-mirror
apt-mirror:
  Installed: 0.4.8-5ubuntu0.2
  Candidate: 0.4.8-5ubuntu0.2
  Version table:
 *** 0.4.8-5ubuntu0.2 0
        500 http://jp.archive.ubuntu.com/ubuntu/ precise-proposed/universe amd64 Packages
        100 /var/lib/dpkg/status
     0.4.8-5 0
        500 http://jp.archive.ubuntu.com/ubuntu/ precise/universe amd64 Packages

$ grep armhf /usr/bin/apt-mirror
    if($config_line =~ /^deb-(alpha|amd64|armel|arm|armhf|hppa|hurd-i386|i386|ia64|kfreebsd-i386|kfreebsd-amd64|lpia|m68k|mipsel|mips|powerpc|s390|s390x|sh|sparc)$/) {

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

See https://bugs.launchpad.net/ubuntu/+source/apt-mirror/+bug/977278/comments/2

 * regexp pattern of "armel|arm|armhf" cause bad matching. Expected: "armel|armhf|arm"

[Benjamin Drung (bdrung)]

Fumihito, have you really tested to mirror a armhf archive?

The patched version matches against full lines (with ^ and $ at the beginning/end of the regular expression). The patch is exactly the same as the one mentioned in comment #2.

[Fumihito YOSHIDA (hito)]

My apologies. It divereted by my mistakes.

re-test with clean-installed precise with below configs, it works fine.

==================================
set nthreads 20
set _tilde 0

deb-armel http://ports.ubuntu.com/ precise main restricted universe multiverse
deb-armel http://ports.ubuntu.com/ precise-updates main restricted universe multiverse
deb-armel http://ports.ubuntu.com/ precise-backports main restricted universe multiverse
deb-armel http://ports.ubuntu.com/ precise-security main restricted universe multiverse
deb-armel http://ports.ubuntu.com/ precise-proposed main restricted universe multiverse
deb-armhf http://ports.ubuntu.com/ precise main restricted universe multiverse
deb-armhf http://ports.ubuntu.com/ precise-updates main restricted universe multiverse
deb-armhf http://ports.ubuntu.com/ precise-backports main restricted universe multiverse
deb-armhf http://ports.ubuntu.com/ precise-security main restricted universe multiverse
deb-armhf http://ports.ubuntu.com/ precise-proposed main restricted universe multiverse

clean http://ports.ubuntu.com/ubuntu
==================================

[Launchpad Janitor (janitor)]

This bug was fixed in the package apt-mirror - 0.4.8-5ubuntu0.2

---------------
apt-mirror (0.4.8-5ubuntu0.2) precise-proposed; urgency=low

  * Fix regex problem: arm overrides armhf. Thanks to hawken.
    (Closes: #682395, LP: #977278)
  * Add support for HTTPS with basic HTTP authentication. Thanks to Pali
    for the patch. (LP: #1015131)
 -- Benjamin Drung <email address hidden> Fri, 22 Nov 2013 00:03:34 +0100

[Brian Murray (brian-murray)]

The verification of the Stable Release Update for apt-mirror has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.