From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: grub-devel@gnu.org
Subject: Re: GRUB 1.99 released
Date: Sun, 15 May 2011 10:20:24 +0200 [thread overview]
Message-ID: <4DCF8CC8.4070508@gmail.com> (raw)
In-Reply-To: <4DCF3B0E.9090000@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 3708 bytes --]
On 15.05.2011 04:31, Bruce Dubbs wrote:
> Vladimir 'φ-coder/phcoder' Serbinenko wrote:
>> Hello, all
>>
>> I'm proud to announce the release of GNU GRUB version 1.99.
>
> Congratulations on the release. As one who builds almost everything
> form source, the download and build went very smoothly. Thank you.
>
> I want to share some nits with you. Use them or ignore them as you
> please.
>
> -- Bruce
>
> 1. The install creates an empty directory: /usr/share/grub
>
It's empty only if no freetype or no unifont was found
> 2. There were several trivial build warnings:
>
> grub_script.yy.h:293: warning: 'yy_flex_strncpy' declared 'static' but
> never defined
> grub_script.yy.c:2241: warning: unused parameter 'yyscanner'
> grub_script.yy.c:2482: warning: unused parameter 'yyscanner'
These warnings come from flex autogenerated code
> config.status: WARNING: 'util/grub.d/10_linux.in' seems to ignore the
> --datarootdir setting
> config.status: WARNING: 'util/grub.d/20_linux_xen.in' seems to ignore
> the --datarootdir setting
> config.status: WARNING: 'util/grub-install.in' seems to ignore the
> --datarootdir setting
> config.status: WARNING: 'util/grub-mknetdir.in' seems to ignore the
> --datarootdir setting
--datarootdir isn't supported by us (yet?).
> fs/btrfs.c: In function 'grub_btrfs_read_logical':
> fs/btrfs.c:592: warning: 'err' may be used uninitialized in this function
Spurious warning with your gcc. I'll silence it.
> grub_script.yy.h:293: warning: 'yy_flex_strncpy' declared 'static' but
> never defined
> grub_script.yy.c: In function 'yy_fatal_error':
> grub_script.yy.c:2241: warning: unused parameter 'msg'
> grub_script.yy.c:2241: warning: unused parameter 'yyscanner'
> grub_script.yy.c: At top level:
> grub_script.yy.c:2279: warning: no previous prototype for 'yyget_column'
> grub_script.yy.c:2322: warning: no previous prototype for 'yyset_column'
> grub_script.yy.c: In function 'yy_flex_strncpy':
> grub_script.yy.c:2482: warning: unused parameter 'yyscanner'
ditto
> lib/libgcrypt-grub/cipher/camellia.c:450: warning: no previous
> prototype for 'camellia_setup128'
> lib/libgcrypt-grub/cipher/camellia.c:661: warning: no previous
> prototype for 'camellia_setup256'
> lib/libgcrypt-grub/cipher/camellia.c:947: warning: no previous
> prototype for 'camellia_setup192'
> lib/libgcrypt-grub/cipher/camellia.c: In function 'camellia_setup192':
> lib/libgcrypt-grub/cipher/camellia.c:951: warning: implicit
> declaration of function 'memcpy'
> lib/libgcrypt-grub/cipher/camellia.c: At top level:
> lib/libgcrypt-grub/cipher/camellia.c:969: warning: no previous
> prototype for 'camellia_encrypt128'
> lib/libgcrypt-grub/cipher/camellia.c:1059: warning: no previous
> prototype for 'camellia_decrypt128'
> lib/libgcrypt-grub/cipher/camellia.c:1152: warning: no previous
> prototype for 'camellia_encrypt256'
> lib/libgcrypt-grub/cipher/camellia.c:1266: warning: no previous
> prototype for 'camellia_decrypt256'
> lib/libgcrypt-grub/cipher/rijndael.c: In function 'do_setkey':
> lib/libgcrypt-grub/cipher/rijndael.c:173: warning: comparison between
> signed and unsigned
> lib/libgcrypt-grub/cipher/serpent.c: In function 'serpent_key_prepare':
> lib/libgcrypt-grub/cipher/serpent.c:589: warning: comparison between
> signed and unsigned
These files copied verbatim from libgcrypt. They must be fixed there first.
> reed_solomon.c: Assembler messages:
> reed_solomon.c:2: Warning: ignoring changed section attributes for .text
>
Fixing this warning would require significant changes. They were too big
to squeeze into 1.99.
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
prev parent reply other threads:[~2011-05-15 8:20 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-14 22:40 GRUB 1.99 released Vladimir 'φ-coder/phcoder' Serbinenko
2011-05-14 22:42 ` Vladimir 'φ-coder/phcoder' Serbinenko
2011-05-15 2:31 ` Bruce Dubbs
2011-05-15 8:20 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4DCF8CC8.4070508@gmail.com \
--to=phcoder@gmail.com \
--cc=grub-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.