commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: How tell which BCEL release?
Date Fri, 08 Jul 2016 23:10:23 GMT
The Commons Parent pom includes various items in the jar manifest,
including the following:

Implementation-Version:
Specification-Version:

These are both set from project.version.



On 8 July 2016 at 23:49, dbrosIus <dbrosIus@baybroadband.net> wrote:
> Typicaly one puts the version in the manifest
>
> -------- Original message --------
> From: Mark Roberts <markro@cs.washington.edu>
> Date: 7/8/16  6:43 PM  (GMT-05:00)
> To: 'Commons Developers List' <dev@commons.apache.org>
> Subject: How tell which BCEL release?
>
> Now that BCEL 6.0 looks close, I'm wondering how a client can tell - programmatically
- which version of BCEL he is running against in order to verify it is correct.   Currently,
we at PLSE add an extra, dummy class to the release in order to do this.   Our goal is to
stop shipping our own version and use the official 6.0 release - but we still need a way to
identify the version.  We could do something hokey like looking for a recently added method
- but I would prefer something more organized.
>
> Any thoughts?
>
> Mark
>
>
>> -----Original Message-----
>> From: Benedikt Ritter [mailto:britter@apache.org]
>> Sent: Thursday, July 07, 2016 11:48 AM
>> To: Commons Developers List
>> Subject: Re: [VOTE] Release Apache Commons BCEL 6.0 based on RC7
>>
>> Thank you Mark, I'll have a look ASAP.
>>
>> Mark Roberts <markro@cs.washington.edu> schrieb am Do., 7. Juli 2016 um
>> 18:41:
>>
>> > Unit test attached to BCEL-262.
>> >
>> > Mark
>> >
>> > > -----Original Message-----
>> > > From: Gary Gregory [mailto:garydgregory@gmail.com]
>> > > Sent: Wednesday, July 06, 2016 2:26 PM
>> > > To: Commons Developers List
>> > > Subject: Re: [VOTE] Release Apache Commons BCEL 6.0 based on RC7
>> > >
>> > > I think we should wait for your unit test before we cut another (and
>> > final I
>> > > hope) RC.
>> > >
>> > > Gary
>> > >
>> > > On Wed, Jul 6, 2016 at 10:51 AM, Mark Roberts
>> > > <markro@cs.washington.edu>
>> > > wrote:
>> > >
>> > > > The patch that (was) attached to bcel-262 was correct, it just
>> > > > looks like it was never applied.  The line numbers are a little
>> > > > off now so I have just updated the patch - the new version is relative
to
>> RC7.
>> > > >
>> > > > The example failure shown in the bug report requires running
>> > > > Daikon to repro.  I will see if I can produce a reduced test case,
>> > > > but it will take a little time.
>> > > >
>> > > > I just noticed that the package coordinates change (from
>> > > > commons/bcel6 back to bcel) was applied to the trunk as well as the
RC
>> candidates.
>> > > > It this change permanent?  I really don't want to edit my sources
>> > > > back only to have to change them again at some point in the future.
>> > > >
>> > > > And here is some good (!) news:  I took the current RC7 sources,
>> > > > applied my InvokeInstruction patch, built the system, used
>> > > > shade52.xml to get the correct :-) names and inserted the
>> > > > resulting bcel.jar into the Daikon system.  It built correctly and
>> > > > passed all the regression
>> > tests. Ta-
>> > > da!
>> > > >
>> > > > Thank you,
>> > > > Mark
>> > > >
>> > > > > -----Original Message-----
>> > > > > From: Benedikt Ritter [mailto:britter@apache.org]
>> > > > > Sent: Wednesday, July 06, 2016 9:47 AM
>> > > > > To: Commons Developers List
>> > > > > Subject: Re: [VOTE] Release Apache Commons BCEL 6.0 based on
RC7
>> > > > >
>> > > > > Gary Gregory <garydgregory@gmail.com> schrieb am Mi., 6.
Juli
>> > > > > 2016 um
>> > > > > 18:37 Uhr:
>> > > > >
>> > > > > > Can you craft a unit test that makes sure the proper behavior
>> > > > > > is in
>> > > > place?
>> > > > > >
>> > > > >
>> > > > > That would be awesome. Furthermore you will have to attach the
>> > > > > patch to jira or create a GitHub PR, since the ML moes not allow
>> > > > > file
>> > > attachments.
>> > > > >
>> > > > > Benedikt
>> > > > >
>> > > > >
>> > > > > > Gary
>> > > > > >
>> > > > > > On Wed, Jul 6, 2016 at 9:16 AM, Mark Roberts
>> > > > > > <markro@cs.washington.edu>
>> > > > > > wrote:
>> > > > > >
>> > > > > > > Hmm - now I'm thinking the code for InvokeInstruction
is not
>> > correct.
>> > > > > > The
>> > > > > > > override method getClassName was added, but my patch
was not
>> > > > > applied
>> > > > > > > to
>> > > > > > the
>> > > > > > > added code.  I have attached the diff.
>> > > > > > >
>> > > > > > > Sorry for not catching this the first time.
>> > > > > > >
>> > > > > > > Thank you,
>> > > > > > > Mark
>> > > > > > >
>> > > > > > >
>> > > > > > > > -----Original Message-----
>> > > > > > > > From: Mark Roberts [mailto:markro@cs.washington.edu]
>> > > > > > > > Sent: Wednesday, July 06, 2016 6:57 AM
>> > > > > > > > To: 'Commons Developers List'
>> > > > > > > > Subject: RE: [VOTE] Release Apache Commons BCEL
6.0 based
>> > > > > > > > on
>> > > > > > > > RC7
>> > > > > > > >
>> > > > > > > > The RELEASE-NOTES.txt entry for BCEL-262 is incorrect.
 It
>> > > > > > > > should be
>> > > > > > > exactly
>> > > > > > > > the opposite.  The override was added to InvokeInstruction
>> > > > > > > > because an
>> > > > > > > array
>> > > > > > > > IS a legal operand.  The code is correct, the
throw has
>> > > > > > > > been
>> > > > removed.
>> > > > > > > >
>> > > > > > > > Mark
>> > > > > > > >
>> > > > > > > > > -----Original Message-----
>> > > > > > > > > From: Benedikt Ritter [mailto:britter@apache.org]
>> > > > > > > > > Sent: Tuesday, July 05, 2016 2:17 AM
>> > > > > > > > > To: Commons Developers List
>> > > > > > > > > Subject: Re: [VOTE] Release Apache Commons
BCEL 6.0
>> > > > > > > > > based on
>> > > > > > > > > RC7
>> > > > > > > > >
>> > > > > > > > > This vote is still pending and nobody has
voted so far.
>> > > > > > > > > Please review this RC and cast your votes!
>> > > > > > > > >
>> > > > > > > > > Thank you!
>> > > > > > > > > Benedikt
>> > > > > > > > >
>> > > > > > > > > Benedikt Ritter <britter@apache.org>
schrieb am Sa., 2.
>> > > > > > > > > Juli
>> > > > > > > > > 2016 um
>> > > > > > > > > 20:52 Uhr:
>> > > > > > > > >
>> > > > > > > > > > Hi,
>> > > > > > > > > >
>> > > > > > > > > > I'd like to release Apache Commons BCEL
6.0 based on RC7.
>> > > > > > > > > > Changes compared to RC6 are:
>> > > > > > > > > >
>> > > > > > > > > > - restored binary compatibility to a
greater degree
>> > > > > > > > > > - fixed issue BCEL-262
>> > > > > > > > > >
>> > > > > > > > > > BCEL 6.0 RC7 is available for review
here:
>> > > > > > > > > > https://dist.apache.org/repos/dist/dev/commons/bcel/
>> > > > > > > > > > (svn revision
>> > > > > > > > > > 14251)
>> > > > > > > > > >
>> > > > > > > > > > The tag is here:
>> > > > > > > > > >
>> > > > > > > > >
>> > > > > > > >
>> > > > >
>> > >
>> http://svn.apache.org/repos/asf/commons/proper/bcel/tags/BCEL_6_0_
>> > > > > > > > RC
>> > > > > > > > > 7/
>> > > > > > > > > > (svn revision 1751084)
>> > > > > > > > > >
>> > > > > > > > > > Maven artifacts are here:
>> > > > > > > > > >
>> > > > > > > >
>> > > > > https://repository.apache.org/content/repositories/orgapachecomm
>> > > > > on
>> > > > > > > > s-
>> > > > > > > > > 11
>> > > > > > > > > > 81/org/apache/bcel/bcel/6.0/
>> > > > > > > > > >
>> > > > > > > > > > These are the Maven artifacts and their
hashes
>> > > > > > > > > >
>> > > > > > > > > > bcel-6.0-javadoc.jar
>> > > > > > > > > >   (SHA1: f1e1534867a901b9ba4884e5805317635c324589)
>> > > > > > > > > > bcel-6.0-sources.jar
>> > > > > > > > > >   (SHA1: 9ba3b50aa95289d01ec119b60be68eb4c608ba1d)
>> > > > > > > > > > bcel-6.0-test-sources.jar
>> > > > > > > > > >   (SHA1: 484b29d3a73fbe0c103d85965c4fd22e6253f545)
>> > > > > > > > > > bcel-6.0-tests.jar
>> > > > > > > > > >   (SHA1: f8b5857f3245e10548ef29cf7006c045b913a199)
>> > > > > > > > > > bcel-6.0.jar
>> > > > > > > > > >   (SHA1: fe1ecaf2ba3b1f9f18cdde4f13943e3ccc1d5e69)
>> > > > > > > > > > bcel-6.0.pom
>> > > > > > > > > >   (SHA1: ea17ee1b2c28804437212970ea2d273efeb3807e)
>> > > > > > > > > >
>> > > > > > > > > > I have tested this with JDK 7, 8 using
Maven 3.3.9.
>> > > > > > > > > >
>> > > > > > > > > > Details of changes since 1.1 are in
the release notes:
>> > > > > > > > > >
>> > > > > > > > > > https://dist.apache.org/repos/dist/dev/commons/bcel/RE
>> > > > > > > > > > LEAS
>> > > > > > > > > > E-
>> > > > > > > > > NOTES.txt
>> > > > > > > > > >
>> > > > > > > > > > http://home.apache.org/~britter/commons/bcel/6.0-
>> > > RC7/chang
>> > > > > > > > > > es-
>> > > > > > > > > report.ht
>> > > > > > > > > > ml
>> > > > > > > > > >
>> > > > > > > > > > Site:
>> > > > > > > > > >
>> > > > > > > > > > http://home.apache.org/~britter/commons/bcel/6.0-RC7/
>> > > > > > > > > > (note some *relative* links are broken
and the 6.0
>> > > > > > > > > > directories are not yet created - these
will be OK
>> > > > > > > > > > once the site is
>> > > > > > > > > > deployed)
>> > > > > > > > > >
>> > > > > > > > > > Clirr Report (compared to 5.2):
>> > > > > > > > > >
>> > > > > > > > > >
>> > > > > > http://home.apache.org/~britter/commons/bcel/6.0-RC7/clirr-
>> report.
>> > > > > > ht
>> > > > > > > > > > ml
>> > > > > > > > > >
>> > > > > > > > > > Note that Clirr reports several errors.
>> > > > > > > > > > These are considered OK for the reasons
stated below.
>> > > > > > > > > > These exceptions are also noted in the
Changes and
>> > > > > > > > > > Release
>> > > > Notes.
>> > > > > > > > > >
>> > > > > > > > > > Errors reported:
>> > > > > > > > > > - methods added to org.apache.bcel.classfile.Visitor
>> > > > > > > > > > interface: OK because that does not
affect binary
>> > > > compatibility.
>> > > > > > > > > > - Removed java.io.Serializable from
all classes: OK,
>> > > > > > > > > > because we don't expect anybody to rely
on
>> > > > > > > > > > serialization for BCEL classes
>> > > > > > > > > > - Return type of method 'public java.lang.Object
>> > > > getElementAt(int)'
>> > > > > > > > > > has been changed to java.lang.String
in class
>> > > > > > > > > > org.apache.bcel.verifier.VerifierFactoryListModel:
OK,
>> > > > > > > > > > because this class is part of an UI
application and
>> > > > > > > > > > for this reason should only used by
>> > > > > > > > > Swing.
>> > > > > > > > > >
>> > > > > > > > > > RAT Report:
>> > > > > > > > > >
>> > > > > > > > > > http://home.apache.org/~britter/commons/bcel/6.0-RC7/r
>> > > > > > > > > > at-
>> > > > > > > > report.html
>> > > > > > > > > >
>> > > > > > > > > > KEYS:
>> > > > > > > > > >   https://www.apache.org/dist/commons/KEYS
>> > > > > > > > > >
>> > > > > > > > > > Please review the release candidate
and vote. This
>> > > > > > > > > > vote will close no sooner that 72 hours
from now, i.e.
>> > > > > > > > > > sometime after
>> > > > > > > > > > 21:00 CEST 05-July
>> > > > > > > > > > 2016
>> > > > > > > > > >
>> > > > > > > > > > [ ] +1 Release these artifacts [ ] +0
OK, but...
>> > > > > > > > > > [ ] -0 OK, but really should fix...
>> > > > > > > > > > [ ] -1 I oppose this release because...
>> > > > > > > > > >
>> > > > > > > > > > Thanks!
>> > > > > > > > > > Benedikt
>> > > > > > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > > ------------------------------------------------------------
>> > > > > > > ----
>> > > > > > > ----
>> > > > > > > - To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > > > > > > For additional commands, e-mail: dev-
>> help@commons.apache.org
>> > > > > > >
>> > > > > >
>> > > > > >
>> > > > > >
>> > > > > > --
>> > > > > > E-Mail: garydgregory@gmail.com | ggregory@apache.org Java
>> > > > > > Persistence with Hibernate, Second Edition
>> > > > > > <http://www.manning.com/bauer3/> JUnit in Action,
Second
>> > > > > > Edition <http://www.manning.com/tahchiev/>
>> > > > > > Spring Batch in Action <http://www.manning.com/templier/>
>> > > > > > Blog: http://garygregory.wordpress.com
>> > > > > > Home: http://garygregory.com/
>> > > > > > Tweet! http://twitter.com/GaryGregory
>> > > > > >
>> > > >
>> > > >
>> > > > ------------------------------------------------------------------
>> > > > --- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > > > For additional commands, e-mail: dev-help@commons.apache.org
>> > > >
>> > > >
>> > >
>> > >
>> > > --
>> > > E-Mail: garydgregory@gmail.com | ggregory@apache.org Java
>> > > Persistence with Hibernate, Second Edition
>> > > <http://www.manning.com/bauer3/> JUnit in Action, Second Edition
>> > > <http://www.manning.com/tahchiev/>
>> > > Spring Batch in Action <http://www.manning.com/templier/>
>> > > Blog: http://garygregory.wordpress.com
>> > > Home: http://garygregory.com/
>> > > Tweet! http://twitter.com/GaryGregory
>> >
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > For additional commands, e-mail: dev-help@commons.apache.org
>> >
>> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message