From: Steven Whitehouse <swhiteho@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [PATCH] i18n: strings review
Date: Wed, 20 Jul 2011 09:49:46 +0100 [thread overview]
Message-ID: <1311151786.2703.3.camel@menhir> (raw)
In-Reply-To: <1311105758-5134-1-git-send-email-cmaiolino@redhat.com>
Hi,
On Tue, 2011-07-19 at 17:02 -0300, Carlos Maiolino wrote:
> This patch is the first review of the gfs2-utils
> strings that will be translated. The changes in
> this patch were based according with the current
> strings added to the .pot file
> ---
> gfs2/convert/gfs2_convert.c | 4 ++--
> gfs2/fsck/fs_recovery.c | 4 ++--
> gfs2/fsck/initialize.c | 12 ++++++------
> gfs2/fsck/main.c | 8 ++++----
> gfs2/fsck/pass1c.c | 4 ++--
> gfs2/fsck/pass4.c | 2 +-
> gfs2/mkfs/main_grow.c | 4 ++--
> gfs2/mkfs/main_jadd.c | 6 +++---
> gfs2/mkfs/main_mkfs.c | 6 +++---
> gfs2/tune/main.c | 2 +-
> 10 files changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/gfs2/convert/gfs2_convert.c b/gfs2/convert/gfs2_convert.c
> index 7f6135f..2d4da10 100644
> --- a/gfs2/convert/gfs2_convert.c
> +++ b/gfs2/convert/gfs2_convert.c
> @@ -1690,11 +1690,11 @@ static void process_parameters(int argc, char **argv, struct gfs2_options *opts)
> strcpy(device, argv[optind]);
> opts->device = device;
> if(!opts->device) {
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> exit(1);
> }
> } else {
> - fprintf(stderr, _("No device specified. Use '-h' for usage.\n"));
> + fprintf(stderr, _("No device specified (try -h for help)\n"));
> exit(1);
> }
If you make that string say "No device specified (Please use '-h' for
help)\n" then the same translation can be used for the second half of
that message, as for the message earlier on.
> } /* process_parameters */
> diff --git a/gfs2/fsck/fs_recovery.c b/gfs2/fsck/fs_recovery.c
> index da06598..7dfb974 100644
> --- a/gfs2/fsck/fs_recovery.c
> +++ b/gfs2/fsck/fs_recovery.c
> @@ -473,8 +473,8 @@ static int gfs2_recover_journal(struct gfs2_inode *ip, int j, int preen,
> log_info( _("jid=%u: Repairing journal...\n"), j);
> error = fix_journal_seq_no(ip);
> if (error) {
> - log_err( _("jid=%u: Unable to repair the bad "
> - "journal.\n"), j);
> + log_err( _("jid=%u: Unable to fix the bad journal.\n"),
> + j);
> goto out;
> }
> error = gfs2_find_jhead(ip, &head);
> diff --git a/gfs2/fsck/initialize.c b/gfs2/fsck/initialize.c
> index 697671c..eafbebc 100644
> --- a/gfs2/fsck/initialize.c
> +++ b/gfs2/fsck/initialize.c
> @@ -771,7 +771,7 @@ static void peruse_system_dinode(struct gfs2_sbd *sdp, struct gfs2_dinode *di,
> if (di->di_num.no_formal_ino == 3) {
> if (fix_md.jiinode || is_journal_copy(ip, bh))
> return;
> - log_warn(_("Found system jindex file at: 0x%llx\n"),
> + log_warn(_("Found system jindex file at: 0x%llx.\n"),
> di->di_num.no_addr);
I don't think we need to have a '.' at the end of the string here, since
there is only a number being presented. Likewise the other places that
you've added a '.' at the end of the messages.
> fix_md.jiinode = ip;
> } else if (S_ISDIR(di->di_mode)) {
> @@ -784,7 +784,7 @@ static void peruse_system_dinode(struct gfs2_sbd *sdp, struct gfs2_dinode *di,
> fix_md.jiinode = child_ip;
> sdp->sd_sb.sb_master_dir.no_addr = di->di_num.no_addr;
> log_warn(_("Found system master directory at: "
> - "0x%llx\n"), di->di_num.no_addr);
> + "0x%llx.\n"), di->di_num.no_addr);
> return;
> }
>
> @@ -887,7 +887,7 @@ static void peruse_user_dinode(struct gfs2_sbd *sdp, struct gfs2_dinode *di,
> gfs2_lookupi(ip, "..", 2, &parent_ip);
> if (parent_ip && parent_ip->i_di.di_num.no_addr ==
> ip->i_di.di_num.no_addr) {
> - log_warn(_("fsck found the root inode at: 0x%llx\n"),
> + log_warn(_("Found the root directory at: 0x%llx.\n"),
> ip->i_di.di_num.no_addr);
> sdp->sd_sb.sb_root_dir.no_addr =
> ip->i_di.di_num.no_addr;
> @@ -1078,7 +1078,7 @@ static int sb_repair(struct gfs2_sbd *sdp)
> "be the root; using master - 1.\n"));
> possible_root = sdp->sd_sb.sb_master_dir.no_addr - 1;
> }
> - log_err(_("Found a root directory candidate at 0x%llx\n"),
> + log_err(_("Found a possible root at: 0x%llx\n"),
> (unsigned long long)possible_root);
> sdp->sd_sb.sb_root_dir.no_addr = possible_root;
> sdp->md.rooti = inode_read(sdp, possible_root);
> @@ -1106,13 +1106,13 @@ static int sb_repair(struct gfs2_sbd *sdp)
> /* Step 3 - Rebuild the lock protocol and file system table name */
> get_lockproto_table(sdp);
> if (query(_("Okay to fix the GFS2 superblock? (y/n)"))) {
> - log_info(_("Master system directory found at: 0x%llx\n"),
> + log_info(_("Found system master directory at: 0x%llx."),
> sdp->sd_sb.sb_master_dir.no_addr);
> sdp->master_dir = inode_read(sdp,
> sdp->sd_sb.sb_master_dir.no_addr);
> sdp->master_dir->i_di.di_num.no_addr =
> sdp->sd_sb.sb_master_dir.no_addr;
> - log_info(_("Root directory found at: 0x%llx\n"),
> + log_info(_("Found the root directory at: 0x%llx.\n"),
> sdp->sd_sb.sb_root_dir.no_addr);
> sdp->md.rooti = inode_read(sdp,
> sdp->sd_sb.sb_root_dir.no_addr);
> diff --git a/gfs2/fsck/main.c b/gfs2/fsck/main.c
> index bb540e7..9ec20ed 100644
> --- a/gfs2/fsck/main.c
> +++ b/gfs2/fsck/main.c
> @@ -99,10 +99,10 @@ static int read_cmdline(int argc, char **argv, struct gfs2_options *gopts)
> break;
> case ':':
> case '?':
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> return FSCK_USAGE;
> default:
> - fprintf(stderr, _("Unknown option %c\n"), c);
> + fprintf(stderr, _("Invalid option: %c\n"), c);
> return FSCK_USAGE;
>
> }
> @@ -110,11 +110,11 @@ static int read_cmdline(int argc, char **argv, struct gfs2_options *gopts)
> if(argc > optind) {
> gopts->device = (argv[optind]);
> if(!gopts->device) {
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> return FSCK_USAGE;
> }
> } else {
> - fprintf(stderr, _("No device specified. Use '-h' for usage.\n"));
> + fprintf(stderr, _("No device specified (try -h for help)\n"));
> return FSCK_USAGE;
> }
> return 0;
> diff --git a/gfs2/fsck/pass1c.c b/gfs2/fsck/pass1c.c
> index 0782459..a75a9ad 100644
> --- a/gfs2/fsck/pass1c.c
> +++ b/gfs2/fsck/pass1c.c
> @@ -189,12 +189,12 @@ static int check_eattr_entry(struct gfs2_inode *ip,
>
> if(max_ptrs > ea_hdr->ea_num_ptrs){
> log_err( _("EA (%s) has incorrect number of pointers.\n"), ea_name);
> - log_err( _(" Required: %d\n Reported: %d\n"),
> + log_err( _(" Pointers Required: %d\n Pointers Reported: %d\n"),
> max_ptrs, ea_hdr->ea_num_ptrs);
> return ask_remove_eattr_entry(sdp, leaf_bh, ea_hdr,
> ea_hdr_prev, 0, 0);
> } else {
> - log_debug( _(" Pointers Required: %d\n Pointers Reported: %d\n"),
> + log_debug( _(" Pointers Required: %d\n Pointers Reported: %d\n"),
> max_ptrs, ea_hdr->ea_num_ptrs);
> }
> }
> diff --git a/gfs2/fsck/pass4.c b/gfs2/fsck/pass4.c
> index 9233ea7..3305fb3 100644
> --- a/gfs2/fsck/pass4.c
> +++ b/gfs2/fsck/pass4.c
> @@ -64,7 +64,7 @@ static int scan_inode_list(struct gfs2_sbd *sbp) {
> (unsigned long long)ii->inode);
> q = block_type(ii->inode);
> if(q == gfs2_bad_block) {
> - log_err( _("Unlinked inode %llu (0x%llx) contains"
> + log_err( _("Unlinked inode %llu (0x%llx) contains "
> "bad blocks\n"),
> (unsigned long long)ii->inode,
> (unsigned long long)ii->inode);
> diff --git a/gfs2/mkfs/main_grow.c b/gfs2/mkfs/main_grow.c
> index 81f492b..f568330 100644
> --- a/gfs2/mkfs/main_grow.c
> +++ b/gfs2/mkfs/main_grow.c
> @@ -106,10 +106,10 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
> case ':':
> case '?':
> /* Unknown flag */
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> exit(EXIT_FAILURE);
> default:
> - fprintf(stderr, _("Unknown option %c\n"), opt);
> + fprintf(stderr, _("Invalid option: %c\n"), opt);
> exit(EXIT_FAILURE);
> break;
> }
> diff --git a/gfs2/mkfs/main_jadd.c b/gfs2/mkfs/main_jadd.c
> index 5f65cce..d722d52 100644
> --- a/gfs2/mkfs/main_jadd.c
> +++ b/gfs2/mkfs/main_jadd.c
> @@ -129,14 +129,14 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
> break;
> case ':':
> case '?':
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> exit(EXIT_FAILURE);
> break;
> case EOF:
> cont = FALSE;
> break;
> default:
> - die( _("Unknown option: %c\n"), optchar);
> + die( _("Invalid option: %c\n"), optchar);
> break;
> };
> }
> @@ -148,7 +148,7 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
> die( _("no path specified (try -h for help)\n"));
>
> if (optind < argc)
> - die( _("Unrecognized option: %s\n"), argv[optind]);
> + die( _("Unrecognized argument: %s\n"), argv[optind]);
>
> if (sdp->debug) {
> printf( _("Command Line Arguments:\n"));
> diff --git a/gfs2/mkfs/main_mkfs.c b/gfs2/mkfs/main_mkfs.c
> index eaf2aa2..fc90e41 100644
> --- a/gfs2/mkfs/main_mkfs.c
> +++ b/gfs2/mkfs/main_mkfs.c
> @@ -182,7 +182,7 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
>
> case ':':
> case '?':
> - fprintf(stderr, _("Please use '-h' for usage.\n"));
> + fprintf(stderr, _("Please use '-h' for help.\n"));
> exit(EXIT_FAILURE);
> break;
>
> @@ -203,7 +203,7 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
> break;
>
> default:
> - die( _("Unknown option: %c\n"), optchar);
> + die( _("Invalid option: %c\n"), optchar);
> break;
> };
> }
> @@ -212,7 +212,7 @@ static void decode_arguments(int argc, char *argv[], struct gfs2_sbd *sdp)
> strcpy(sdp->device_name, argv[optind++]);
>
> if (sdp->device_name[0] == '\0')
> - die( _("no device specified (try -h for help)\n"));
> + die( _("No device specified (try -h for help)\n"));
>
> if (optind < argc)
> sdp->orig_fssize = atol(argv[optind++]);
> diff --git a/gfs2/tune/main.c b/gfs2/tune/main.c
> index 6a0daff..e4cef82 100644
> --- a/gfs2/tune/main.c
> +++ b/gfs2/tune/main.c
> @@ -89,7 +89,7 @@ int main(int argc, char **argv)
> version();
> return 0;
> default:
> - fprintf(stderr, _("Invalid option.\n"));
> + fprintf(stderr, _("Invalid option: %c\n"), c);
> usage(argv[0]);
> return EX_USAGE;
> }
Otherwise this looks good,
Steve.
next prev parent reply other threads:[~2011-07-20 8:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-19 20:02 [Cluster-devel] [PATCH] i18n: strings review Carlos Maiolino
2011-07-20 8:49 ` Steven Whitehouse [this message]
-- strict thread matches above, loose matches on Subject: below --
2011-07-26 13:42 Carlos Maiolino
2011-07-26 13:57 ` Bob Peterson
2011-07-26 14:08 ` Steven Whitehouse
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=1311151786.2703.3.camel@menhir \
--to=swhiteho@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).