Bug 143863 - imlib2 build failed on x86_64
Summary: imlib2 build failed on x86_64
Alias: None
Product: Fedora
Classification: Fedora
Component: imlib2   
(Show other bugs)
Version: 3
Hardware: All Linux
Target Milestone: ---
Assignee: Dams
QA Contact:
Depends On:
TreeView+ depends on / blocked
Reported: 2004-12-30 09:31 UTC by Thorsten Leemhuis
Modified: 2007-11-30 22:10 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2005-01-17 10:55:42 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Patch for imlib2.spec that fixes build on x86_64 (1.55 KB, patch)
2004-12-30 09:31 UTC, Thorsten Leemhuis
no flags Details | Diff
fix x86_64 issues, update to 1.2.0 (4.31 KB, patch)
2005-01-09 19:03 UTC, Thorsten Leemhuis
no flags Details | Diff

Description Thorsten Leemhuis 2004-12-30 09:31:15 UTC
imlib2 did not build for pre-extras x86_64. The attached patch fixes this.
Tested build on x86_64 and i386.

Comment 1 Thorsten Leemhuis 2004-12-30 09:31:15 UTC
Created attachment 109173 [details]
Patch for imlib2.spec that fixes build on x86_64

Comment 2 Rex Dieter 2005-01-04 13:48:28 UTC
For the record, I was able to build imlib2 on fc2/x86_64 without this

Comment 3 Michael Schwendt 2005-01-04 15:44:22 UTC
Exactly the same version as found in CVS? Because this is how it failed:

Comment 4 Matthias Saou 2005-01-05 18:29:53 UTC
Your failure is due to the --enable-mmx, which doesn't want to work on
x86_64 unfortunately. You didn't put --with-pic either, and I think
it'll also be needed.
OTOH, those libdir being overridden in the above patch were never
needed for me. Basically, what works for me is (I don't have any ia64
to test ;-)) :

%configure \
    --with-pic \
%ifarch %{ix86}
%{__make} %{?_smp_mflags}

This also works on ppc.

Comment 5 Thorsten Leemhuis 2005-01-09 14:50:47 UTC
Re 4 From Matthias Saou  on 2005-01-05 13:29 
> Your failure is due to the --enable-mmx, which doesn't want to work on
> x86_64 unfortunately. You didn't put --with-pic either, and I think
> it'll also be needed.

??? That's all covered by the patch already ;-)

>OTOH, those libdir being overridden in the above patch were never
>needed for me.

Otherwise it tried to install to /usr/lib here; see


Comment 6 Thorsten Leemhuis 2005-01-09 19:03:21 UTC
Created attachment 109538 [details]
fix x86_64 issues, update to 1.2.0 

Created an attachment (id=109537)
fix x86_64 issues and update to 1.2.0

Updated package:
  MD5:	5f664997927eb02a4b25da67ee187585



- Ship .la files ue to a bug in kdelibs; see
- Use make param LIBTOOL=/usr/bin/libtool - fixes hardcoded rpath on x86_64
- fix hardcoded rpath im Makefiles on x86_64 due to freetype-config --libs
returning "-L/usr/lib64 -Wl,--rpath -Wl,/usr/lib64 -lfreetype -lz"
- Update to 1.2.0 -- fixes several security issues
- remove explicit libdir=_libdir - 1.2.9 does not need it anymore
- removeddemo compile/install;
- use configure param --x-libraries={_prefix}/X11R6/{_lib} and patch to fix
"cannot find -lX11"

- Build tested on x86_64 and i386; did not have time for more yet

Comment 7 Michael Schwendt 2005-01-09 22:52:06 UTC
>  rm -f \
> -  $RPM_BUILD_ROOT%{_libdir}/libImlib2.la \
> -  $RPM_BUILD_ROOT%{_libdir}/imlib2_loaders/*/*.*a \
>    $RPM_BUILD_ROOT%{_bindir}/{color_spaces,imlib2,*test}

This change is a bit too much, since it puts back in the static
archives in /usr/lib/imlib2/filters/ and /usr/lib/imlib2/loaders/,
which are of no use.

Comment 8 Michael Schwendt 2005-01-10 08:08:48 UTC
Done a bit of Python metadata tool hacking: Matthias' giblib, libcaca
and camE packages seem to be the only imlib2 dependencies. All three
build with the 1.2.0 upgrade at least.

Comment 9 Thorsten Leemhuis 2005-01-10 17:40:58 UTC
Updated package:
  MD5:  3eb092ad3ccd138878553501ff8c81e3



- Don't ship *.?.a in {_libdir}/imlib/filters/ and loaders/

- Checked build with ffmpeg -- works also fine
- Is there somewhere a abicheck-(for-rpmbuilders-)howto? Or can I
safely assume there are no abi/api problems with already build
packages? Probably not...

Comment 10 Matthias Saou 2005-01-14 17:45:40 UTC
Why did you move the filter and loader libraries into the -devel
sub-package? Seems like a wrong thing to do.

Comment 11 Thorsten Leemhuis 2005-01-14 19:11:30 UTC
>Why did you move the filter and loader libraries into the -devel
>sub-package? Seems like a wrong thing to do.

That happend accidentally... Thanks for noticing.

Updated package:
  MD5:  90dd359c285df40067a9210ea2ed46bd



- Move filters and loaders back into main package where they belong

Comment 12 Michael Schwendt 2005-01-16 16:19:17 UTC
Thorsten, as pointed out in comment 8, Matthias' packages seem to be
the only ones which depend on imlib2. I have no means of testing them.
freshrpms still uses imlib2 1.1.2.

API compatibility can be tested with rebuilds, reading changelogs and
reading imlib2-devel rpmdiff against previous release. For testing ABI
compatibility, the "abicheck" tool (in package "abicheck") is helpful,
but experimental. At fedora.us we keep it in unstable, because it is a
large Perl script which breaks easily when output of glibc-tools and
binutils changes.

Comment 13 Thorsten Leemhuis 2005-01-16 16:42:55 UTC
Thanks Michael.

Just for the record; I had (and have) no intention to take over this
package ; I just wanted to fix x86_64 build issues ;-) But while at it
I thought I could do the update to 1.2.0 because of the security
issues mentioned in Bug 144596 . And also fix another fedora.us bug
(missing .la files, see http://bugzilla.fedora.us/show_bug.cgi?id=2284 ). 

So Anvil I think it's now up to you as maintainer. If you don't have
time just say and I'll try to bring the above package through
fedora.us QA.

Comment 14 Matthias Saou 2005-01-16 21:35:52 UTC
Two things :
1) We are nowhere near "production" for Extras, so if things seem ok
at first sight, just update and let any bug reports of breakage come
in later on.
2) I wouldn't mind taking ownership of this package, especially as you
mentionned, I already take care of all the only current ones that
depend on it.

Comment 15 Dams 2005-01-17 10:55:42 UTC
updated in cvs. Thanks.
I suppose cvs resolution is considered as rawhide..

Comment 16 Michael Schwendt 2005-01-18 20:46:55 UTC
* File imlib2-1.2.0.tar.gz Size 890457 STORED OK
* Submitted 1.2.0-4 to really fix fedora.us bug #2284.
* Going to request FC3 build.

Comment 17 Rex Dieter 2005-01-19 13:20:25 UTC
It appears the update also includes /usr/lib/libImlib2.la which *is*
safe to delete and IMO should not be included.

Comment 18 Rex Dieter 2005-01-19 13:21:13 UTC
Re: comment #2, my success was only with the older imlib2-1.1.2, so
not relavent here.

Note You need to log in before you can comment on or make changes to this bug.