[PATCH 2/2] util_lib: Silence ar informational message

[PATCH 2/2] util_lib: Silence ar informational message

[Geoff Levand]

Add the -c option to ar to silence the informational
message 'ar creating libutil.a'.  ar outputs this
message to stderr, and some tools interpret this as
a build warning.

Signed-off-by: Geoff Levand <geoff@infradead.org>
---
 util_lib/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util_lib/Makefile b/util_lib/Makefile
index 54ccdc5..15b84e5 100644
--- a/util_lib/Makefile
+++ b/util_lib/Makefile
@@ -18,5 +18,5 @@ $(UTIL_LIB): CPPFLAGS += -I$(srcdir)/util_lib/include
 
 $(UTIL_LIB): $(UTIL_LIB_OBJS)
 	@$(MKDIR) -p $(@D)
-	$(AR) rs $(UTIL_LIB) $(UTIL_LIB_OBJS)
+	$(AR) rs -c $(UTIL_LIB) $(UTIL_LIB_OBJS)
 
-- 
2.5.0


_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

Re: [PATCH 2/2] util_lib: Silence ar informational message

[Simon Horman]

Hi Geoff,

On Thu, Oct 08, 2015 at 04:21:24PM -0700, Geoff Levand wrote:
> Add the -c option to ar to silence the informational
> message 'ar creating libutil.a'.  ar outputs this
> message to stderr, and some tools interpret this as
> a build warning.

I am wondering about the portability of -c.
I tried to find some documentation of it and drew a blank.
Do you have any thoughts on this?

> Signed-off-by: Geoff Levand <geoff@infradead.org>
> ---
>  util_lib/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/util_lib/Makefile b/util_lib/Makefile
> index 54ccdc5..15b84e5 100644
> --- a/util_lib/Makefile
> +++ b/util_lib/Makefile
> @@ -18,5 +18,5 @@ $(UTIL_LIB): CPPFLAGS += -I$(srcdir)/util_lib/include
>  
>  $(UTIL_LIB): $(UTIL_LIB_OBJS)
>  	@$(MKDIR) -p $(@D)
> -	$(AR) rs $(UTIL_LIB) $(UTIL_LIB_OBJS)
> +	$(AR) rs -c $(UTIL_LIB) $(UTIL_LIB_OBJS)
>  
> -- 
> 2.5.0
> 

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

Re: [PATCH 2/2] util_lib: Silence ar informational message

[Geoff Levand]

Hi Simon,

On Fri, 2015-10-16 at 10:38 +0900, Simon Horman wrote:
> On Thu, Oct 08, 2015 at 04:21:24PM -0700, Geoff Levand wrote:
> > Add the -c option to ar to silence the informational
> > message 'ar creating libutil.a'.  ar outputs this
> > message to stderr, and some tools interpret this as
> > a build warning.
> 
> I am wondering about the portability of -c.
> I tried to find some documentation of it and drew a blank.
> Do you have any thoughts on this?

I couldn't find any info either.  At this point I would say
don't include this.

I'll think about something else.  Maybe capture the output
from ar, then only print it if ar returns an error.

-Geoff

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

Re: [PATCH 2/2] util_lib: Silence ar informational message

[Simon Horman]

On Mon, Oct 19, 2015 at 12:30:36PM -0700, Geoff Levand wrote:
> Hi Simon,
> 
> On Fri, 2015-10-16 at 10:38 +0900, Simon Horman wrote:
> > On Thu, Oct 08, 2015 at 04:21:24PM -0700, Geoff Levand wrote:
> > > Add the -c option to ar to silence the informational
> > > message 'ar creating libutil.a'.  ar outputs this
> > > message to stderr, and some tools interpret this as
> > > a build warning.
> > 
> > I am wondering about the portability of -c.
> > I tried to find some documentation of it and drew a blank.
> > Do you have any thoughts on this?
> 
> I couldn't find any info either.  At this point I would say
> don't include this.
> 
> I'll think about something else.  Maybe capture the output
> from ar, then only print it if ar returns an error.

Sounds messy but it may be the lest-worst solution.


_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec