Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Andreas Jochens
Hello,

On 06-Sep-28 15:43, Robert Millan wrote:
> On Thu, Sep 28, 2006 at 09:57:24AM +0200, Andreas Jochens wrote:
> > This occurs because util/console.c is compiled with the '-m32' switch
> > and the linker does not find a corresponding 32-bit-powerpc libncurses 
> > library on ppc64.
> 
> I believe upstream chose to build the host part in 32-bit for a reason.  If 
> you
> (or someone else) can confirm this is no longer needed (i.e., grub2 builds,
> installs and runs), then we should check with upstream to get this check
> disabled.
> 
> Otherwise, can we just add lib32ncurses5-dev et al to Build-Depends ?  If you
> can confirm it builds this way, I'd rather do this instead.

yes it builds with lib32ncurses5-dev, thanks.

I wonder why the host part is 32-bit on ppc64 while it is 64-bit on amd64.
Usually ppc64 is treated similar to amd64.

Regards
Andreas Jochens


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Robert Millan
On Thu, Sep 28, 2006 at 12:00:48PM -0300, Otavio Salvador wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
> 
> > [ Please don't use -quiet.  I didn't see your reply and had to copy it from 
> > BTS ]
> 
> Ok, sorry!

Np ;)

> >> IIRC, bootloading on ppc64 needs to be done at 32bits space. 
> >> 
> >> Can someone confirm it?
> >
> > Yes, it needs to be 32-bit.  But the part that links with ncurses is 
> > grub-emu
> > which is a host tool.
> 
> humm... to allow it we would like to change the code to allow
> different parts of compilation use different flags. I hadn't check if
> the code already allow it but would be better to stay in a single mode
> (32 or 64) to keep simple.

The upstream code already does that (host_m32 is for the host components, and
target_m32 for the bootloader itself).  It's clear we need target_m32, but for
host_m32 we don't know what works and what doesn't, so we can only stick to
what upstream decided for us.

Our problem, however, is not really to decide wether to use 32 or 64 bit for the
host part, but to add biarch build-dependencies in case 32-bit is going to be
used.

To summarise:

  - To change host_m32 variable please check if that works and get in touch with
upstream (grub-devel@gnu.org).

  - In case host_m32 is going to be set, we (debian) need to add the appropiate
biarch build-dependencies, but we still need someone to confirm these work
at least to get the thing building.

-- 
Robert Millan

My spam trap is [EMAIL PROTECTED]  Note: this address is only intended for
spam harvesters.  Writing to it will get you added to my black list.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Otavio Salvador
Robert Millan <[EMAIL PROTECTED]> writes:

> [ Please don't use -quiet.  I didn't see your reply and had to copy it from 
> BTS ]

Ok, sorry!

>> IIRC, bootloading on ppc64 needs to be done at 32bits space. 
>> 
>> Can someone confirm it?
>
> Yes, it needs to be 32-bit.  But the part that links with ncurses is grub-emu
> which is a host tool.

humm... to allow it we would like to change the code to allow
different parts of compilation use different flags. I hadn't check if
the code already allow it but would be better to stay in a single mode
(32 or 64) to keep simple.

-- 
O T A V I OS A L V A D O R
-
 E-mail: [EMAIL PROTECTED]  UIN: 5906116
 GNU/Linux User: 239058 GPG ID: 49A5F855
 Home Page: http://www.freedom.ind.br/otavio
-
"Microsoft gives you Windows ... Linux gives
 you the whole house."


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Robert Millan

[ Please don't use -quiet.  I didn't see your reply and had to copy it from BTS 
]

> IIRC, bootloading on ppc64 needs to be done at 32bits space. 
> 
> Can someone confirm it?

Yes, it needs to be 32-bit.  But the part that links with ncurses is grub-emu
which is a host tool.

-- 
Robert Millan

My spam trap is [EMAIL PROTECTED]  Note: this address is only intended for
spam harvesters.  Writing to it will get you added to my black list.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Robert Millan
On Thu, Sep 28, 2006 at 09:57:24AM +0200, Andreas Jochens wrote:
> Package: grub2
> Version: 1.94+20060926-1
> Severity: wishlist
> Tags: patch
> 
> Hello,
> 
> thank you for uploading the new version of grub2 which also targets ppc64.
> 
> When building 'grub2' on ppc64/unstable, I get the following error:
> 
> [...]
> 
> This occurs because util/console.c is compiled with the '-m32' switch
> and the linker does not find a corresponding 32-bit-powerpc libncurses 
> library on ppc64.

I believe upstream chose to build the host part in 32-bit for a reason.  If you
(or someone else) can confirm this is no longer needed (i.e., grub2 builds,
installs and runs), then we should check with upstream to get this check
disabled.

Otherwise, can we just add lib32ncurses5-dev et al to Build-Depends ?  If you
can confirm it builds this way, I'd rather do this instead.

Thanks!

> diff -urN ../tmp-orig/grub2-1.94+20060926/configure.ac ./configure.ac
> --- ../tmp-orig/grub2-1.94+20060926/configure.ac  2006-07-12 
> 20:42:52.0 +
> +++ ./configure.ac2006-09-28 07:12:47.0 +
> @@ -40,10 +40,6 @@
>  AC_CANONICAL_HOST
>  AC_CANONICAL_TARGET
>  
> -case "$host_cpu" in
> -  powerpc64) host_m32=1 ;;
> -esac
> -
>  case "$target_cpu" in
>i[[3456]]86) target_cpu=i386 ;;
>x86_64) target_cpu=i386 target_m32=1 ;;

-- 
Robert Millan

My spam trap is [EMAIL PROTECTED]  Note: this address is only intended for
spam harvesters.  Writing to it will get you added to my black list.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#389873: grub2: FTBFS (ppc64): util/console.c:275: undefined reference to `stdscr'

2006-09-28 Thread Andreas Jochens
Package: grub2
Version: 1.94+20060926-1
Severity: wishlist
Tags: patch

Hello,

thank you for uploading the new version of grub2 which also targets ppc64.

When building 'grub2' on ppc64/unstable, I get the following error:

util/console.c:89: undefined reference to `stdscr'
util/console.c:89: undefined reference to `stdscr'
util/console.c:89: undefined reference to `waddch'
grub_emu-util_console.o: In function `grub_ncurses_init':
util/console.c:271: undefined reference to `stdscr'
util/console.c:268: undefined reference to `initscr'
util/console.c:269: undefined reference to `raw'
util/console.c:270: undefined reference to `noecho'
util/console.c:271: undefined reference to `stdscr'
util/console.c:271: undefined reference to `scrollok'
util/console.c:273: undefined reference to `nonl'
util/console.c:274: undefined reference to `stdscr'
util/console.c:274: undefined reference to `intrflush'
util/console.c:275: undefined reference to `stdscr'
util/console.c:275: undefined reference to `keypad'
util/console.c:276: undefined reference to `start_color'
collect2: ld returned 1 exit status
make[1]: *** [grub-emu] Error 1
make[1]: Leaving directory `/grub2-1.94+20060926'
make: *** [debian/stamp-makefile-build] Error 2

This occurs because util/console.c is compiled with the '-m32' switch
and the linker does not find a corresponding 32-bit-powerpc libncurses 
library on ppc64.

With the attached patch 'grub2' can be compiled on ppc64.

Regards
Andreas Jochens

diff -urN ../tmp-orig/grub2-1.94+20060926/configure ./configure
--- ../tmp-orig/grub2-1.94+20060926/configure   2006-09-26 17:24:33.0 
+
+++ ./configure 2006-09-28 07:12:59.0 +
@@ -1877,10 +1877,6 @@
 NONENONEs,x,x, &&
   program_prefix=${target_alias}-
 
-case "$host_cpu" in
-  powerpc64) host_m32=1 ;;
-esac
-
 case "$target_cpu" in
   i[3456]86) target_cpu=i386 ;;
   x86_64) target_cpu=i386 target_m32=1 ;;
diff -urN ../tmp-orig/grub2-1.94+20060926/configure.ac ./configure.ac
--- ../tmp-orig/grub2-1.94+20060926/configure.ac2006-07-12 
20:42:52.0 +
+++ ./configure.ac  2006-09-28 07:12:47.0 +
@@ -40,10 +40,6 @@
 AC_CANONICAL_HOST
 AC_CANONICAL_TARGET
 
-case "$host_cpu" in
-  powerpc64) host_m32=1 ;;
-esac
-
 case "$target_cpu" in
   i[[3456]]86) target_cpu=i386 ;;
   x86_64) target_cpu=i386 target_m32=1 ;;


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]