* [PATCH] librados: Bump the version to 0.48 @ 2012-07-04 13:46 Wido den Hollander 2012-07-04 16:29 ` Gregory Farnum 0 siblings, 1 reply; 6+ messages in thread From: Wido den Hollander @ 2012-07-04 13:46 UTC (permalink / raw) To: ceph-devel; +Cc: Wido den Hollander Signed-off-by: Wido den Hollander <wido@widodh.nl> --- src/include/rados/librados.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/include/rados/librados.h b/src/include/rados/librados.h index 9f723f7..4870b0f 100644 --- a/src/include/rados/librados.h +++ b/src/include/rados/librados.h @@ -23,7 +23,7 @@ extern "C" { #endif #define LIBRADOS_VER_MAJOR 0 -#define LIBRADOS_VER_MINOR 44 +#define LIBRADOS_VER_MINOR 48 #define LIBRADOS_VER_EXTRA 0 #define LIBRADOS_VERSION(maj, min, extra) ((maj << 16) + (min << 8) + extra) -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] librados: Bump the version to 0.48 2012-07-04 13:46 [PATCH] librados: Bump the version to 0.48 Wido den Hollander @ 2012-07-04 16:29 ` Gregory Farnum 2012-07-04 16:33 ` Sage Weil 0 siblings, 1 reply; 6+ messages in thread From: Gregory Farnum @ 2012-07-04 16:29 UTC (permalink / raw) To: Wido den Hollander; +Cc: ceph-devel Hmmm — we generally try to modify these versions when the API changes, not on every sprint. It looks to me like Sage added one function in 0.45 where we maybe should have bumped it, but that was a long time ago and at this point we should maybe just eat it? -Greg On Wednesday, July 4, 2012 at 6:46 AM, Wido den Hollander wrote: > > Signed-off-by: Wido den Hollander <wido@widodh.nl (mailto:wido@widodh.nl)> > --- > src/include/rados/librados.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/include/rados/librados.h b/src/include/rados/librados.h > index 9f723f7..4870b0f 100644 > --- a/src/include/rados/librados.h > +++ b/src/include/rados/librados.h > @@ -23,7 +23,7 @@ extern "C" { > #endif > > #define LIBRADOS_VER_MAJOR 0 > -#define LIBRADOS_VER_MINOR 44 > +#define LIBRADOS_VER_MINOR 48 > #define LIBRADOS_VER_EXTRA 0 > > #define LIBRADOS_VERSION(maj, min, extra) ((maj << 16) + (min << 8) + extra) > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@vger.kernel.org (mailto:majordomo@vger.kernel.org) > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] librados: Bump the version to 0.48 2012-07-04 16:29 ` Gregory Farnum @ 2012-07-04 16:33 ` Sage Weil 2012-07-05 7:26 ` Wido den Hollander 2012-07-05 22:33 ` Gregory Farnum 0 siblings, 2 replies; 6+ messages in thread From: Sage Weil @ 2012-07-04 16:33 UTC (permalink / raw) To: Gregory Farnum; +Cc: Wido den Hollander, ceph-devel [-- Attachment #1: Type: TEXT/PLAIN, Size: 584 bytes --] On Wed, 4 Jul 2012, Gregory Farnum wrote: > Hmmm ÿÿ we generally try to modify these versions when the API changes, > not on every sprint. It looks to me like Sage added one function in 0.45 > where we maybe should have bumped it, but that was a long time ago and > at this point we should maybe just eat it? Yeah, I went ahead and applied this to stable (argonaut) since it's as good a reference point as any. Moving forward, we should try to sync this up with API changes as they happen. Hmm, like that assert ObjectOperation that just went into master... sage ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] librados: Bump the version to 0.48 2012-07-04 16:33 ` Sage Weil @ 2012-07-05 7:26 ` Wido den Hollander 2012-07-05 22:33 ` Gregory Farnum 1 sibling, 0 replies; 6+ messages in thread From: Wido den Hollander @ 2012-07-05 7:26 UTC (permalink / raw) To: Sage Weil; +Cc: Gregory Farnum, ceph-devel On 07/04/2012 06:33 PM, Sage Weil wrote: > On Wed, 4 Jul 2012, Gregory Farnum wrote: >> Hmmm ÿÿ we generally try to modify these versions when the API changes, >> not on every sprint. It looks to me like Sage added one function in 0.45 >> where we maybe should have bumped it, but that was a long time ago and >> at this point we should maybe just eat it? > > Yeah, I went ahead and applied this to stable (argonaut) since it's as > good a reference point as any. Moving forward, we should try to sync > this up with API changes as they happen. Hmm, like that assert > ObjectOperation that just went into master... That was my reasoning. I compiled phprados against 0.48 and saw that librados was reporting 0.44 as version. That could confuse users and they might think they still have an old library in place. Imho the version numbering should be totally different from Ceph if you only want to bump the version on an API change. Wido > > sage > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] librados: Bump the version to 0.48 2012-07-04 16:33 ` Sage Weil 2012-07-05 7:26 ` Wido den Hollander @ 2012-07-05 22:33 ` Gregory Farnum 2012-07-06 8:52 ` Wido den Hollander 1 sibling, 1 reply; 6+ messages in thread From: Gregory Farnum @ 2012-07-05 22:33 UTC (permalink / raw) To: Sage Weil; +Cc: Wido den Hollander, ceph-devel On Wed, Jul 4, 2012 at 9:33 AM, Sage Weil <sage@inktank.com> wrote: > On Wed, 4 Jul 2012, Gregory Farnum wrote: >> Hmmm ÿÿ we generally try to modify these versions when the API changes, >> not on every sprint. It looks to me like Sage added one function in 0.45 >> where we maybe should have bumped it, but that was a long time ago and >> at this point we should maybe just eat it? > > Yeah, I went ahead and applied this to stable (argonaut) since it's as > good a reference point as any. Moving forward, we should try to sync > this up with API changes as they happen. Hmm, like that assert > ObjectOperation that just went into master... Yep, should probably bump it to .49 then! (Since that's the version it will be part of, and nobody will get confused and try to bump it again before that release.) On Thu, Jul 5, 2012 at 12:26 AM, Wido den Hollander <wido@widodh.nl> wrote: > That was my reasoning. I compiled phprados against 0.48 and saw that > librados was reporting 0.44 as version. > > That could confuse users and they might think they still have an old library > in place. > > Imho the version numbering should be totally different from Ceph if you only > want to bump the version on an API change. Well, the problem with bumping it on every Ceph version is that it becomes a lot harder for tools to sync up to a known version of the API. Perhaps we should have divorced it from the Ceph versioning completely, but I don't know if we can still do that in a reasonable way or not. :/ -Greg -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] librados: Bump the version to 0.48 2012-07-05 22:33 ` Gregory Farnum @ 2012-07-06 8:52 ` Wido den Hollander 0 siblings, 0 replies; 6+ messages in thread From: Wido den Hollander @ 2012-07-06 8:52 UTC (permalink / raw) To: Gregory Farnum; +Cc: Sage Weil, ceph-devel On 07/06/2012 12:33 AM, Gregory Farnum wrote: > On Wed, Jul 4, 2012 at 9:33 AM, Sage Weil <sage@inktank.com> wrote: >> On Wed, 4 Jul 2012, Gregory Farnum wrote: >>> Hmmm ÿÿ we generally try to modify these versions when the API changes, >>> not on every sprint. It looks to me like Sage added one function in 0.45 >>> where we maybe should have bumped it, but that was a long time ago and >>> at this point we should maybe just eat it? >> >> Yeah, I went ahead and applied this to stable (argonaut) since it's as >> good a reference point as any. Moving forward, we should try to sync >> this up with API changes as they happen. Hmm, like that assert >> ObjectOperation that just went into master... > > Yep, should probably bump it to .49 then! (Since that's the version it > will be part of, and nobody will get confused and try to bump it again > before that release.) > > > On Thu, Jul 5, 2012 at 12:26 AM, Wido den Hollander <wido@widodh.nl> wrote: >> That was my reasoning. I compiled phprados against 0.48 and saw that >> librados was reporting 0.44 as version. >> >> That could confuse users and they might think they still have an old library >> in place. >> >> Imho the version numbering should be totally different from Ceph if you only >> want to bump the version on an API change. > > Well, the problem with bumping it on every Ceph version is that it > becomes a lot harder for tools to sync up to a known version of the > API. Perhaps we should have divorced it from the Ceph versioning > completely, but I don't know if we can still do that in a reasonable > way or not. :/ You could always say: 0.48 was the "stable" release, from here on we are only going to bump the librados version if any API change happends. Sent out an e-mail on the devel list and write a blogpost? I don't think we have a lot of developers using native librados at the moment. Wido > -Greg > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-06 9:19 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-07-04 13:46 [PATCH] librados: Bump the version to 0.48 Wido den Hollander 2012-07-04 16:29 ` Gregory Farnum 2012-07-04 16:33 ` Sage Weil 2012-07-05 7:26 ` Wido den Hollander 2012-07-05 22:33 ` Gregory Farnum 2012-07-06 8:52 ` Wido den Hollander
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.