Skip to content

Conversation

@jglick
Copy link
Member

@jglick jglick commented Jun 5, 2018

Picks up jenkinsci/apache-httpcomponents-client-4-api-plugin#9 so that network problems need not be fatal. Follows up #100.

Copy link
Member

@ikedam ikedam left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@jglick
Let me know about jenkinsci/apache-httpcomponents-client-4-api-plugin#9 . It says that RobustHTTPClient is a beta API. And it's annotated with Restricted(Beta.class). Will the annotation be removed when releasing apache-httpcomponents-client-4-api-plugin-4.5.5-2.2? If not, how Restricted(Beta.class) works when this request is merged?

As far as I had a quick look on Beta.class, the check by maven should fail without useBeta=true, but it looks succeed even without useBeta=true for apache-httpcomponents-client-4-api-plugin.

@jglick
Copy link
Member Author

jglick commented Jun 15, 2018

Will the annotation be removed

Not that quickly, no.

it looks succeed even without useBeta=true for apache-httpcomponents-client-4-api-plugin

The property is set on the plugin consuming the beta API.

Copy link
Member

@svanoort svanoort left a comment

Choose a reason for hiding this comment

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

I do not have enough 👍 to give in approval of this. Glad to see it.

@ikedam
Copy link
Member

ikedam commented Jun 19, 2018

it looks succeed even without useBeta=true for apache-httpcomponents-client-4-api-plugin
The property is set on the plugin consuming the beta API.

Oh, I see. I misunderstood that.

It looks annotated as beta for VirtualFile#toExternalURL is beta.
https://github.com/jenkinsci/jenkins/blob/jenkins-2.121/core/src/main/java/jenkins/util/VirtualFile.java#L352

@ikedam ikedam merged commit 671851b into jenkinsci:master Jun 19, 2018
@jglick jglick deleted the RobustHTTPClient-JENKINS-50597 branch June 19, 2018 17:59
@jglick
Copy link
Member Author

jglick commented Jun 21, 2018

@ikedam do you have a plan to release this? jenkinsci/jenkins#3480 remains on hold until then.

@ikedam
Copy link
Member

ikedam commented Jun 21, 2018

@jglick
I plan to make a new release after resolving #105.

@ikedam
Copy link
Member

ikedam commented Jun 30, 2018

@jglick I released copyartifact-1.41 with this change.

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.

5 participants