On 7 November 2017 at 16:41, Keith W <[email protected]> wrote: > As Alex says, we opted to keep the bundle names, directories structure > within bundles and maven artefact names unchanged, The reasoning was > that the change would need to ripple through the users job control > scripts, distribution systems etc. If there are strongly held opinions > let's hear them now. If a rename is going happen, I think it would > be better now. > > >
As I said before, it's not a blocker for me, just seems a little weird. For the maven artefacts I think there's a reasonably strong case for retaining the same artefact names. For the apache distributions is seems a little weirder... and the qpid-java-7.0.0 directory in the source bundle seems particularly out of place. I agree on "if a rename is going to happen, I think it would be better now." and I certainly feel a little strange about keeping to the qpid-java-x.x.x directory going forward... the others seem less jarring. -- Rob > > On 7 November 2017 at 16:14, Oleksandr Rudyy <[email protected]> wrote: > > Rob, > > > > The binary bundle names and jar names were left unchanged in v7.0. > > They are the same is in 6.x. That might help people having downloads > > automated. We also publish binary bundles into maven. We agreed > > before to keep the maven artefacts unchanged. Though, I would also > > prefer to name bundles using qpid-broker-j-.*. I personally even would > > vote to rename broker jars into qpid-broker-j-.*. The source bundle > > was named in similar way as the source bundle for qpid-java but we > > missed to rename directory "qpid-java-7.0.0" inside of the source > > bundle. It is definitely a bug. Though, nothing stops us from > > repackaging of the source bundle and renaming folder there for v7.0. > > I'll raise JIRA about renaming distribution bundles into > > qpid-broker-j-.*. > > > > Kind Regards, > > Alex > > > > > > On 7 November 2017 at 15:29, Rob Godfrey <[email protected]> > wrote: > >> On 7 November 2017 at 15:14, Oleksandr Rudyy <[email protected]> wrote: > >> > >>> Hi all, > >>> > >>> I've built Qpid Broker-J 7.0.0 RC2. The following issues have been > resolved > >>> since RC1: > >>> > >>> * QPID-7998 - [Broker-J] Allow global shared subscriptions but discard > >>> their links on detach > >>> * QPID-8019 - [Broker-J][WMC] Change link to broker documentation in > web > >>> management console > >>> > >>> Qpid Broker-J 7.0 brings the following major changes: > >>> > >>> * Improved AMQP 1.0 support > >>> * Support for AMQP JMS Mapping Version 1.0 WD09 > >>> * Support for JMS 2.0 shared subscriptions (QPIDJMS-220) > >>> * Improved message conversion for applications using different AMQP > >>> protocols > >>> * Improved direct memory management > >>> > >>> The list of new features, defect fixes and improvements can be found in > >>> Jira: > >>> https://issues.apache.org/jira/issues/?jql=project%20% > >>> 3D%20QPID%20AND%20fixVersion%20%3D%20qpid-java-broker-7.0.0 > >>> > >>> The source and binary archives can be grabbed from here: > >>> https://dist.apache.org/repos/dist/dev/qpid/broker-j/7.0.0-rc2 > >>> > >>> > >> Out of interest, is there a reason why these archives are named > >> "qpid-broker-7.0.0*", with the src then being within a "qpid-java-7.0.0" > >> directory? Would it not make more sense for these to both be > >> "qpid-broker-j-7.0.0"? > >> > >> Certainly not a blocker for me, just seems a little inconsistent. > >> > >> -- Rob > >> > >> > >>> Those files and the other maven artifacts are also staged for now at: > >>> https://repository.apache.org/content/repositories/orgapacheqpid-1120 > >>> > >>> Please test and cast your vote accordingly. > >>> > >>> Kind regards > >>> > >>> P.S. If you want to test it out using maven, you can temporarily add > this > >>> to your poms to access > >>> the staging repo: > >>> > >>> <repositories> > >>> <repository> > >>> <id>staging</id> > >>> <url> > >>> https://repository.apache.org/content/repositories/orgapacheqpid-1120 > >>> </url> > >>> </repository> > >>> </repositories> > >>> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
