[Patch] Nice help Output - SmallProjectIdea[1]

[Patch] Nice help Output - SmallProjectIdea[1]

[Alangi Derick]

This is a patch to fix the problem as given in the site:
https://git.wiki.kernel.org/index.php/SmallProjectsIdeas

diff --git a/help.c b/help.c
index 2072a87..e347bbc 100644
--- a/help.c
+++ b/help.c
@@ -229,9 +229,18 @@ void list_common_cmds_help(void)

  puts(_("The most commonly used git commands are:"));
  for (i = 0; i < ARRAY_SIZE(common_cmds); i++) {
- printf("   %s   ", common_cmds[i].name);
- mput_char(' ', longest - strlen(common_cmds[i].name));
- puts(_(common_cmds[i].help));
+ if(i%3 == 1){
+ printf("   %s   ", common_cmds[i].name);
+ mput_char(' ', longest - strlen(common_cmds[i].name));
+ puts(_(common_cmds[i].help));
+ printf("\n");
+ }
+ else {
+ printf("   %s   ", common_cmds[i].name);
+ mput_char(' ', longest - strlen(common_cmds[i].name));
+ puts(_(common_cmds[i].help));
+ }
+
  }
 }


Regards
Alangi Derick Ndimnain

Re: [Patch] Nice help Output - SmallProjectIdea[1]

[Thomas Gummerer]

Hi,

Alangi Derick <alangiderick@gmail.com> writes:

> This is a patch to fix the problem as given in the site:
> https://git.wiki.kernel.org/index.php/SmallProjectsIdeas

Please have a look at Documentation/SubmittingPatches, on how to submit
well formed patches.  In particular, the following things should be
improved:

- The commit should have a meaningful commit message (See section 2 in
  SubmittingPatches)
- The patch should be created with git format-patch, so it can be
  applied using git am. (See section 3 and 4)
- Your work should be signed off, otherwise it can not be applied to the
  project. (See section 5)

> diff --git a/help.c b/help.c
> index 2072a87..e347bbc 100644
> --- a/help.c
> +++ b/help.c
> @@ -229,9 +229,18 @@ void list_common_cmds_help(void)
>
>   puts(_("The most commonly used git commands are:"));
>   for (i = 0; i < ARRAY_SIZE(common_cmds); i++) {
> - printf("   %s   ", common_cmds[i].name);
> - mput_char(' ', longest - strlen(common_cmds[i].name));
> - puts(_(common_cmds[i].help));
> + if(i%3 == 1){
> + printf("   %s   ", common_cmds[i].name);
> + mput_char(' ', longest - strlen(common_cmds[i].name));
> + puts(_(common_cmds[i].help));
> + printf("\n");
> + }
> + else {
> + printf("   %s   ", common_cmds[i].name);
> + mput_char(' ', longest - strlen(common_cmds[i].name));
> + puts(_(common_cmds[i].help));
> + }
> +

Patch formatting issues aside, I don't think this does what is suggested
in the project.  You are just adding a blank line after three lines in
the help output, which I don't think improves the current state by much.

The bzr help output on the other hand groups similar commands together,
and shows them to the user in groups.  So the first step would be to
find out which commands should be grouped together and then to output
the groups here.

>   }
>  }
>
>
> Regards
> Alangi Derick Ndimnain
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [Patch] Nice help Output - SmallProjectIdea[1]

[Alangi Derick]

Thanks for the correction. I will work on that and see if i can solve
the problem. With respect to my git send-email, it is not working so i
don't know whether i have a problem with my smtp. I don't know how to
configure it.

Re: [Patch] Nice help Output - SmallProjectIdea[1]

[Thomas Gummerer]

Hi,

On 05/02, Alangi Derick wrote:
> Thanks for the correction. I will work on that and see if i can solve
> the problem.

I've noticed there was another patch series, which attempts at solving
the same problem at
http://thread.gmane.org/gmane.comp.version-control.git/268198.  So
maybe you could coordinate with Sébastien, or work on another project,
to avoid duplicating the work.

> With respect to my git send-email, it is not working so i
> don't know whether i have a problem with my smtp. I don't know how to
> configure it.

Maybe you can send us the configuration here, so someone might be able
to help.  Though it might be faster to get help on IRC, on the #git
channel on freenode.