Skip to content

Conversation

@stain
Copy link
Member

@stain stain commented Feb 26, 2015

Fixes JENA-849 in updated jena-maven-tools/demo

.. tries to not list the version numbers explicitly as they would go out of date.

@stain stain changed the title Jena 849 maven tools demo JENA-849 maven tools demo Feb 26, 2015
@stain stain changed the title JENA-849 maven tools demo JENA-849 maven tools demo versions Feb 26, 2015
@afs
Copy link
Member

afs commented Feb 26, 2015

Relationship to #37? To first approximation, this includes all of #37 so should #37 by closed?

@stain
Copy link
Member Author

stain commented Feb 26, 2015

Uhm.. well, it accidentally got based on #37 as that was the plugin version I updated it for, but in the final version the demo/ also works straight of master - so I'll rebase this.

.. specially the new groupId/artifactId
(why isn't it jena-maven-plugin ?)

.. leaving <plugin><version> blank and RELEASE
for jena-core.

Ideally this should be proper versions updated by the release-plugin,
but then we would need to change to multiple modules.
@stain stain force-pushed the JENA-849-maven-tools-demo branch from a6db31d to 99ca3fc Compare February 26, 2015 13:39
@stain
Copy link
Member Author

stain commented Feb 26, 2015

Now independent of #37

@afs
Copy link
Member

afs commented Feb 26, 2015

Thanks.

@stain
Copy link
Member Author

stain commented Mar 23, 2015

Anything blocking this from going in? Sorry I forgot to check if it was applied for the last release..

@asfgit asfgit merged commit 99ca3fc into apache:master Apr 20, 2015
asfgit pushed a commit that referenced this pull request Apr 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants