Skip to content

Conversation

@georgweiss
Copy link
Contributor

Tweaks to pom file due to changed behaviour of the javadoc tool. A change in java source and target version is also needed to compile successfully.

<name>OSS Snapshots</name>
<url>https://oss.sonatype.org/content/repositories/snapshots</url>
</repository>
<!-- Temporary - needed for old snapshots of ChannelfinderAPI -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this comment? It still applies.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, if it still applies then it's not temporary?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me explain.
The ChannelFinder API has changed. The only place where Maven can find the old versions is the deprecated EPICS repository on SourceForge.
As soon as the directoryService example has been updated to use the current API of ChannelFinder, the pointer to this deprecated repository should be removed.

Is that "temporary"? Do you have a better term?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then I think the comment should state what you just wrote. Plus the version numbers for the "old" and "current" ChannelFinder API.

@ralphlange
Copy link
Contributor

In general, I prefer unrelated changes being in separate commits, but as in this case this is really minor, I will merge anyway.

@ralphlange ralphlange merged commit 5940cc7 into master Jun 26, 2018
@georgweiss
Copy link
Contributor Author

Thanks.

@ralphlange ralphlange deleted the issue_1 branch June 26, 2018 12:41
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