Re: Please review wip/grub2

On Fri, 27 Jul 2012 07:23:09 +0900, Grégoire Sutre <[email protected]> wrote:

> WARN: Makefile:39: Please use BUILD_USES_MSGFMT=yes instead of this
> dependency.
>
> This comes from the dependency:
>
> DEPENDS+=               gettext>=0.17:../../devel/gettext
>
> But I've not found any documentation on BUILD_USES_MSGFMT, nor
> any package using it.  GRUB requires gettext 0.17 or later,
> and it's also a runtime dependency.

(pkglint should be fixed, USE_TOOLS+=msgfmt should be suggested instead).

Usually, whole gettext will not be required, just libintl and/or gettext tools.

Why 0.17 or later is required?
INSTALL files is probably wrong, because it is impossible to build
with native gcc-4.1.3 on NetBSD-5.1 :(

> WARN: Makefile:63: "" is not valid for ABI. Use one of { 32 64 } instead.
>
> I unset ABI on purpose, as otherwise the build fails on Debian
> GNU/Linux amd64.  The reason is that ABI gets set to 64, which
> makes the gcc wrapper append the flag -m64, causing problems
> (as some files need to be compiled with -m32).  Likewise, if
> ABI is set to 32, then the build fails.  In short, pkgsrc's
> gcc wrapper shouldn't append a -mXX flag at all.  Is there a
> better way to do this than by unsetting ABI?

Is it means that some part must be build with native ABI,
but rest of them be ABI=32 (i386)?

-- 
OBATA Akio / [email protected]

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
pkgsrc-wip-review mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review