Patchwork [1/2] libtool : bump to version 2.4.2

login
register
mail settings
Submitter Jeremy Rosen
Date May 13, 2013, 8:57 a.m.
Message ID <1368435466-8189-1-git-send-email-jeremy.rosen@openwide.fr>
Download mbox | patch
Permalink /patch/243335/
State Superseded
Headers show

Comments

Jeremy Rosen - May 13, 2013, 8:57 a.m.
Signed-off-by: Jérémy Rosen <jeremy.rosen@openwide.fr>
---
 package/libtool/libtool.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Thomas Petazzoni - May 13, 2013, 9:27 a.m.
Dear Jérémy Rosen,

On Mon, 13 May 2013 10:57:45 +0200, Jérémy Rosen wrote:
> Signed-off-by: Jérémy Rosen <jeremy.rosen@openwide.fr>
> ---
>  package/libtool/libtool.mk |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

How much testing did you do? Bumping libtool is a very sensitive thing,
because it used for all packages that we autoreconfigure. Therefore, I
believe such a bump should come with results of an allyespackageconfig
build before and after the bump.

Best regards,

Thomas
Jeremy Rosen - May 13, 2013, 3:29 p.m.
I needed to bump libtool because pulseaudio needed it.

I built my system (including pulseaudio) and tested it, but I didn't 
give it further (enough) thought.

This prbably means that this patch should not go in master but in 
next... It's probably way too late to bump libtool.

I'll do the allyesconfig test and reply with the results when done...

    Cordialement

    Jérémy Rosen

fight key loggers : write some perl using vim

----- Mail original -----
> Dear Jérémy Rosen,
> 
> On Mon, 13 May 2013 10:57:45 +0200, Jérémy Rosen wrote:
> > Signed-off-by: Jérémy Rosen <jeremy.rosen@openwide.fr>
> > ---
> >  package/libtool/libtool.mk |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> How much testing did you do? Bumping libtool is a very sensitive
> thing,
> because it used for all packages that we autoreconfigure. Therefore,
> I
> believe such a bump should come with results of an
> allyespackageconfig
> build before and after the bump.
> 
> Best regards,
> 
> Thomas
> --
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com
>
Thomas Petazzoni - May 13, 2013, 3:38 p.m.
Dear Jeremy Rosen,

On Mon, 13 May 2013 17:29:42 +0200 (CEST), Jeremy Rosen wrote:
> I needed to bump libtool because pulseaudio needed it.
> 
> I built my system (including pulseaudio) and tested it, but I didn't 
> give it further (enough) thought.
> 
> This prbably means that this patch should not go in master but in 
> next... It's probably way too late to bump libtool.

Version bumps are no longer going to master anyway, not even the
pulseaudio version bump. At this point, master is only about bug fixes,
so it's certainly not the time to bump packages!

> I'll do the allyesconfig test and reply with the results when done...

If you want, I can test the patch here on an allyespackageconfig build.

Best regards,

Thomas
Jeremy Rosen - May 13, 2013, 4:19 p.m.
> Dear Jeremy Rosen,
> 
[...]
> 
> Version bumps are no longer going to master anyway, not even the
> pulseaudio version bump. At this point, master is only about bug
> fixes,
> so it's certainly not the time to bump packages!
> 

fair enough... the bump is not urgent anyway...

> > I'll do the allyesconfig test and reply with the results when
> > done...
> 
> If you want, I can test the patch here on an allyespackageconfig
> build.
> 


that would save me a lot of time, thx.



> Best regards,
> 
> Thomas
> --
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com
>

Patch

diff --git a/package/libtool/libtool.mk b/package/libtool/libtool.mk
index 6f0b904..25cd89d 100644
--- a/package/libtool/libtool.mk
+++ b/package/libtool/libtool.mk
@@ -3,7 +3,7 @@ 
 # libtool
 #
 #############################################################
-LIBTOOL_VERSION = 2.2.10
+LIBTOOL_VERSION = 2.4.2
 LIBTOOL_SOURCE = libtool-$(LIBTOOL_VERSION).tar.gz
 LIBTOOL_SITE = $(BR2_GNU_MIRROR)/libtool
 LIBTOOL_INSTALL_STAGING = YES