[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] #10 OS400/EBCDIC Port: Disable unsupported subcommands and options

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2006-02-28 02:59:35 CET

Paul Burba wrote:
> [[[
> OS400/EBCDIC Port:

Were you meaning to put something like "Disable unsupported features." after
the colon?

> This is one of several patches to allow Subversion to run on IBM's
> OS400 V5R4. It disables subcommands and CL options not currently
> supported on OS400.
>
> * subversion/svn/main.c
> (svn_cl__options): Disable --encoding, --editor-cmd, --diff-cmd,
> --diff3-cmd, and --extensions options.
> (svn_cl__cmd_table): Tweak svn ci help message. Disable svn
> propedit.
> ]]]

> - { "commit", svn_cl__commit, {"ci"}, N_
> - ("Send changes from your working copy to the repository.\n"
> + { "commit", svn_cl__commit, {"ci"},
> +#ifndef AS400
> + N_("Send changes from your working copy to the repository.\n"
> "usage: commit [PATH...]\n"
> "\n"
> " A log message must be provided, but it can be empty. If it is not\n"
> " given by a --message or --file option, an editor will be started.\n"
> " If any targets are (or contain) locked items, those will be\n"
> " unlocked after a successful commit.\n"),
> +#else
> + N_("Send changes from your working copy to the repository.\n"
> + "usage: commit [PATH...]\n"
> + "\n"
> + " A log message must be provided, but it can be empty.\n"
> + " OS400 does not support the starting of an editor,\n"
> + " so --message or --file must be used. If any targets are\n"
> + " (or contain) locked items, those will be unlocked after a\n"
> + " successful commit.\n"),
> +#endif

Heh! To begin with I thought it would be nice if you could put the "#if"
around just the part of the string that differs, but then I realised that would
be trying to use a directive inside the invocation of a macro, so impossible.

Please could you reindent the existing text strings to match the new position
of the opening parenthesis?

Does something suitable happen if the user doesn't provide "--message" or "--file"?

Other than that, +1.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Feb 28 03:01:19 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.