diff mbox

[1/2] exportfs: drop extra newline in xlog

Message ID 1314890192-11075-1-git-send-email-vapier@gentoo.org (mailing list archive)
State New, archived
Headers show

Commit Message

Mike Frysinger Sept. 1, 2011, 3:16 p.m. UTC
Since xlog() itself appends a newline, we don't want to add our own
otherwise we get extra in the output.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
 utils/exportfs/exportfs.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

Comments

Steve Dickson Sept. 1, 2011, 3:35 p.m. UTC | #1
On 09/01/2011 11:16 AM, Mike Frysinger wrote:
> Since xlog() itself appends a newline, we don't want to add our own
> otherwise we get extra in the output.
> 
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Committed...

steved.

> ---
>  utils/exportfs/exportfs.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/utils/exportfs/exportfs.c b/utils/exportfs/exportfs.c
> index 12e8bf1..986a272 100644
> --- a/utils/exportfs/exportfs.c
> +++ b/utils/exportfs/exportfs.c
> @@ -401,7 +401,7 @@ validate_export(nfs_export *exp)
>  	int fs_has_fsid = 0;
>  
>  	if (stat(path, &stb) < 0) {
> -		xlog(L_ERROR, "Failed to stat %s: %m \n", path);
> +		xlog(L_ERROR, "Failed to stat %s: %m", path);
>  		return;
>  	}
>  	if (!S_ISDIR(stb.st_mode) && !S_ISREG(stb.st_mode)) {
> @@ -530,7 +530,7 @@ export_d_read(const char *dname)
>  
>  	n = scandir(dname, &namelist, NULL, versionsort);
>  	if (n < 0)
> -		xlog(L_NOTICE, "scandir %s: %s\n", dname, strerror(errno));
> +		xlog(L_NOTICE, "scandir %s: %s", dname, strerror(errno));
>  	else if (n == 0)
>  		return;
>  
> @@ -558,7 +558,7 @@ export_d_read(const char *dname)
>  
>  		fname_len = snprintf(fname, PATH_MAX +1, "%s/%s", dname, d->d_name);
>  		if (fname_len > PATH_MAX) {
> -			xlog(L_WARNING, "Too long file name: %s in %s\n", d->d_name, dname);
> +			xlog(L_WARNING, "Too long file name: %s in %s", d->d_name, dname);
>  			continue;
>  		}
>  
> @@ -672,7 +672,7 @@ dump(int verbose)
>  static void
>  error(nfs_export *exp, int err)
>  {
> -	xlog(L_ERROR, "%s:%s: %s\n", exp->m_client->m_hostname,
> +	xlog(L_ERROR, "%s:%s: %s", exp->m_client->m_hostname,
>  		exp->m_export.e_path, strerror(err));
>  }
>  
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/utils/exportfs/exportfs.c b/utils/exportfs/exportfs.c
index 12e8bf1..986a272 100644
--- a/utils/exportfs/exportfs.c
+++ b/utils/exportfs/exportfs.c
@@ -401,7 +401,7 @@  validate_export(nfs_export *exp)
 	int fs_has_fsid = 0;
 
 	if (stat(path, &stb) < 0) {
-		xlog(L_ERROR, "Failed to stat %s: %m \n", path);
+		xlog(L_ERROR, "Failed to stat %s: %m", path);
 		return;
 	}
 	if (!S_ISDIR(stb.st_mode) && !S_ISREG(stb.st_mode)) {
@@ -530,7 +530,7 @@  export_d_read(const char *dname)
 
 	n = scandir(dname, &namelist, NULL, versionsort);
 	if (n < 0)
-		xlog(L_NOTICE, "scandir %s: %s\n", dname, strerror(errno));
+		xlog(L_NOTICE, "scandir %s: %s", dname, strerror(errno));
 	else if (n == 0)
 		return;
 
@@ -558,7 +558,7 @@  export_d_read(const char *dname)
 
 		fname_len = snprintf(fname, PATH_MAX +1, "%s/%s", dname, d->d_name);
 		if (fname_len > PATH_MAX) {
-			xlog(L_WARNING, "Too long file name: %s in %s\n", d->d_name, dname);
+			xlog(L_WARNING, "Too long file name: %s in %s", d->d_name, dname);
 			continue;
 		}
 
@@ -672,7 +672,7 @@  dump(int verbose)
 static void
 error(nfs_export *exp, int err)
 {
-	xlog(L_ERROR, "%s:%s: %s\n", exp->m_client->m_hostname,
+	xlog(L_ERROR, "%s:%s: %s", exp->m_client->m_hostname,
 		exp->m_export.e_path, strerror(err));
 }