I wasn't aware of semver.org. I think that sine we're talking about osgi it might be more advisable to rely on the osgi semantic versioning white paper www.osgi.org/wiki/uploads/Links/SemanticVersioning.pdf
I don't think it's too late for karat. I'm not sure aries is in as much of a hell as people think :-). But to the extent it is it may be due to an attempt to keep the bundle version as small as possible. If we accept that: -- the bundle (and maven) version can be much larger than the semantic versioning rules require (incremented either due to semantics or a release) -- all bundles will always be released at once whether or not they have any changes from the last release -- all actual bug fixes (micro version increments, or non-api changes) need to get ported (pretty much immediately) to all branches containing the pre-existing version of a package (so the next change in the package will get the right version number). I'm not sure why there would be a problem. I wonder if there's an effective way to monitor the 3rd point. Anyway, if only api packages are exported, and all implementation is in private packages, there shouldn't often be bug fixes that change the version of an exported package. thanks david jencks On May 24, 2014, at 2:35 AM, Achim Nierbeck <[email protected]> wrote: > Please find my comments below. > > Regards, Achim > > sent from mobile device > Am 24.05.2014 00:57 schrieb "David Jencks" <[email protected]>: >> >> Well, your description blurs the distinction between providers and > consumers. Slightly more precisely > > Because I anticipated this kind of answer I copied the definition from > semvers.org. :-) > >> >> MAJOR changes signal a consumer-incompatible change >> MINOR changes signal a provider-incompatible change >> MICRO changes signal a change not visible in the API >> > > I'm not sure about that last definition. Cause this would lead has into > that exact definition of Guillaume. Just imagine us changing something > internal. Because of released artefacts need to be unique we bump the micro > version as it isn't a "provider-incompatible" change. That is why I prefer > the other definition. > >> I don't understand Guillaumes original example (is this your point too > Achim?). Making an addition to a package at 4.0.0 will raise it to 4.1.0? > If not, what kind of addition are you (Guillaume) talking about? I think > both the changes would have to be bug fixes. >> > > I think it is best to always increase the minor version if we have to bump > the version, but my guess right now is that Guillaume feared the same > implications that I fear in case we take the "non-blured" definition. > >> I thinksSemantic versioning is really supposed to apply to a single > unbranched stream of releases. You can graft it onto some branched release > structures but not all. I think the problem is that a package at version > 4.0.0 present in 2 branches can try to get independent bug fixes in each > branch. But if they are really bug fixes and not api changes then it > should be ok to share the bug fixes among all the branches so even if you > have 2 copies in your version control they could be identical. >> > > I'm not sure I understood this correctly, but this mean we cant do this > cause its to late for Karaf? > On the other hand if this means we end up in the versioning hell of the > Aries project, I'm strongly against using this versioning schema. > >> david jencks >> >> On May 23, 2014, at 2:38 PM, Achim Nierbeck <[email protected]> > wrote: >> >>> Hi Guillaume, >>> >>> I like the idea of using this plugin, and I also like the idea of > semantic >>> versioning. But I also share your concern about the strictness those >>> versions could apply on us. >>> >>> One thing we really should be aware of, we might end up in the release > hell >>> of the apache aries. >>> Another thing we should make sure, if we touch an API and do change it > we >>> (or actually any exported package) we should always increase the minor >>> version, this way we don't run into that "lockin" you have mentioned >>> regarding bugfix-version. >>> >>> If I understood semantic versioning right, this is the appropriate way. >>> >>> Given a version number MAJOR.MINOR.PATCH, increment the: >>> MAJOR version when you make incompatible API changes, >>> MINOR version when you add functionality in a backwards-compatible > manner, >>> and >>> PATCH version when you make backwards-compatible bug fixes. >>> >>> so to get back to the previous ascii-art documentations ;) >>> >>> Instead of: >>> | Karaf 4.0.0 | Karaf 4.1.0 | >>> | abc-4.0.0 | abc-4.0.1 | >>> >>> it should be: >>> | Karaf 4.0.0 | Karaf 4.1.0 | >>> | abc-4.0.0 | abc-4.1.0 | >>> >>> in case of the API change or in case of no change: >>> >>> | Karaf 4.0.0 | Karaf 4.1.0 | >>> | abc-4.0.0 | abc-4.0.0 | >>> >>> >>> regards, Achim >>> >>> >>> >>> 2014-05-23 16:19 GMT+02:00 Guillaume Nodet <[email protected]>: >>> >>>> First, we're talking about package versioning, not bundle versioning. >>>> Those are independent and the bundle version is of little use (as > usually >>>> the requirement is on the package version using a range). >>>> And this also only apply to *exported* packages. >>>> >>>> The main thing to consider is that the package content / version has > to be >>>> consistent, i.e. for a given package and version, the content of the >>>> package should be the same in all branches. >>>> >>>> So it's perfectly valid to have >>>> >>>> | Karaf 4.0.0 | Karaf 4.0.1 | Karaf 4.1.0 | >>>> | abc-4.0.0 | abc-4.0.1 | abc-4.0.1 | >>>> >>>> as long as the content of the package abc in 4.0.1 and 4.1.0 are the > same. >>>> And that goes down to the implementation too, as the interface is the >>>> contract really. >>>> >>>> The "problems" come when you have a modification which is small new >>>> feature, but fully compatible. For example, you add a method to an >>>> interface which is consumed by users (i.e. not implemented). In that > case, >>>> the semantic versioning rules lead to raising the version from 4.0.0 to >>>> 4.0.1. so let's say we release this in the 4.1.x branch. >>>> If we later find a bug that require another modification of this > interface, >>>> we need to increase the package to 4.0.2 and release that package in > both >>>> 4.0.x and 4.1.x branches, meaning we also need to backport the new > feature >>>> added with the 4.0.1 package. >>>> >>>> Note that the whole scenario is questionable, as changing the api for > a bug >>>> fix might not be considered a bug fix anymore, but i think we had to do >>>> that in the past, and I don't see how we could avoid that in the > future. >>>> If such a case happen, we'd have to decide on a case by case to either > not >>>> backport the bug that require an api change, or have to backport all > api >>>> changes (and related implementations). That's why some projects > end-up >>>> actually having a separate release cycle for each bundle. >>>> >>>> >>>> >>>> 2014-05-23 14:09 GMT+02:00 Jamie G. <[email protected]>: >>>> >>>>> I believe that last point needs to be a little clearer too ;) >>>>> >>>>> Forgive my attempt at ascii table: >>>>> >>>>> | Karaf 4.0.0 | Karaf 4.1.0 | >>>>> | abc-4.0.0 | abc-4.0.1 | >>>>> >>>>> Bug in abc is found, requiring a patch: >>>>> >>>>> | Karaf 4.0.1 | Karaf 4.1.0 | Karaf 4.1.1 | >>>>> | abc-4.0.2 | abc-4.0.1 | abc-4.0.2 | >>>>> >>>>> Would the above table describe the releases? >>>>> >>>>> Why can't we have the following, where abc-4.0.1 are the same bug fix >>>>> release? >>>>> >>>>> | Karaf 4.0.0 | Karaf 4.0.1 | Karaf 4.1.0 | >>>>> | abc-4.0.0 | abc-4.0.1 | abc-4.0.1 | >>>>> >>>>> Cheers, >>>>> Jamie >>>>> >>>>> On Thu, May 22, 2014 at 11:31 AM, Guillaume Nodet <[email protected]> >>>>> wrote: >>>>>> A recent contribution to the felix maven-bundle-plugin is that > baseline >>>>>> goal which checks and/or reports semantic versioning changes. >>>>>> It basically does a binary diff with the latest release of the same >>>>>> artifact for changes and verify that the changes are compatible with >>>> the >>>>>> semantic versioning rules. >>>>>> >>>>>> I think we should try to leverage those, but without changing our >>>> release >>>>>> lifecycle. This could be done the following way: >>>>>> * keep our artifacts versioning and release mechanism untouched >>>>>> * add the baseline plugin >>>>>> * in 4.0, exported packages would all have a version of 4.0.0 >>>>>> * follow semantic versioning for all releases > 4.0 >>>>>> >>>>>> This would mean that karaf 4.1 would have packages in version 4.0, >>>> 4.0.1 >>>>> or >>>>>> 4.1. Note that the version used for karaf (and for bundles) isn't > tied >>>>> in >>>>>> any way to the version of the packages. >>>>>> But this would also help making sure we stay compatible, and whenever >>>> we >>>>> do >>>>>> a new release (be it 4.1 or 5.0), help users with migration because > the >>>>>> package versions would not be changed unless needed. >>>>>> >>>>>> Downsides: >>>>>> * users may be impacted in the future with the fact that not all >>>>> packages >>>>>> have the same version, but this should usually be minor, as tools >>>> usually >>>>>> grab the version of the packages from the binary when writing an > import >>>>>> package. However, for those which do not leverage those features, > this >>>>>> will have an impact >>>>>> * bug fix releases may be more tricky : the main problem is that > with >>>>>> semantic versioning, once a package is released with a minor upgrade, >>>>>> there's no room left >>>>>> >>>>>> The last point needs an explanation. Let's say we have a package in >>>>>> version 4.0.0 in karaf 4. We later release karaf 4.1 with a very > minor >>>>>> addition to that package, so that this package is now in version > 4.0.1. >>>>> If >>>>>> we later find a bug which require a fix, we won't be able to > release a >>>>>> karaf 4.0.1 with a micro change in that package, so we'd have to at >>>> least >>>>>> backport the interface change from 4.1 in 4.0.1 (even if we don't >>>>> backport >>>>>> the implementation) and then do the fix with a 4.0.2 package version. >>>>>> >>>>>> Not sure this is very clear, but the last point has always been my >>>> worry >>>>>> about semantic versioning. >>>>>> >>>>>> Thoughts ? >>>>>> >>>>>> Guillaume >>>>> >>>> >>> >>> >>> >>> -- >>> >>> Apache Karaf <http://karaf.apache.org/> Committer & PMC >>> OPS4J Pax Web <http://wiki.ops4j.org/display/paxweb/Pax+Web/> Committer > & >>> Project Lead >>> blog <http://notizblog.nierbeck.de/> >>> >>> Software Architect / Project Manager / Scrum Master >>
