Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for WFCORE-4402, Upgrade CLI to use aesh 2.3 #3725

Merged
merged 2 commits into from Apr 14, 2019

Conversation

jfdenise
Copy link
Contributor

@jfdenise jfdenise commented Apr 5, 2019

Fix for: https://issues.jboss.org/browse/WFCORE-4402 and https://issues.jboss.org/browse/WFCORE-4408

  • Aesh release contains jansi upgrade needed to fix dll copy on Windows.
  • jansi upgrade to the one expected by aesh.

@jmesnil jmesnil added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 8, 2019
@jfdenise
Copy link
Contributor Author

jfdenise commented Apr 8, 2019

@jmesnil , I realize that Wildfly has its own version of jansi, I also need to upgrade this dependency.

@jfdenise
Copy link
Contributor Author

jfdenise commented Apr 8, 2019

@darranl , please wait before to merge this one, it is incomplete.

@darranl darranl removed the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 8, 2019
Copy link
Contributor

@darranl darranl left a comment

Choose a reason for hiding this comment

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

Add missing dependencies.

@jfdenise
Copy link
Contributor Author

jfdenise commented Apr 8, 2019

@darranl , @jmesnil I just pushed the jansi upgrade.

@darranl darranl added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 9, 2019
@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Apr 9, 2019
@darranl
Copy link
Contributor

darranl commented Apr 9, 2019

@jfdenise The Linux JDK8 job appears a couple of times now to have a number of identity operation failures and a hang, as this PR is in relation to UI component upgrades that are likely involved in these test scenarios are you able to take a look?

@darranl darranl removed the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 9, 2019
@jfdenise
Copy link
Contributor Author

@darranl , I don't see evidence of usage of aesh/jansi in these tests, where do you see these UI components in use in these tests?

@darranl
Copy link
Contributor

darranl commented Apr 10, 2019

Another PR has the same errors so looks like not related to this component upgrade #3736

@darranl darranl added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 10, 2019
@bstansberry bstansberry merged commit ca8613e into wildfly:master Apr 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
5 participants