[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: libtool versioning
From: |
Ralf Wildenhues |
Subject: |
Re: libtool versioning |
Date: |
Tue, 4 May 2010 20:00:44 +0200 |
User-agent: |
Mutt/1.5.20 (2009-10-28) |
* Peter Rosin wrote on Tue, May 04, 2010 at 10:05:37AM CEST:
> Den 2010-05-03 22:03 skrev Ralf Wildenhues:
[...]
> >>>>># 3. If the library source code has changed at all since the last
> >>>>>update,
> >>>>># then increment revision (`c:r:a' becomes `c:r+1:a').
> >>>>># 4. If any interfaces have been added, removed, or changed since the
> >>>>>last
> >>>>># update, increment current, and set revision to 0.
> >>>>># 5. If any interfaces have been added since the last public release,
> >>>>>then
> >>>>># increment age.
> >>>>># 6. If any interfaces have been removed since the last public release,
> >>>>># then set age to 0.
> >>>>
> >>>>Shouldn't step #6 included "changed" as well as "removed"? If you
> >>>>change the interface (for example modifying function parameters),
> >>>>backwards compatibility is broken too. But only "removed" is listed
> >>>>here.
> >>>
> >>>Well, it is listed in step 4 already.
> >>
> >>Sure, but if interfaces have been changed, but not removed, the
> >>algorithm stops at step #4,
> >
> >Read the algorithm without any implicit stops in mind, and it will work.
> >
> >Anyway, your point is good that this is hard to understand, and we've
> >addressed that in git master.
>
> Errrm, is that really so? I tend to agree with Jef here...
I take it that your response is to my "... it will work" sentence, not
the paragraph below that.
> The algorithm *could* be interpreted such that e.g. the interface change
> "int foo(void)" -> "int foo(int)" is an interface addition of int foo(int)
> and an interface remove of int foo(void), thus triggering both #5 and #6.
> But in that case "changed" need not be mentioned in #4 either. So, because
> "changed" is mentioned in #4, it also needs to be explicitly mentioned
> in #6.
Ah, ok. Yes, you're right. Feel free to commit a patch to
s/removed/& or changed/ in 6.
Thanks for perservering!
Ralf
- libtool versioning, Jef Driesen, 2010/05/01
- Re: libtool versioning, Bob Friesenhahn, 2010/05/01
- Re: libtool versioning, Jef Driesen, 2010/05/03
- Re: libtool versioning, Bob Friesenhahn, 2010/05/03
- Re: libtool versioning, Ralf Wildenhues, 2010/05/03
- Re: libtool versioning, Jef Driesen, 2010/05/03
- Re: libtool versioning, Ralf Wildenhues, 2010/05/03
- Re: libtool versioning, Peter Rosin, 2010/05/04
- Re: libtool versioning,
Ralf Wildenhues <=
- Re: libtool versioning, Peter Rosin, 2010/05/04
- Re: libtool versioning, Jason Curl, 2010/05/06
- Re: libtool versioning, Peter Rosin, 2010/05/06
- Re: libtool versioning, Michel Briand, 2010/05/03
- Re: libtool versioning, Tor Lillqvist, 2010/05/04
Re: libtool versioning, Matěj Týč, 2010/05/03
Re: libtool versioning, Ralf Wildenhues, 2010/05/03
Re: libtool versioning, Jason Curl, 2010/05/06