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.

Sign up for GitHub

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

Jump to bottom

Install perl XML::Parser module on test machines #622

Closed
llxia opened this issue Dec 5, 2018 · 2 comments
Closed

Install perl XML::Parser module on test machines #622

llxia opened this issue Dec 5, 2018 · 2 comments
Labels
enhancement invalid

Comments

@llxia
Copy link

llxia commented Dec 5, 2018

Since we want to update testKitGen to use standard XML parser to parse *.xml files, we need to install XML::Parser module on test machines.

XML Parser is in the Adopt Unix playbooks, but we need to make sure it is installed on all test machines. See comment from Adam:
eclipse-openj9/openj9#499 (comment)
jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this issue Dec 5, 2018
@jdekonin
Add XML::parser to AIX playbook
ed4293e 
* related adoptium#622

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
@jdekonin jdekonin mentioned this issue Dec 5, 2018
Merged
@jdekonin
Copy link
Contributor

jdekonin commented Dec 5, 2018

I think cygwin installs this by default when also installing perl. At least that appears to be the case on the systems that I have that are setup with the windows playbook. /usr/lib/perl5/vendor_perl/5.26/x86_64-cygwin-threads/XML/Parser.pm is where its put.

But Strawberry perl is also a role on windows, which version is this needed under? Which version of perl should be used for windows?

jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this issue Jan 18, 2019
@jdekonin
Add XML::parser to AIX playbook
5fa9202 
* related adoptium#622

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this issue Jan 18, 2019
@jdekonin
Add XML::parser to AIX playbook
149921e 
* related adoptium#622
* move env settings to base of playbook
* add check for XML::Parser
* install if not present

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this issue Jan 30, 2019
@jdekonin
Add XML::parser to AIX playbook
0ad2cf6 
* related adoptium#622
* move env settings to base of playbook
* add check for XML::Parser
* install if not present

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
karianna pushed a commit that referenced this issue Jan 31, 2019
@jdekonin @karianna
Add XML::parser to AIX playbook (#623)
d094d72 
* related #622
* move env settings to base of playbook
* add check for XML::Parser
* install if not present

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
@smlambert smlambert mentioned this issue Feb 22, 2019
Closed
@sxa sxa added this to the Backlog milestone Mar 7, 2019
@llxia
Copy link
Author

llxia commented Sep 9, 2019

Close this issue as we are going to switch to use TKG for java soon. Perl module is no longer needed.

@llxia llxia closed this as completed Sep 9, 2019
@karianna karianna removed this from the Backlog milestone Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement invalid
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@karianna @sxa @jdekonin @llxia