Re: [Ltib] mtd-utils fails to build with gcc-4.5+

[Stuart Hughes]

Hi Peter,

I've applied this.  I've kept the patch number the same to keep things
lined up.  This is what's been checked-in:

Index: dist/lfs-5.1/mtd-utils/mtd-utils.spec
===================================================================
RCS file: /sources/ltib/ltib/dist/lfs-5.1/mtd-utils/mtd-utils.spec,v
retrieving revision 1.6
diff -u -r1.6 mtd-utils.spec
--- dist/lfs-5.1/mtd-utils/mtd-utils.spec    13 Apr 2010 08:45:30
-0000    1.6
+++ dist/lfs-5.1/mtd-utils/mtd-utils.spec    4 Mar 2012 18:04:35 -0000
@@ -15,6 +15,7 @@
 Patch4        : mtd-utils-20080618-pres-int-files.patch
 Patch5        : mtd-utils-20080618-macro-incr.patch
 Patch6        : mtd-utils-20080618-add-ecclayout.patch
+Patch10     : mtd-utils-20080618-gcc-4.6.1-01.patch
 BuildRoot    : %{_tmppath}/%{name}
 Prefix        : %{pfx}
 
@@ -33,6 +34,7 @@
 %patch4 -p1
 %patch5 -p1
 %patch6 -p1
+%patch10 -p1
 
 %Build
 make -j1 WITHOUT_XATTR=1

Thanks for the patch.

Regards, Stuart

On 03/03/12 16:23, Peter Barada wrote:
> On 03/03/2012 07:29 AM, Stuart Hughes wrote:
>> Hi Peter,
>>
>> I tried to apply this, but the current CVS version does not have
>> patches: 7,8 and 9.
>>
>> Could you send these and a I'll get them all applied.
> Stuart, those patches are to fix particulars in my system (dealing with
> linux-3.0 MTD changes, and problems with OOB layout for a Micron in-chip
> ECC where jffs2's cleanmarker "polutted" the ECC).  I'd recommend
> skipping them (the mtd-utils-20080618-gcc-4.6.1-01.patch should apply
> cleanly without them).
>
>> Regards, Stuart
>>
>> On 28/02/12 19:37, Peter Barada wrote:
>>> When building mtd-utils-20080618 using CodeSourcery 2011.09-70 for ARM
>>> (gcc-4.6.1) it fails with (at least):
>>>
>>> gcc -I./inc -I./src -I../include -std=gnu99 -DPACKAGE_VERSION=\"1.0\"
>>> -O2 -g -Werror -Wall -Wwrite-strings -W -c -o error.o ./src/error.c -g
>>> -Wp,-MD,.error.c.dep
>>> ./src/error.c: In function 'info_msg':
>>> ./src/error.c:77:6: error: variable 'n' set but not used
>>> [-Werror=unused-but-set-variable]
>>> cc1: all warnings being treated as errors
>>>
>>> make[1]: *** [error.o] Error 1
>>> make[1]: Leaving directory
>>> `/home/peter/work/logic/eps_svn/software/products/linux/LTIB/trunk/ltib-3.0/rpm/BUILD/mtd-utils/ubi-utils'
>>> make: *** [all] Error 2
>>> error: Bad exit status from
>>> /home/peter/work/logic/eps_svn/software/products/linux/LTIB/trunk/ltib-3.0/tmp/rpm-tmp.85594
>>> (%build)
>>>
>>>
>>> RPM build errors:
>>>     Bad exit status from
>>> /home/peter/work/logic/eps_svn/software/products/linux/LTIB/trunk/ltib-3.0/tmp/rpm-tmp.85594
>>> (%build)
>>> Build time for mtd-utils: 7 seconds
>>>
>>> Failed building mtd-utils
>>>
>>>
>>> The attached patch fixes this (at least makes mtd-utils build) for
>>> gcc-4.5+...
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> LTIB home page: http://ltib.org
>>>
>>> Ltib mailing list
>>> address@hidden
>>> https://lists.nongnu.org/mailman/listinfo/ltib
>