On Sat, Mar 12, 2016 at 09:29:15PM +0300, Andrei Borzenkov wrote:
> > cc -Wall -W -Wshadow -Wpointer-arith -Wundef -Wchar-subscripts -Wcomment
> > -Wdeprecated-declarations -Wdisabled-optimization -Wdiv-by-zero
> > -Wfloat-equal -Wformat-extra-args -Wformat-security -Wformat-y2k -Wimplicit
> > -Wimplicit-function-declaration -Wimplicit-int -Wmain -Wmissing-braces
> > -Wmissing-format-attribute -Wmultichar -Wparentheses -Wreturn-type
> > -Wsequence-point -Wshadow -Wsign-compare -Wswitch -Wtrigraphs
> > -Wunknown-pragmas -Wunused -Wunused-function -Wunused-label
> > -Wunused-parameter -Wunused-value -Wunused-variable -Wwrite-strings
> > -Wnested-externs -Wstrict-prototypes -Wcast-align -Wextra -Wattributes
> > -Wendif-labels -Winit-self -Wint-to-pointer-cast -Winvalid-pch
> > -Wmissing-field-initializers -Wnonnull -Woverflow -Wvla
> > -Wpointer-to-int-cast -Wstrict-aliasing -Wvariadic-macros
> > -Wvolatile-register-var -Wpointer-sign -Wmissing-include-dirs
> > -Wmissing-prototypes -Wmissing-declarations -Wformat=2 -Wno-undef
> > -Wno-sign-compare -Wno-unused -Wno-unused-parameter -Wno-redundant-decls -W
no
> -
> unreachable-code -Wno-conversion -O2 -pipe -o grub-macbless
> util/grub_macbless-grub-macbless.o grub-core/osdep/grub_macbless-init.o
> grub-core/kern/emu/grub_macbless-argp_common.o libgrubmods.a libgrubgcry.a
> libgrubkern.a grub-core/gnulib/libgnu.a /usr/local/lib/libintl.so.6.0
> -L/usr/local/lib /usr/local/lib/libiconv.so.6.0 -lc -Wl,-rpath,/usr/local/lib
>
> > libgrubkern.a(libgrubkern_a-hostdisk.o): In function `read_device_map':
> > hostdisk.c:(.text+0x7ab): warning: warning: strcpy() is almost always
> > misused, please use strlcpy()
> > libgrubkern.a(libgrubkern_a-misc.o): In function `grub_util_get_path':
> > misc.c:(.text+0x12a): warning: warning: sprintf() is often misused, please
> > use snprintf()
> > libgrubkern.a(libgrubkern_a-getroot.o): In function
> > `grub_util_part_to_disk':
> > getroot.c:(.text+0x2b): undefined reference to `getrawpartition'
> >
>
> Does attached patch help?
>
> diff --git a/Makefile.util.def b/Makefile.util.def
> index ed9b4c6..f9caccb 100644
> --- a/Makefile.util.def
> +++ b/Makefile.util.def
> @@ -416,7 +416,7 @@ program = {
> ldadd = libgrubgcry.a;
> ldadd = libgrubkern.a;
> ldadd = grub-core/gnulib/libgnu.a;
> - ldadd = '$(LIBINTL) $(LIBDEVMAPPER) $(LIBZFS) $(LIBNVPAIR) $(LIBGEOM)';
> + ldadd = '$(LIBINTL) $(LIBDEVMAPPER) $(LIBUTIL) $(LIBZFS) $(LIBNVPAIR)
> $(LIBGEOM)';
> };
>
> data = {
Thank you, your diff makes compilation pass.
Well, I see something went wrong now with compressing a file:
$ tar tzvf /home/jirib/openbsd/packages/amd64/all/grub-2.02-beta3.tgz | grep
lzma_decompress
-r-xr-xr-x 1 root bin 3904 Jan 1 1970
lib/grub/i386-pc/lzma_decompress.image
-r--r--r-- 1 root bin 134480024 Jan 1 1970
lib/grub/i386-pc/lzma_decompress.img
It should not be 128MB, right? :)
I uploaded whole build log here: http://devio.us/~jirib/grub.txt (2.3MB)
> > -find . -iname '*.[ch]' ! -ipath './grub-core/lib/libgcrypt-grub/*' !
> > -ipath './build-aux/*' ! -ipath './grub-core/lib/libgcrypt/src/misc.c' !
> > -ipath './grub-core/lib/libgcrypt/src/global.c' ! -ipath
> > './grub-core/lib/libgcrypt/src/secmem.c' ! -ipath
> > './util/grub-gen-widthspec.c' ! -ipath './util/grub-gen-asciih.c' |sort >
> > po/POTFILES.in
> > +find . -iname '*.[ch]' ! -path './grub-core/lib/libgcrypt-grub/*' ! -path
> > './build-aux/*' ! -path './grub-core/lib/libgcrypt/src/misc.c' ! -path
> > './grub-core/lib/libgcrypt/src/global.c' ! -path
> > './grub-core/lib/libgcrypt/src/secmem.c' ! -path
> > './util/grub-gen-widthspec.c' ! -path './util/grub-gen-asciih.c' |sort >
> > po/POTFILES.in
> > find util -iname '*.in' ! -name Makefile.in |sort > po/POTFILES-shell.in
> >
>
> Hmm ... we support building on Windows which is case insensitive. What
> are other options to compare full path name case insenstive?
I can live with patching it. OpenBSD find doesn't have 'ipath' and IIUC
it's not POSIX requiremement, so I doubt it will be implemented.
> > Another thing - 'INSTALL' file says >= autoconf 2.60 is needed but
> > it is >= 2.62 otherwise there is:
> >
> > "error: possibly undefined macro: AS_ECHO"
> >
>
> Indeed. According to git log, AS_ECHO appeared first in 2.61a. What
> version you have?
IIUC it was firstly available in "normal" autoconf release in 2.62.
> > IIRC there's also a difference between NetBSD and OpenBSD regarding
> > to disks. NetBSD uses eg. /dev/rwd0c for NetBSD part, /dev/rwd0d for
> > whole disk, OpenBSD uses 'c' for whole disk and 'd' has no special
> > reason.
> >
>
> If I understand it correctly, getrawpartition() is supposed to handle it
> (i.e. return entire disk partition number). What exactly does not work?
The above comment was just comment. Not sure if there's real issue in
grub code based on those NetBSD/OpenBSD differences.
j.
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
