Yaboot 1.3.17 released.

[Tony Breeds]

On Tue, Oct 18, 2011 at 11:00:36PM +0200, acrux wrote:
> On Tue, 18 Oct 2011 19:39:21 +0200
> acrux <acrux_it at libero.it> wrote:
> 
> > On Tue, 18 Oct 2011 15:33:35 +1100
> > Tony Breeds <tony at bakeyournoodle.com> wrote:
> > 
> > > Hi All,
> > > 	After (yet again) far to long 1.3.17 is released.
> > > 
> > 
> > thanks for the new release.
> > I have a build issue with second/fs_swap.c
> > [...]
> > gcc -Os -m32 -O2 -mcpu=powerpc -pipe -nostdinc -Wall -isystem `gcc
> > -m32 -print-file-name=include` -fsigned-char -DVERSION="\"1.3.17\""
> > -DTEXTADDR=0x100000 -DDEBUG=0 -DMALLOCADDR= -DMALLOCSIZE=0x100000
> > -DKERNELADDR=0x00000000 -Werror -fdiagnostics-show-option
> > -I ./include -fno-strict-aliasing -DCONFIG_COLOR_TEXT
> > -DCONFIG_SET_COLORMAP -DUSE_MD5_PASSWORDS -DCONFIG_FS_XFS
> > -DCONFIG_FS_REISERFS -c -o second/fs_swap.o second/fs_swap.c cc1:
> > warnings being treated as errors second/fs_swap.c: In function
> > 'swap_open': second/fs_swap.c:90:15: error: unused variable
> > 'rc' [-Wunused-variable] make: *** [second/fs_swap.o] Error 1
> > 
> 
> BTW, removing -Werror from the Makefile i'm able to build it.
> Here attached my full build log.

Thanks.

---
+ patch -p0 -i /mnt/DEVEL/next/yaboot/work/src/chrpfix.patch
+ patch -p0

Are either of these 2 patches things that I should consider for
upstream?

second/fs_swap.c:90:15: warning: unused variable 'rc' [-Wunused-variable]
second/fs_reiserfs.c:221:17: warning: operation on '*journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:221:17: warning: operation on '*journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:221:17: warning: operation on '*journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:224:12: warning: operation on 'journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:224:12: warning: operation on 'journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:224:12: warning: operation on 'journal_table' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:243:12: warning: operation on 'i' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:243:12: warning: operation on 'i' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:243:12: warning: operation on 'i' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:255:10: warning: operation on 'i' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:255:10: warning: operation on 'i' may be undefined [-Wsequence-point]
second/fs_reiserfs.c:255:10: warning: operation on 'i' may be undefined [-Wsequence-point]
---

I've not seen those warnings before, what version of gcc are you using?
The ones in fs_reiserfs.c are frsutrating as I thought I'd fixed that
function in .17

Yours Tony
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/yaboot-devel/attachments/20111019/71722f52/attachment.pgp>