Bug 145882

Summary: gcc >= 3.3 deprecated multi-line asm breaks test for assembler
Product: [Fedora] Fedora Reporter: Arenas Belon, Carlo Marcelo <carenas>
Component: lzoAssignee: Hans de Goede <hdegoede>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: matthias
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://bugs.gentoo.org/show_bug.cgi?id=28359
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-07-25 20:00:19 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch for configure.in to fix the assembler tests none

Description Arenas Belon, Carlo Marcelo 2005-01-23 04:52:12 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.7.5)
Gecko/20041110 Firefox/1.0

Description of problem:
patch applies to macros for configure.in and will require that
configure be recreated (autoconf).



Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. tar -xvzf lzo-1.08.tar.gz
2. cd lzo-1.08
3. ./configure

Actual Results:  checking the alignment of the assembler... 0
checking whether to build assembler versions... no

Expected Results:  checking the alignment of the assembler... 4
checking wheter global symbols need underscores... no
checking wheter the assembler understands .type... yes
checking wheter the assembler works... yes
checking wheter to build assembler versions... yes

Additional info:

the only "affected" architecture should be i386 as it is the only one
which currently has assembler code which could be used, eventhough all
architectures fail to detect the working assembler because of the
broken test.

fc3 has no workaround, for all others using the old compiler (from the
compat packages) could be considered as a workaround.
Comment 1 Arenas Belon, Carlo Marcelo 2005-01-23 04:54:33 UTC 
Created attachment 110100 [details]
patch for configure.in to fix the assembler tests
Comment 2 Matthias Saou 2005-02-01 12:15:14 UTC 
I can confirm that this patch fixes the assembler detection on i386,
and the package then builds fine with asm enabled on x86, and still
builds fine (with asm disabled) on x86_64.
Comment 3 Hans de Goede 2006-07-25 20:00:19 UTC 
I'm taking over as maintainer from Anvil, since Anvil has other priorities.

Thanks for the patch.

1.08-7 should show up on a mirror near you soon, containing a modified version
of your patch, fixing this. The changes in 1.08-7 are:
* Mon Jul 24 2006 Hans de Goede <j.w.r.degoede> 1.08-7
- Taking over as maintainer since Anvil has other priorities
- Add a patch to fix asm detection on i386 (bug 145882, 145893). Thanks to
  Carlo Marcelo Arenas Belon <carenas.pe> for the initial patch.
- Removed unused build dependency on nasm
- Remove static lib
- Cleanup %doc a bit