Re: man nilfs-clean

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2 May 2012 15:44:33 +0100, Tiago Caxias wrote:
> Hi all.
> 
> First of all let me thank and congratulate everyone on the nilfs2 project.
> 
> I believe I found a typo in the nilfs-clean man page. (is an interger
> value -> is an integer value)

Thanks, I will apply it.

> I cloned the git repo, edited the file and then I did a git diff.
> Not sure if this is the proper way to post a "patch" but here's my first try.

That's OK.

One thing I will ask you to do, is to include a "Singed-off-by" line
(sob line for short) in your patch, which certifies that you wrote it
or otherwise have the right to pass it on as an open-source patch.
even if it's a typo fix ;)

The sob line is inserted before the source code diff in the following
form:

Signed-off-by: Tiago Caxias <your e-mail address>

This is automatically done if you used "git commit" command with an -s
option; usually you can make a patch from the local change by "git
commit -a -s".

Then you can confirm or output the patch by "git show" or "git
format-patch HEAD^.."

I think that it is good to learn the function of git little by little
starting from hand working since git has various functions.


Thanks,
Ryusuke Konishi

> diff --git a/man/nilfs-clean.8 b/man/nilfs-clean.8
> index 758b9e1..04e11c6 100644
> --- a/man/nilfs-clean.8
> +++ b/man/nilfs-clean.8
> @@ -45,7 +45,7 @@ Display help message and exit.
>  .TP
>  \fB\-p\fR, \fB\-\-protection-period=\fIinterval\fR
>  Set protection period for a cleaner run.  The \fIinterval\fR parameter
> -is an interger value and specifies the minimum time that deleted or
> +is an integer value and specifies the minimum time that deleted or
>  overrode data will be preserved in the file system.  Smaller values will
>  increase free space made by garbage collection.  Optionally, the
>  \fIinterval\fP parameter may be suffixed by one of the following
> 
> --
> Best regards,
> Tiago Caxias
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux BTRFS]     [Linux CIFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux