Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 33777 - Unnecessary ARCH check in net-mail/ximian-connector-1.2.2_p1
Summary: Unnecessary ARCH check in net-mail/ximian-connector-1.2.2_p1
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: Normal critical
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-18 12:20 UTC by Sven Blumenstein (RETIRED)
Modified: 2003-11-20 03:28 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Said patch. Yes, its small I know ;) (ximian-connector-1.2.2_p1.ebuild.patch,340 bytes, patch)
2003-11-18 12:24 UTC, Sven Blumenstein (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Blumenstein (RETIRED) gentoo-dev 2003-11-18 12:20:10 UTC
net-mail/ximian-connector-1.2.2_p1 does check for a supported ARCH and dies if the current ARCH is unsupported:

!!! ERROR: net-mail/ximian-connector-1.2.2_p1 failed.
!!! Function , Line 22, Exitcode 1
!!! Unsupported ARCH: 

The part of the ebuild:

if [ `use ppc` ]; then 
    XIMIAN_DIST="yellowdog-22-ppc"
    XIMIAN_ARCH="ppc"
elif [ `use x86` ]; then 
    XIMIAN_DIST="redhat-73-i386"
    XIMIAN_ARCH="i386"
else
    die "Unsupported ARCH: ${ARCH}"
fi


This check is unnecessary as the KEYWORDS already take care of filtering out unsupported ARCH's:

KEYWORDS="x86 ~ppc -sparc -alpha -mips"

Main problem is that this 'die' causes some tools to break, like for example esearch/eupdatedb:

root@darwin bazik # eupdatedb 
 * indexing: 1514 ebuilds to go
aux_get(): (0) Error in net-mail/ximian-connector-1.2.2_p1 ebuild.
               Check for syntax error or corruption in the ebuild. (--debug)


And repoman doesnt like the ebuild either. My suggested change attached as patch below.
Comment 1 Sven Blumenstein (RETIRED) gentoo-dev 2003-11-18 12:24:25 UTC
Created attachment 20923 [details, diff]
Said patch. Yes, its small I know ;)
Comment 2 Alastair Tse (RETIRED) gentoo-dev 2003-11-20 03:28:24 UTC
ok, should be fixed in the next couple of minutes